Skip to content

feat(Host): add a 下一首 button beside 收卷並計分 - #10

Merged
xinshoutw merged 1 commit into
mainfrom
feat/host-next-button
Oct 2, 2026
Merged

xinshoutw merged 1 commit into
mainfrom
feat/host-next-button

Conversation

@xinshoutw

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a 下一首 button to the /host view, taking the right third of 收卷並計分's half (styled like the dashboard's 下一首)
  • It unlocks only after the selected song is closed and locks again once the next song is selected, until that song is closed too; it stays locked on the last song
  • The lock is derived from the server state (round.songId === songId and closed), so it survives reloads and switching between the dashboard and host views
  • On phones (≤759px) the button shrinks to its icon, so the row fits a 320px screen

Test plan

  • Headless Chromium against wrangler dev: open → close → next → open → close → next → close on the last song, checking each button's disabled state
  • Screenshots at 1280, 768, 375 and 320px; no horizontal overflow
  • npm test

- Takes the right third of the close button's half; icon only on phones
- Unlocks once the selected song is closed and locks again on the next song
@coderabbitai

coderabbitai Bot commented Oct 2, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 73e8820a-d4eb-46c9-b9e4-9e99b1fad0c7

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@xinshoutw
xinshoutw merged commit a9d6d43 into main Oct 2, 2026
2 checks passed
@xinshoutw
xinshoutw deleted the feat/host-next-button branch October 2, 2026 08:05
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.

1 participant