Bug 1518422 - Implement focus ring for panel footer buttons across platforms. r=ntim

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

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Dão Gottwald 2019-01-08 10:26:20 +00:00
parent 4c7d3d1ae5
commit 2f87c412d5
6 changed files with 11 additions and 10 deletions

View File

@ -14,7 +14,6 @@
@keyfocus@ @itemFocused@,
@keyfocus@ #downloadsSummary:focus,
@keyfocus@ .downloadsPanelFooterButton:focus,
.downloadButton:focus {
outline: 1px -moz-dialogtext dotted;
outline-offset: -1px;

View File

@ -6,8 +6,7 @@
/*** Panel and outer controls ***/
@keyfocus@ #downloadsSummary:focus,
@keyfocus@ .downloadsPanelFooterButton:focus {
@keyfocus@ #downloadsSummary:focus {
outline: 2px -moz-mac-focusring solid;
outline-offset: -2px;
}

View File

@ -63,6 +63,10 @@
padding: 0;
}
@notKeyfocus@ .downloadsPanelFooterButton:-moz-focusring {
outline: none;
}
#downloadsSummary {
/* Reserve the same space as the button and separator in download items. */
padding-inline-end: 59px;

View File

@ -10,8 +10,7 @@
outline-offset: -5px;
}
@notKeyfocus@ #downloadsSummary:-moz-focusring,
@notKeyfocus@ .downloadsPanelFooterButton:-moz-focusring {
@notKeyfocus@ #downloadsSummary:-moz-focusring {
outline: none;
}

View File

@ -7,11 +7,6 @@
%include ../../shared/notification.inc.css
%include ../../shared/popupnotification.inc.css
.popup-notification-button:-moz-focusring {
outline: 1px dotted;
outline-offset: -5px;
}
.popup-notification-secondary-button:not([hidden="true"]) ~ .popup-notification-primary-button:not([default]) {
border-inline-start: 1px solid var(--panel-separator-color);
}

View File

@ -65,6 +65,11 @@
color: var(--panel-disabled-color) !important;
}
.panel-footer > button:-moz-focusring {
outline: 1px dotted;
outline-offset: -3px;
}
.panel-footer > button:not([disabled]):hover {
background-color: var(--arrowpanel-dimmed);
}