-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Conversation
782ac8d
to
924df08
Compare
22846de
to
0d12179
Compare
There was a problem hiding this 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
@lharries Please rebase. |
@tinayuangao this has been fixed by #9603 which has now been merged so I'm not sure this PR is needed too? |
@lharries Just need to rebase to master and resolve the conflicts. |
992ab4c
to
73170ba
Compare
@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
354dd01
to
8194d0e
Compare
I have managed to rebase, however as you can see there are no changes |
No worries, this can be closed since the changes were in the other PR. |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
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