From b475720e427907e5eb7e6ef3259ec3556f7e8bba Mon Sep 17 00:00:00 2001 From: Christian Oliff Date: Wed, 5 Mar 2025 21:29:23 +0900 Subject: [PATCH] Remove `outline: 0` from modals This was [added about 12 years ago](https://github.com/twbs/bootstrap/pull/10951) and is no longer needed from what I can tell. I've tested with Chrome 133 and Edge 122 on Windows 11. The modal itself is never focused anyway, when the modal opens the focus is only within the modal. --- scss/_modal.scss | 3 --- 1 file changed, 3 deletions(-) diff --git a/scss/_modal.scss b/scss/_modal.scss index ee58032ee8..24bce41a66 100644 --- a/scss/_modal.scss +++ b/scss/_modal.scss @@ -41,9 +41,6 @@ height: 100%; overflow-x: hidden; overflow-y: auto; - // Prevent Chrome on Windows from adding a focus outline. For details, see - // https://github.com/twbs/bootstrap/pull/10951. - outline: 0; // We deliberately don't use `-webkit-overflow-scrolling: touch;` due to a // gnarly iOS Safari bug: https://bugs.webkit.org/show_bug.cgi?id=158342 // See also https://github.com/twbs/bootstrap/issues/17695