Skip to content

perf(material-experimental/mdc-progress-bar): Use class for MDC adapter #20088

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

ngwattcos
Copy link
Contributor

@ngwattcos ngwattcos commented Jul 24, 2020

Updates mdc-progress-bar to use a class-based adapter, which improves performance, just like in:
#19980

Hopefully, test failures will be resolved after this PR is merged:
material-components/material-components-web#6256

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jul 24, 2020
@mmalerba mmalerba changed the title Optimize progress bar perf(material-experimental/mdc-progress-bar): Use class for MDC adapter Jul 27, 2020
@mmalerba
Copy link
Contributor

Are the checkbox changes here intentionally or was that some kind of bad rebase?

Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

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

Looks good aside from the changes in unrelated components

@ngwattcos
Copy link
Contributor Author

@mmalerba I accidentally left changes in checkbox and radio on my branch - should be gone now.

@mmalerba
Copy link
Contributor

still seeing checkbox and radio changes

@ngwattcos ngwattcos force-pushed the optimize-progress-bar branch from bc32873 to b9fd9b4 Compare July 29, 2020 02:15
@ngwattcos
Copy link
Contributor Author

ngwattcos commented Jul 29, 2020

@mmalerba I accidentally committed all files again. Silly me! Should be fixed now.

@ngwattcos ngwattcos force-pushed the optimize-progress-bar branch from b9fd9b4 to 256ff6e Compare July 29, 2020 02:26
Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

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

LGTM, just one nit

@mmalerba
Copy link
Contributor

mmalerba commented Aug 3, 2020

It looks like there are some failing tests

@andrewseguin andrewseguin removed the cla: yes PR author has agreed to Google's Contributor License Agreement label Dec 28, 2021
@andrewseguin
Copy link
Contributor

Closing due to inactivity

@angular-automatic-lock-bot
Copy link

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-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Apr 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants