Index: chrome/browser/resources/settings/settings_shared_css.html |
diff --git a/chrome/browser/resources/settings/settings_shared_css.html b/chrome/browser/resources/settings/settings_shared_css.html |
index aa9e0ae1ace9b2055bc84886ae02c13b04277ef4..2debced03017687ef9209d3fc0ea286671afd903 100644 |
--- a/chrome/browser/resources/settings/settings_shared_css.html |
+++ b/chrome/browser/resources/settings/settings_shared_css.html |
@@ -73,6 +73,11 @@ |
text-decoration: none; |
} |
+ a[href][tabindex='-1'] { |
dschuyler
2017/04/18 23:22:44
Please add a comment about why this is here (or wh
Dan Beam
2017/04/20 00:24:34
this is kindddda ghetto
scottchen
2017/04/20 18:41:50
Done.
scottchen
2017/04/20 18:41:50
I checked code-search and no other anchors seem to
Dan Beam
2017/04/20 18:49:41
a.settings-box?
Dan Beam
2017/04/20 18:52:54
ah, so i guess they're not all .setting-box
maybe
dschuyler
2017/04/20 18:58:17
How about
a[actionable]
Is that a 1:1 with wanting
Dan Beam
2017/04/20 21:15:40
on that note: why do we need [actionable] on a[hre
scottchen
2017/04/20 21:48:34
I kept it around because of this:
https://cs.chro
|
+ color: var(--paper-grey-900); |
Dan Beam
2017/04/20 00:28:35
is this value being copied from somewhere else?
scottchen
2017/04/20 18:41:50
Body text color's hard-coded in settings_ui -
http
scottchen
2017/04/20 18:43:23
Sorry - I meant I swapped it to use --primary-text
|
+ outline: none; |
+ } |
+ |
/* There are three main button styles, .primary-button, .secondary-button, |
* and .tertiary-button. The primary is the action button (e.g. "edit", |
* "delete") while the secondary is often a "Cancel" button. A tertiary |