-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add want to read dropper on author's page #8527
Add want to read dropper on author's page #8527
Conversation
for more information, see https://pre-commit.ci
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 @benbdeitch!
It looks like the button's text is very small on these pages:
Updating the font-size to 1rem
should fix it. That can be changed here:
font-size: 1em; |
Changed the font-size on the button from 1em to 1rem.
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.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #8527 +/- ##
==========================================
+ Coverage 16.00% 16.51% +0.51%
==========================================
Files 86 85 -1
Lines 4692 4456 -236
Branches 822 782 -40
==========================================
- Hits 751 736 -15
+ Misses 3416 3227 -189
+ Partials 525 493 -32 ☔ View full report in Codecov by Sentry. |
Closes #8520
This pull request seeks to add the 'Want to Read' dropper onto the various books listed on an Author page.
Technical
It's a two line alteration, that imports and adds the dropper component to the calling of the searchResultsWork macro.