angular / angular/components

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

オープン
#26,133 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
area: cdk/scrolling feature P3
主要言語
TypeScript
スター
25k
フォーク
6.8k
平均マージ
1日 8時間
マージ済み PR(30日)
91

説明

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

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず cdk-virtual-scroll-viewport コンポーネントのエントリポイントと既存のテストを見つけます。高さがない static-data のケースを再現し、その後、viewport が初期寸法をどのように処理するかを追跡します。missing-height のケースで、何も表示せずに終わるのではなく、ユーザーに明確で捕捉可能な警告、または合意した別の通知が提供されれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
angular, typescript
領域
frontend
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。