🎨 Palette: Add 'B' keyboard shortcut for Watch Later and UI hints#203
🎨 Palette: Add 'B' keyboard shortcut for Watch Later and UI hints#203ruhdevops wants to merge 1 commit into
Conversation
- Implement 'B' keyboard shortcut to toggle Watch Later (toggles video if open, otherwise toggles panel). - Add (B) shortcut hints to 'title' and 'aria-label' attributes for discoverability and accessibility. - Sync dynamic aria-label updates in JS with shortcut hints. Co-authored-by: ruhdevops <203426218+ruhdevops@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
ytstudio | cc6c38b | Jun 06 2026, 03:57 AM |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
ytstudio | cc6c38b | Jun 06 2026, 03:57 AM |
❌ Deploy Preview for ytr-studio failed.
|
✅ Deploy Preview for elegant-seahorse-d9d460 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cc6c38bdab
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if (AppState.current) { | ||
| toggleWatchLater(AppState.current); | ||
| } else if (DOM.watchLaterPage) { | ||
| const isVisible = window.getComputedStyle(DOM.watchLaterPage).display !== 'none'; |
There was a problem hiding this comment.
On a fresh page load, the Watch Later panel is hidden off-canvas by .side-panel[aria-hidden]/right, but its computed display is still block, so the first press of B outside a video takes this branch as “visible” and calls closeWatchLater() instead of opening the panel. This makes the advertised shortcut require two presses until the panel has been closed once via inline display: none; checking aria-hidden (or the right/open state) would match how the panel is initially hidden.
Useful? React with 👍 / 👎.
1 participant
💡 What: The UX enhancement added
Implemented the 'B' keyboard shortcut to toggle the Watch Later feature and added visual/accessible hints to the UI.
🎯 Why: The user problem it solves
Improves efficiency for power users by providing a quick way to save videos or access the Watch Later list without using the mouse.
♿ Accessibility: Any a11y improvements made
aria-labelattributes to ensure screen reader users are aware of the shortcut.PR created automatically by Jules for task 16227352940846452898 started by @ruhdevops