🎨 Palette: Keyboard shortcut for Dashboard and UI hints#208
Conversation
- Implement 'D' keyboard shortcut to toggle Dashboard panel. - Update #dashboardBtn title with '(D)' hint for discoverability. - Consolidate 'B' and 'T' shortcuts in global keydown listener. - Add e.preventDefault() to global shortcuts to prevent default browser actions. 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 | 756a21a | Jun 14 2026, 04:04 AM |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
ytstudio | 756a21a | Jun 14 2026, 04:04 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. |
- Implement 'D' shortcut to toggle Dashboard. - Update #dashboardBtn title with '(D)' hint. - Refactor keydown listener to prevent default browser behavior. - Remove significant redundant code blocks in js/app.js (duplicated nav listeners). - Ensure shortcut consistency and better accessibility. Co-authored-by: ruhdevops <203426218+ruhdevops@users.noreply.github.com>
- Implement 'D' keyboard shortcut to toggle Dashboard panel. - Update #dashboardBtn title with '(D)' hint for discoverability. - Add e.preventDefault() to 'T', 'B', and 'D' shortcuts to prevent default browser actions. - Ensure 'B' shortcut is context-aware (toggles save state when modal is open). Co-authored-by: ruhdevops <203426218+ruhdevops@users.noreply.github.com>
- Implement 'D' keyboard shortcut to toggle Dashboard panel. - Update #dashboardBtn title with '(D)' hint for discoverability. - Add e.preventDefault() to 'T', 'B', and 'D' shortcuts to prevent default browser actions. - Ensure 'B' shortcut is context-aware (toggles save state when modal is open). Co-authored-by: ruhdevops <203426218+ruhdevops@users.noreply.github.com>
![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: 756a21a8a3
ℹ️ 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 (DOM.dashboardModal.style.display === 'block') { | ||
| closeDashboard(); | ||
| } else { | ||
| openDashboard(); |
There was a problem hiding this comment.
When another overlay is already open, this global shortcut still calls openDashboard() instead of closing or ignoring the existing panel. For example, open Watch Later, press D, then press D again: closeDashboard() clears body.modal-open/overflow and restores lastFocused even though Watch Later is still visible, leaving the background scrollable and focus state inconsistent. Please guard against other active panels/modals or close them before opening the dashboard.
Useful? React with 👍 / 👎.
1 participant
💡 What: Added 'D' keyboard shortcut to toggle the Dashboard panel and updated the Dashboard button's title with the '(D)' hint.
🎯 Why: Improves power-user navigation and discoverability of features, ensuring consistency with existing Search (/) and Watch Later (B) shortcuts.
♿ Accessibility: Added shortcut hints to tooltips and ensured shortcuts don't conflict with input fields.
📸 Before/After: Dashboard button title changed from "View Dashboard" to "View Dashboard (D)". Keyboard navigation is now more complete.
PR created automatically by Jules for task 15904331516081921583 started by @ruhdevops