fix: star ratings hidden on Already Read page at mobile/narrow viewports#12798
Conversation
|
Thanks for the PR, @Armansiddiqui9. 🤖 Copilot has been assigned for an initial review. The linked issue #12796 hasn't been triaged yet — triage happens on Mondays and Fridays. There are currently 71 open non-draft PRs ahead of yours. PR triage checklist (maintainers / Pam)
Note This comment was automatically generated by Pam, Open Library's Project AI Manager, on behalf of @mekarpeles. Pam is designed to provide status visibility, perform basic project management functions and relevant codebase research, and provide actionable feedback so contributors aren't left waiting. |
There was a problem hiding this comment.
Pull request overview
Fixes a responsive CSS regression where star rating widgets on the Reading Log “Already Read” page were being hidden at narrow/mobile viewports due to an overly broad selector in the read panel styles.
Changes:
- Scoped the mobile rule that hides the desktop star rating form so it only applies within book detail page containers (
.read-options/.btn-notice), rather than globally. - Preserved existing behavior of hiding
.modal-links.desktopon mobile to ensure the mobile modal-links variant is used.
|
Nice job! Thank you @Armansiddiqui9 |
Closes #12796
fix
Bonus
Star ratings are now also accessible on mobile devices for the first time.
Previously patrons on phones could never rate books directly from their
reading log. This fix unblocks that use case at no extra cost.
Technical
Scope the rule to
.read-optionsand.btn-notice— the containersthat wrap the form exclusively on book detail pages:
The reading log renders its form inside
.list-books, which is nevermatched, so ratings now stay visible at all viewport widths.
Testing
/account/books/already-readScreenshot
Before


After
Mobile view before


Mobile view After
Stakeholders
@cdrini @lokesh