Skip to content

feat(ux): Make filled stars more readable (vs. outlined stars) - #3124

Merged
enjeck merged 1 commit into
mainfrom
Jerome-Herbinet-make-filled-stars-more-readable
Oct 21, 2026
Merged

enjeck merged 1 commit into
mainfrom
Jerome-Herbinet-make-filled-stars-more-readable

Conversation

@Jerome-Herbinet

@Jerome-Herbinet Jerome-Herbinet commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Currently, filled stars (vs outlined stars) are difficult to read. This PR fixes this UX issue. This is also supposed to change hovering styles, to make selection easier.

⚠️ I have not test environment : screenshots where made thanks to my browser dev tools (changed CSS properties for simulation). This means that reviewers must test before merging.

🖼️ Screenshots : light theme

🏚️ Before 🏡 After
2026-10-19_14-59 2026-10-19_14-59_1

🖼️ Screenshots : dark theme

🏚️ Before 🏡 After
2026-10-19_14-58_1 2026-10-19_14-58

@Jerome-Herbinet Jerome-Herbinet added design Related to the design papercut Small issues that doesn't break the ux/ui 3. to review Waiting for reviews skill: frontend labels Oct 19, 2026
@Jerome-Herbinet Jerome-Herbinet changed the title feat(ux): Make filled stars more readable feat(ux): Make filled stars more readable (vs. outlined stars) Oct 19, 2026
Signed-off-by: Jérôme Herbinet <33763786+Jerome-Herbinet@users.noreply.github.com>
@Jerome-Herbinet
Jerome-Herbinet force-pushed the Jerome-Herbinet-make-filled-stars-more-readable branch from 4fd6473 to 4718577 Compare October 20, 2026 06:36
@samin-z
samin-z self-requested a review October 20, 2026 15:29
@enjeck
enjeck merged commit 5465ba8 into main Oct 21, 2026
66 checks passed
@enjeck
enjeck deleted the Jerome-Herbinet-make-filled-stars-more-readable branch October 21, 2026 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews design Related to the design papercut Small issues that doesn't break the ux/ui skill: frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants