Skip to content

Commit

Permalink
fix(material-experimental/mdc-dialog): blending in with background in…
Browse files Browse the repository at this point in the history
… high contrast mode (#20387)

The MDC dialog was blending in with the page background, because it doesn't have a border or an outline.
  • Loading branch information
crisbeto committed Aug 28, 2020
1 parent f2953dd commit 1219b1e
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/material-experimental/mdc-dialog/dialog.scss
Expand Up @@ -2,6 +2,7 @@
@import '@material/dialog/variables.import';
@import '../mdc-helpers/mdc-helpers';
@import './mdc-dialog-structure-overrides';
@import '../../cdk/a11y/a11y';

// Dialog content max height. This has been copied from the standard dialog
// and is needed to make the dialog content scrollable.
Expand All @@ -16,6 +17,10 @@ $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 cdk-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 1219b1e

Please sign in to comment.