Prtksxna has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/165662

Change subject: Revert "buttons: Update focus for quiet buttons"
......................................................................

Revert "buttons: Update focus for quiet buttons"

This reverts commit 778ecee16fe38c27c4e313a5bea9c65f6bb65a53.

The implementation of the new design specification
is imperfect. Zooming into the element shows that
the borders aren't equal on all sides and there is
a different colored border on the top and bottom.

Change-Id: Ic8c40498abe9201bdf4e740737467d2d88211442
---
M resources/src/mediawiki.less/mediawiki.ui/mixins.less
1 file changed, 3 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/62/165662/1

diff --git a/resources/src/mediawiki.less/mediawiki.ui/mixins.less 
b/resources/src/mediawiki.less/mediawiki.ui/mixins.less
index 17fcc60..d3220cc 100644
--- a/resources/src/mediawiki.less/mediawiki.ui/mixins.less
+++ b/resources/src/mediawiki.less/mediawiki.ui/mixins.less
@@ -44,8 +44,8 @@
 
        &:focus {
                box-shadow:inset 0px 0px 0px 3px rgba(0, 0, 0, 20%);
+//             box-shadow: 0 1px rgba(0, 0, 0, 10%), inset 0 -3px rgba(0, 0, 
0, 20%);
                outline: none;
-
                // remove outline in Firefox
                &::-moz-focus-inner {
                        border-color: transparent;
@@ -107,21 +107,11 @@
        // constructive/progressive/destructive color on hover and active.
        color: @colorButtonText;
 
-       &:hover {
+       &:hover,
+       &:focus {
                // lessphp doesn't implement tint, see above
                // color: tint(@textColor, 20%);
                color: mix(#fff, @textColor, 20%);
-       }
-
-       &:focus {
-               box-shadow:inset 0px 0px 0px 3px rgba(0, 0, 0, 20%);
-               outline: none;
-
-               // remove outline in Firefox
-               &::-moz-focus-inner {
-                       border-color: transparent;
-               }
-
        }
 
        &:active,

-- 
To view, visit https://gerrit.wikimedia.org/r/165662
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic8c40498abe9201bdf4e740737467d2d88211442
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: wmf/1.25wmf2
Gerrit-Owner: Prtksxna <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to