Warning message (cdk-virtual-scroll-viewport): When no height is set.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 25k
- Forks
- 6.8k
- Ø Merge
- 1 T. 8 Std.
- Gemergte PRs (30 T.)
- 91
Beschreibung
### Feature Description
I know this is a very insignificant "feature", but I think it will be helpful.
`` require fixed height to be set.
Without height it doesn't render any elements at all (even the html structure stays empty). This can be verry misleading and time consuming in some cases.
I just taked sandbox example with static data and modifyed it to take async data.
I was tired and spend few hours thinking about that is a bug in my code flow (timing, data refreshing or something else).
Today on a fresh mind I realize it's just because it has no height.
Maybe if you just `warn` the user to set height if there is no height or maybe set `min-height` = single element height. This can help to prevent others to make this mistake.
### Use Case

When we have data and no height - no html is rendered. If there is a `console.warn('there is no height man.')` will be very obviously and catchable.
Beitragsleitfaden
Rechercherichtung
Beginne damit, den Einstiegspunkt der Komponente cdk-virtual-scroll-viewport und ihre vorhandenen Tests zu finden. Reproduziere den Fall mit statischen Daten ohne Höhe und verfolge anschließend, wie der Viewport seine initialen Abmessungen verarbeitet. Als erledigt gilt die Aufgabe, wenn der Fall mit fehlender Höhe den Benutzern eine eindeutige, abfangbare Warnung oder eine andere vereinbarte Anzeige gibt, statt stillschweigend nichts zu rendern.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- angular, typescript
- Bereich
- frontend
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100