Skip to content

Commit

Permalink
fix(material-experimental/mdc-dialog): remove extra outline in high c…
Browse files Browse the repository at this point in the history
…ontrast mode (angular#24140)

MDC have started including their own `border` in high contrast mode so our `outline` isn't necessary anymore.
  • Loading branch information
crisbeto authored Jan 7, 2022
1 parent a5fb8f8 commit 3f727a2
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions src/material-experimental/mdc-dialog/dialog.scss
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
@use '@material/dialog' as mdc-dialog;
@use '../mdc-helpers/mdc-helpers';
@use './mdc-dialog-structure-overrides';
@use '../../cdk/a11y';

// Dialog content max height. This has been copied from the standard dialog
// and is needed to make the dialog content scrollable.
Expand All @@ -17,10 +16,6 @@ $mat-dialog-button-horizontal-margin: 8px !default;
// The dialog container is focusable. We remove the default outline shown in browsers.
.mat-mdc-dialog-container {
outline: 0;

@include a11y.high-contrast(active, off) {
outline: solid 1px;
}
}

// MDC sets the display behavior for title and actions, but not for content. Since we support
Expand Down

0 comments on commit 3f727a2

Please sign in to comment.