angular / angular/components

bug(DIALOG): dialog's container-color does not follow Material 3 design specs

Open Beginner friendly
#30,536 0 comments 3 reactions 0 assignees View on GitHub
area: material/dialog P2
Dominant language
TypeScript
Stars
25k
Forks
6.8k
Avg merge
1d 8h
Merged PRs (30d)
91

Description

### Is this a regression?

- [ ] Yes, this behavior used to work in the previous version

### The previous version in which this bug was not present was

_No response_

### Description

As per [Material 3 guidelines](https://m3.material.io/components/dialogs/specs), `container-color` should use value `surface-container-high` token, but it's using value `surface` token.

We can override it by using `dialog-overrides` mixin like below:

```scss
@include mat.dialog-overrides((
container-color: var(--mat-sys-surface-container-high),
));
```

But, it should be the default behaviour as per Material 3 specs.

### Reproduction

Not needed

### Expected Behavior

As per [Material 3 guidelines](https://m3.material.io/components/dialogs/specs), `container-color` should use value `surface-container-high` token.

### Actual Behavior

Dialog's `container-color` is using value `surface` token.

### Environment

- Angular: 19
- CDK/Material: 19
- Browser(s): Chrome
- Operating System (e.g. Windows, macOS, Ubuntu): Windows

Contributor guide

Open the contributing guide

Research direction

No file or test is named. Locate the Angular Material dialog theming implementation and its existing tests, then compare the container-color token with the linked Material 3 dialog specification. Done means the default token uses surface-container-high and the relevant test coverage passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
angular, typescript
Domain
design, frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
64/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.