Skip to content

fix(table): use static queries for examples - #15483

Merged
mmalerba merged 1 commit into
angular:masterfrom
andrewseguin:table-examples-static-query
Mar 15, 2019
Merged

fix(table): use static queries for examples#15483
mmalerba merged 1 commit into
angular:masterfrom
andrewseguin:table-examples-static-query

Conversation

@andrewseguin

Copy link
Copy Markdown
Contributor

Adds {static: true} to any queries in our examples where the query is used in ngOnInit

@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Mar 14, 2019

@ViewChild(MatPaginator) paginator: MatPaginator;
@ViewChild(MatSort) sort: MatSort;
@ViewChild(MatPaginator, {static: true}) paginator: MatPaginator;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couldn't we move the usages to ngAfterViewInit, rather than marking them as static? Since it's an example it won't be a breaking change.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The issue with the table is that it needs the paginator and sort to figure out the initially rendered rows before initialization. If we wait until after view init, the table will initially render all rows.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see, although it's a little unfortunate that we have to require people to use static queries.

@crisbetocrisbeto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@crisbetocrisbeto added pr: lgtm action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Mar 14, 2019
@mmalerba
mmalerba merged commit d525f9c into angular:masterMar 15, 2019
@mmalerbammalerba added target: major This PR is targeted for the next major release and removed target: patch This PR is targeted for the next patch release labels Mar 18, 2019
Suresh918 pushed a commit to Suresh918/material2 that referenced this pull request Apr 15, 2019
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 10, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreementtarget: majorThis PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@andrewseguin@crisbeto@googlebot@mmalerba