jenkins-bot has submitted this change and it was merged.
Change subject: No-js css tweaks
......................................................................
No-js css tweaks
* post and topic flyouts start small and transition width on hover
* reposition timestamps slightly to accomidate always visible flyouts
* shrink non-active empty textareas
Bug: 58019
Mingle: 589
Change-Id: Id0c651e2cc20ee2fc7a9adbffabebac270e7666d
---
M Resources.php
M modules/base/styles/actionbox.less
A modules/discussion/styles/nojs.less
3 files changed, 49 insertions(+), 0 deletions(-)
Approvals:
Matthias Mullie: Looks good to me, but someone else must approve
EBernhardson: Looks good to me, approved
jenkins-bot: Verified
diff --git a/Resources.php b/Resources.php
index 79cac71..5fba780 100644
--- a/Resources.php
+++ b/Resources.php
@@ -55,6 +55,7 @@
'discussion/styles/post.less',
'discussion/styles/collapse.less',
'discussion/styles/modified.less',
+ 'discussion/styles/nojs.less',
),
'scripts' => array(
'discussion/ui.js',
diff --git a/modules/base/styles/actionbox.less
b/modules/base/styles/actionbox.less
index 938cdb6..ddb7944 100644
--- a/modules/base/styles/actionbox.less
+++ b/modules/base/styles/actionbox.less
@@ -59,8 +59,24 @@
display: none;
}
+ .flow-tipsy {
+ width: 100%;
+ }
+
.flow-tipsy-flyout {
display: block;
+
+ // give the action menu an animated width
+ ul li .mw-ui-button {
+ transition: width 0.1s, text-indent 0.1s, padding-right
0.1s;
+ }
+
+ // unless when hovered, shorten buttons to only display the icon
+ &:not(:hover) ul li .mw-ui-button {
+ width: 0;
+ text-indent: -9999px;
+ padding-right: 4px;
+ }
}
}
diff --git a/modules/discussion/styles/nojs.less
b/modules/discussion/styles/nojs.less
new file mode 100644
index 0000000..94280f7
--- /dev/null
+++ b/modules/discussion/styles/nojs.less
@@ -0,0 +1,32 @@
+.client-nojs .flow-container {
+ // position tipsy window in upper right hand corner
+ .flow-tipsy {
+ position: absolute;
+ top: 0;
+ right: 0;
+ }
+
+ // flyout doesn't need to be pushed down below the icon; there's no JS,
so
+ // no event handlers when clicking the flyout; so the flyout will
always be
+ // displayed
+ .flow-tipsy-flyout {
+ position: inherit;
+ top: 0;
+ }
+
+ // shrink textarea's until they are focused
+ .flow-post-container,
+ .flow-new-topic-container {
+ textarea:empty:not(:focus) {
+ height: 34px;
+ }
+ }
+
+ // make room for the always visible flyout
+ .flow-element-container .flow-datestamp {
+ right: 50px;
+ }
+ .flow-post-content {
+ padding-right: 45px;
+ }
+}
--
To view, visit https://gerrit.wikimedia.org/r/103763
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id0c651e2cc20ee2fc7a9adbffabebac270e7666d
Gerrit-PatchSet: 5
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: EBernhardson <ebernhard...@wikimedia.org>
Gerrit-Reviewer: EBernhardson <ebernhard...@wikimedia.org>
Gerrit-Reviewer: Matthias Mullie <mmul...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
