Skip to content

Fix star rating display on books page#5

Merged
marksmith1994 merged 3 commits into
masterfrom
claude/add-claude-documentation-MabfT
May 21, 2026
Merged

Fix star rating display on books page#5
marksmith1994 merged 3 commits into
masterfrom
claude/add-claude-documentation-MabfT

Conversation

@marksmith1994

Copy link
Copy Markdown
Owner

Summary

  • Replaces the broken CSS overlay star widget (which rendered as 3 separate rows) with individual <span> elements per star
  • Full stars: gold , empty stars: grey , half stars: CSS gradient clip splitting the character 50/50
  • Removed the /5 number label — stars only

Test plan

  • 5★ books show all gold (e.g. Project Hail Mary, Fellowship of the Ring)
  • 3.5★ books show 3 gold + 1 half + 1 grey
  • No stacked/duplicate rows of stars

https://claude.ai/code/session_01Qj49iUs6QNA2MQcCVP5cFN


Generated by Claude Code

claude added 3 commits May 21, 2026 10:00
…pans

The overlay approach rendered as 3 separate rows. Now each star is its
own span (star-full / star-half / star-empty). Half stars use a
left-to-right gradient clip on the ★ character — no number shown.

https://claude.ai/code/session_01Qj49iUs6QNA2MQcCVP5cFN
Bootstrap's active tab sets colour via --bs-nav-tabs-link-active-color
which was matching the background. Use !important to ensure our colour
variables always win.

https://claude.ai/code/session_01Qj49iUs6QNA2MQcCVP5cFN
- SteamService fetches top 15 most-played games from Steam Web API,
  cached 24h; uses public CDN header images (no extra API calls)
- Gaming tab shows game art, hours played, and a 'Playing' badge for
  anything played in the last 2 weeks
- Steam:ApiKey and Steam:UserId added to appsettings.json structure
  (secrets stored in .env / Azure App Settings)

https://claude.ai/code/session_01Qj49iUs6QNA2MQcCVP5cFN
@marksmith1994 marksmith1994 merged commit 26a9b17 into master May 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants