🎨 Palette: Dashboard Accessibility and Shortcut Enhancement#213
🎨 Palette: Dashboard Accessibility and Shortcut Enhancement#213ruhdevops wants to merge 1 commit into
Conversation
- Added aria-expanded and aria-controls to #dashboardBtn in index.html. - Added '(D)' shortcut hint to dashboard button title. - Implemented 'D' keyboard shortcut to toggle dashboard panel. - Refactored js/app.js to remove redundant DOM entries and scroll listeners. - Synchronized aria-expanded state in openDashboard/closeDashboard functions. 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 | 494f75b | Jun 18 2026, 03:55 AM |
❌ Deploy Preview for ytr-studio failed.
|
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ❌ Deployment failed View logs |
ytstudio | 494f75b | Jun 18 2026, 03:55 AM |
✅ 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: 494f75bdd6
ℹ️ 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".
| } | ||
|
|
||
| // Dashboard toggle | ||
| if (key === 'd') { |
There was a problem hiding this comment.
When focus is anywhere except an input/textarea, this treats every key event whose key is d as the dashboard toggle, including Ctrl+D/Cmd+D that users use to bookmark the page. In that scenario the dashboard opens or closes unexpectedly while the browser bookmark shortcut is invoked; check e.ctrlKey/e.metaKey/e.altKey before handling the new global shortcut.
Useful? React with 👍 / 👎.
1 participant
💡 What: This PR enhances the Dashboard's UX by improving its accessibility and discoverability.
🎯 Why: The dashboard lacked proper ARIA attributes to communicate its state to screen readers. Furthermore, while Search and Watch Later had keyboard shortcuts, the Dashboard did not, making the experience inconsistent for power users.
♿ Accessibility:
aria-expandedandaria-controlsto the dashboard toggle button.titleto include the keyboard shortcut hint(D).aria-expandedattribute with the panel's visibility state in JavaScript.✨ Improvements:
Dglobal keyboard shortcut to toggle the Dashboard panel.js/app.js(multiple duplicate DOM selector entries and event listeners) to improve maintainability and performance.PR created automatically by Jules for task 12508629615568804500 started by @ruhdevops