]> git.ipfire.org Git - thirdparty/bootstrap.git/commit
Remove explicit suppression of focus outline
authorPatrick H. Lauke <redux@splintered.co.uk>
Tue, 9 Feb 2021 18:19:31 +0000 (18:19 +0000)
committerMark Otto <otto@github.com>
Thu, 11 Feb 2021 03:35:52 +0000 (19:35 -0800)
commitdf79aad5e449abb14168668150902c45f5d29efa
treeac34fd8ee62c02f726a2bfc48a1fd129c60b7d48
parent4c7a3e8adf73bf55a5fba5905b356468be17bc0e
Remove explicit suppression of focus outline

It's unclear what the reason for first introducing the original hack here (for `[tabindex="-1"]:focus {...}`) was. Seems something that may have been useful/necessary in SuitCSS, but don't think BS ever relied on this. https://github.com/twbs/bootstrap/issues/18330
It's since been modified to only apply when the browser wouldn't apply a visible outline anyway based on its own heuristics (the `:not(:focus-visible)` part) https://github.com/twbs/bootstrap/pull/28437/

But now, thinking this through more...in browsers that do support this pseudo-selector, what this is essentially saying is redundant: don't apply outline in cases where a `tabindex="-1"` element receives focus but the browser wouldn't normally apply focus outline". at best, this is unnecessary. at worst, this actually overrides things an author may explicitly be trying to do with adding `:focus { outline: ... }` explicitly.
scss/_reboot.scss