Skip to content
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

fix(paginator): change class name of increment and decrement icons #9597

Closed
wants to merge 1 commit into from

Conversation

lharries
Copy link
Contributor

Fix such that the previous page navigation button contains an icon with class decrement,
and the next page navigation button contains an icon with class increment. Previously
they were incorrectly swapped.

Noticed during fix for #9278

@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jan 25, 2018
@lharries lharries force-pushed the paginator-class-fix branch from 782ac8d to 924df08 Compare January 25, 2018 14:10
@lharries lharries force-pushed the paginator-class-fix branch from 22846de to 0d12179 Compare January 25, 2018 14:49
Copy link
Contributor

@andrewseguin andrewseguin left a comment

Choose a reason for hiding this comment

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

Oh goodness, nice catch and thanks for the fix

@andrewseguin andrewseguin 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 P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent labels Jan 25, 2018
@tinayuangao
Copy link
Contributor

@lharries Please rebase.
Presubmit passes, after rebase it's ready to go.

@lharries
Copy link
Contributor Author

@tinayuangao this has been fixed by #9603 which has now been merged so I'm not sure this PR is needed too?

@tinayuangao
Copy link
Contributor

@lharries Just need to rebase to master and resolve the conflicts.

@lharries lharries force-pushed the paginator-class-fix branch 3 times, most recently from 992ab4c to 73170ba Compare January 31, 2018 00:47
@lharries
Copy link
Contributor Author

@tinayuangao it won't let me rebase as there are no changes to be made, as they have already been merged in #9603

Fix such that the previous page navigation button contains an icon with class decrement,
and the next page navigation button contains an icon with class increment. Previously
they were incorrectly swapped.

Noticed during fix for angular#9278
@lharries lharries force-pushed the paginator-class-fix branch from 354dd01 to 8194d0e Compare January 31, 2018 00:53
@lharries
Copy link
Contributor Author

I have managed to rebase, however as you can see there are no changes

@andrewseguin
Copy link
Contributor

No worries, this can be closed since the changes were in the other PR.

@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 Sep 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker cla: yes PR author has agreed to Google's Contributor License Agreement P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants