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
LB-579: Improve pagination buttons on "Recent Listens" and "User Artists" page #1211
Conversation
Can one of the admins verify this patch? |
3 similar comments
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
Can one of the admins verify this patch? |
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.
Looks good. Thanks! Can you please update the test snapshots by running ./test.sh fe -u
. Otherwise it will break the tests once the admins verify the patch.
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.
Thanks for updating the snapshots. This looks okay now but I'd wait for the test results. @alastair can you please whitelist them.
@brainzbot add to whitelist |
Are these failed tests relevant here ? Do i need to make some changes? @alastair |
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.
Thanks for the PR @kshitijk83 !
This is a good improvement, it's been bothering me for a while :)
The failing tests are unrelated to the changes in the PR, so I'll go ahead and merge it.
Problem
Solution
Changed classname from "hidden" to "disabled"