angular / angular/components

Warning message (cdk-virtual-scroll-viewport): When no height is set.

Offen
#26,133 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
area: cdk/scrolling feature P3
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

![image](https://user-images.githubusercontent.com/7819492/204823532-93cf3d6c-f705-45ca-9207-f9d6b92f8abb.png)
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

Beitragsleitfaden öffnen

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

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.