From 3efd6ef5fdd921b522fc3de2aedcad0d028a792a Mon Sep 17 00:00:00 2001 From: Prateek Saxena Date: Thu, 5 Mar 2015 13:57:24 +0530 Subject: [PATCH] mw.ui: button: Update focus state The focus and hover states were the same. Updating it according to the design specification - https://phabricator.wikimedia.org/M31 Bug: T88449 Change-Id: Ibdd5e1553174bfde82af3a23b746a3918b8f0416 --- resources/src/mediawiki.less/mediawiki.ui/mixins.less | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/resources/src/mediawiki.less/mediawiki.ui/mixins.less b/resources/src/mediawiki.less/mediawiki.ui/mixins.less index 92d0a76977..2d68457204 100644 --- a/resources/src/mediawiki.less/mediawiki.ui/mixins.less +++ b/resources/src/mediawiki.less/mediawiki.ui/mixins.less @@ -36,11 +36,16 @@ .button-colors(@bgColor) { background: @bgColor; - &:hover, - &:focus { + &:hover { // The inner bottom bevel should match the active background color. box-shadow: 0 1px rgba(0, 0, 0, 10%), inset 0 -3px rgba(0, 0, 0, 20%); border-bottom-color: mix(#000, @bgColor, 20%); + } + + &:focus { + border-color: rgba(0,0,0,0.2); + box-shadow: inset 0 0 0 1px rgba(0,0,0,0.2); + outline: none; // remove outline in Firefox &::-moz-focus-inner { -- 2.20.1