Skip to content

Commit

Permalink
Bug 1476296 - Remove confusing devtools toggle button focused state r…
Browse files Browse the repository at this point in the history
…=Honza,fvsch

Differential Revision: https://phabricator.services.mozilla.com/D63292

--HG--
extra : moz-landing-system : lando
  • Loading branch information
vtstydev committed Feb 24, 2020
1 parent 85c3d9e commit 675474d
Showing 1 changed file with 4 additions and 6 deletions.
10 changes: 4 additions & 6 deletions devtools/client/themes/common.css
Original file line number Diff line number Diff line change
Expand Up @@ -428,7 +428,7 @@ checkbox:-moz-focusring {
color: var(--theme-toolbarbutton-color);
}

.devtools-togglebutton:not([aria-pressed="true"]):hover {
.devtools-togglebutton:not([aria-pressed="true"]):hover:not(:active) {
background-color: var(--theme-toolbarbutton-hover-background);
}

Expand All @@ -447,19 +447,17 @@ checkbox:-moz-focusring {
background-color: var(--toolbarbutton-checked-focus-background);
}

.devtools-togglebutton[aria-pressed="true"],
.devtools-togglebutton:focus {
.devtools-togglebutton[aria-pressed="true"] {
background-color: var(--theme-toolbarbutton-checked-background);
color: var(--theme-toolbarbutton-checked-color);
}

.devtools-togglebutton[aria-pressed="true"]:hover {
.devtools-togglebutton[aria-pressed="true"]:hover:not(:active) {
background: var(--theme-toolbarbutton-checked-hover-background);
color: var(--theme-toolbarbutton-checked-hover-color);
}

.devtools-togglebutton:active,
.devtools-togglebutton:hover:active {
.devtools-togglebutton:active {
background: var(--theme-toolbarbutton-active-background);
color: var(--theme-toolbarbutton-active-color);
}
Expand Down

0 comments on commit 675474d

Please sign in to comment.