Skip to content

fix: release 6.4.10 sidebar navigation improvements - #111

Merged
chrisae9 merged 3 commits into
mainfrom
dev
Oct 12, 2026
Merged

chrisae9 merged 3 commits into
mainfrom
dev

Conversation

@chrisae9

Copy link
Copy Markdown
Owner

Summary

The exit icon in the Watch party header looked like a sidebar collapse control and could disconnect viewers accidentally. Both sidebar headers now use the same labeled X Close button, which only hides its sidebar. Leave room is a separate labeled action at the bottom of the navigation sidebar, with explanatory text and room-only visibility.

Prepare 6.4.10 and update compatible dependencies to clear the release security gate.

Verification

  • Build and lint passed.
  • Node suite passed; 50 frontend test files / 551 tests passed.
  • Four focused regression cases verify sidebar close does not disconnect, explicit room exit does, and exit is hidden outside a room.
  • Desktop and 390px phone-width local UI fixture inspected; both close controls exercised.
  • npm audit: zero vulnerabilities.
  • Live Plex playback and physical device behavior were not tested in this change.

@chrisae9
chrisae9 enabled auto-merge October 12, 2026 00:54
@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 55 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: chrisae9/synclounge/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 26227b43-1a6b-43bc-8a69-db09e891a19d

📥 Commits

Reviewing files that changed from the base of the PR and between c0aec42 and 0342746.


⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json, !package-lock.json

📒 Files selected for processing (7)
  • package.json
  • src/__tests__/sidebar-room-actions.test.js
  • src/components/SidebarCloseButton.vue
  • src/components/TheSidebarLeft.vue
  • src/components/TheSidebarRight.vue
  • test/fixtures/ui-preview.js
  • test/fixtures/ui-preview.vue

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

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.

@chrisae9
chrisae9 deployed to development October 12, 2026 00:56 — with GitHub Actions Active
@chrisae9
chrisae9 merged commit a8a9df8 into main Oct 12, 2026
16 checks passed

This branch was successfully deployed

1 active deployment
development — 03427461 Deployed Oct 12, 2026 by chrisae9 via Publish multi-platform dev image #75
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