Skip to content

feat: update Player buttons UI#552

Merged
santosral merged 1 commit into
mainfrom
feat/update-player-buttons
Apr 29, 2026
Merged

feat: update Player buttons UI#552
santosral merged 1 commit into
mainfrom
feat/update-player-buttons

Conversation

@santosral
Copy link
Copy Markdown
Collaborator

@santosral santosral commented Apr 29, 2026

🚀 Summary

Updates the Learning Unit page's content-action buttons to better reflect what they trigger and to balance their visual weight. Renames the buttons to match their underlying content types ("Podcast"/"Video" instead of the verb-based "Listen"/"Watch"), swaps the Video button's icon to the more recognisable monitor-with-play, and adjusts variant hierarchy so neither action visually dominates when both content types are available on a unit.

✏️ Changes

  • Renamed the "Listen" button → "Podcast" across all three podcast play states (initial play, resume, resume-from-checkpoint). The "Pause" label is intentionally unchanged.
  • Renamed the "Watch" button → "Video".
  • Replaced the lucide Video icon on the Video button with MonitorPlay (monitor outline with a play triangle).
  • Added a hasPodcastAndVideo derived state on +page.svelte. When both content types are present on a unit, both buttons render as secondary so neither dominates; when only one type is present, that single CTA renders as primary.
  • File touched: src/routes/(main)/(protected)/unit/[id]/+page.svelte (only).
Screenshot 2026-04-29 at 3 37 13 PM

@santosral santosral self-assigned this Apr 29, 2026
@santosral santosral requested a review from nicholasjjlim April 29, 2026 07:36
Copy link
Copy Markdown
Collaborator

@nicholasjjlim nicholasjjlim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@santosral santosral merged commit 3755323 into main Apr 29, 2026
8 checks passed
@santosral santosral deleted the feat/update-player-buttons branch April 29, 2026 07:44
@santosral santosral mentioned this pull request Apr 29, 2026
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