angular / angular/components

[Table] MatTableDataSource constructor should include MatSort, MatPaginator

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

説明

#### Bug, feature request, or proposal:
feature request

#### What is the expected behavior?
Have a constructor that, at least optionally, takes a MatSort and/or a MatPaginator as parameters.

#### What is the current behavior?
Currently, you have to create a new MatTableDataSource with only data as a parameter. However, you always have to set the matSort and paginator of the MatTableDataSource after construction. I had to make a class that extends this class to have this, which seems superfluous.

#### What are the steps to reproduce?

#### What is the use-case or motivation for changing an existing behavior?
To make the creation of MatTableDataSources easier.

#### Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular: 5.0.1
Material: 5.0.0-rc0
OS: Windows 10 Pro
TypeScript: 2.6.1
Browser: Google Chrome 62.0.3202.94 (Official Build) (64-bit)
#### Is there anything else we should know?

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

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

調査の方向性

まず MatTableDataSource のエントリーポイントと既存のテストを見つけ、次に構築後に MatSort と MatPaginator が現在どのように割り当てられているかを確認します。コンストラクターがこれらのオブジェクトをオプションで受け付け、サポートされている各組み合わせでの構築をテストがカバーすれば完了です。

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

評価

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

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

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