Skip to content

Commit

Permalink
feat(focus-visible): drop outline removals everywhere
Browse files Browse the repository at this point in the history
  • Loading branch information
ffoodd committed May 27, 2020
1 parent 0ab40fb commit 982dcec
Show file tree
Hide file tree
Showing 8 changed files with 1 addition and 14 deletions.
1 change: 0 additions & 1 deletion scss/_buttons.scss
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@

&:focus,
&.focus {
outline: 0;
box-shadow: $btn-focus-box-shadow;
}

Expand Down
1 change: 0 additions & 1 deletion scss/_carousel.scss
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,6 @@
&:focus {
color: $carousel-control-color;
text-decoration: none;
outline: 0;
opacity: $carousel-control-hover-opacity;
}
}
Expand Down
5 changes: 0 additions & 5 deletions scss/_modal.scss
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,6 @@
width: 100%;
height: 100%;
overflow: hidden;
// Prevent Chrome on Windows from adding a focus outline. For details, see
// https:/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:/twbs/bootstrap/issues/17695
Expand Down Expand Up @@ -88,8 +85,6 @@
border: if($modal-content-border-color == null, null, $modal-content-border-width solid $modal-content-border-color); // Boosted mod
@include border-radius($modal-content-border-radius);
@include box-shadow($modal-content-box-shadow-xs);
// Remove focus outline from opened modal
outline: 0;
}

// Modal background
Expand Down
1 change: 0 additions & 1 deletion scss/_navbar.scss
Original file line number Diff line number Diff line change
Expand Up @@ -205,7 +205,6 @@

&:focus {
text-decoration: none;
outline: 0;
box-shadow: 0 0 0 $navbar-toggler-focus-width;
}

Expand Down
3 changes: 1 addition & 2 deletions scss/forms/_form-check.scss
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@

&:focus {
border-color: $form-check-input-focus-border;
outline: 0;
box-shadow: $form-check-input-focus-box-shadow;
}

Expand Down Expand Up @@ -166,7 +165,7 @@
}

&:focus {
outline: 0; // @TODO: custom focus
// @TODO: custom focus
}

&:active {
Expand Down
1 change: 0 additions & 1 deletion scss/forms/_form-control.scss
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
color: $input-focus-color;
background-color: $input-focus-bg;
border-color: $input-focus-border-color;
outline: 0;
@if $enable-shadows {
@include box-shadow($input-box-shadow, $input-focus-box-shadow);
} @else {
Expand Down
2 changes: 0 additions & 2 deletions scss/forms/_form-range.scss
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,6 @@

&:focus {
// Boosted mod: better focus visibility
outline: none;

// Pseudo-elements must be split across multiple rulesets to have an effect.
// No box-shadow() mixin for focus accessibility.
&::-webkit-slider-thumb {
Expand Down
1 change: 0 additions & 1 deletion scss/forms/_form-select.scss
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@

&:focus {
border-color: $form-select-focus-border-color;
outline: 0;
@if $enable-shadows {
@include box-shadow($form-select-box-shadow, $form-select-focus-box-shadow);
} @else {
Expand Down

0 comments on commit 982dcec

Please sign in to comment.