angular / angular/components

bug(Density): Setting the density via mat.all-component-densities gives different result than setting via mat.theme density

Aperta
#30,681 1 commento 6 reazioni 0 assegnatari Vedi su GitHub
area: material/core P4
Lingua principale
TypeScript
Stelle
25k
Fork
6.8k
Merge medio
1g 8h
PR unite (30g)
91

Descrizione

### 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

When defining the density via mat.theme, e.g.
```
@include mat.theme(
(
color: (
theme-type: light,
primary: mat.$azure-palette,
tertiary: mat.$blue-palette,
),
typography: Roboto,
density: (
scale: 0,
),
)
);
```

the button height is 40px (`--mdc-text-button-container-height`).

However, using the `mat.all-component-densities(0)` mixin results in button height equals to 36px.

Similar issue applies to other densities - setting the density through the mixin results in different height values than setting it through the `mat.theme` mixin.

### Reproduction

StackBlitz link: https://stackblitz.com/edit/czrubf43?file=src%2Fstyles.scss
Steps to reproduce:
1. inspect the button height without setting the density. Height is 40px
2. set the density height with the dropdown to 0. Height changes to 36px.

changing density through the dropdown adds respectively classes to the body: .density0, .density-1 ... .density-5

the scss for those is (see styles.scss)
```
.body {
&.density0 {
@include mat.all-component-densities(0);
}

...

&.density-5 {
@include mat.all-component-densities(-5);
}

}
```

### Expected Behavior

densities (component heights) set through `mat.all-component-densities(0)` should be the same when set through
`mat.theme((density: 0))`

### Actual Behavior

densities (component heights) set through `mat.all-component-densities(0)` are different when set through
`mat.theme((density: 0))`

### Environment

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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia con la riproduzione StackBlitz e il relativo styles.scss, quindi confronta il CSS generato da mat.all-component-densities(0) con la configurazione della densità passata a mat.theme. Controlla l'altezza del pulsante e i valori di densità corrispondenti per 0 e per le altre scale; il lavoro è completato quando entrambi i percorsi di configurazione producono le stesse altezze dei componenti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
angular, sass
Ambito
frontend
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.