Skip to content

🎨 Palette: Semantic Navigation & Accessibility Improvements - #58

Open
ISTEC-FOR340 wants to merge 1 commit into
mainfrom
palette/semantic-navigation-ux-123987103147693066
Open

🎨 Palette: Semantic Navigation & Accessibility Improvements#58
ISTEC-FOR340 wants to merge 1 commit into
mainfrom
palette/semantic-navigation-ux-123987103147693066

Conversation

@ISTEC-FOR340

Copy link
Copy Markdown
Owner

πŸ’‘ What: The UX enhancement added

Refactored the core navigation of the application to use semantic HTML <a> tags instead of div elements with JavaScript onclick listeners. This includes the project cards on the landing page and the "Back" buttons within the p5.js sketches.

🎯 Why: The user problem it solves

Non-semantic navigation breaks standard browser features (like middle-clicking to open in a new tab) and is completely invisible to screen readers as interactive links. Additionally, the lack of visible focus states made the site difficult to navigate for keyboard users.

β™Ώ Accessibility: Any a11y improvements made

  • Semantic Links: Screen readers now correctly identify navigation elements as links.
  • Keyboard Support: Added high-contrast :focus-visible states that mirror the hover effects, ensuring keyboard users have clear visual feedback.
  • ARIA Labels: Added aria-label="Back to home" to back buttons to provide clear context for assistive technologies.
  • Document Language: Corrected lang="es" to lang="en" to match the primary language of the content, ensuring correct pronunciation by screen readers.

πŸ“Έ Before/After:

Verification Screenshot shows the new semantic link structure and visible focus indicator on the "Circle Matrix" card.


PR created automatically by Jules for task 123987103147693066 started by @ISTEC-FOR340

- Refactor landing page cards from div+JS to semantic <a> tags
- Convert project back buttons to <a> tags with aria-label
- Add :focus-visible states for better keyboard navigation
- Update HTML lang="en" and remove redundant script.js
- Ensure focus rings are not clipped with overflow: visible

Co-authored-by: ISTEC-FOR340 <269002038+ISTEC-FOR340@users.noreply.github.com>
@google-labs-jules

Copy link
Copy Markdown
Contributor

πŸ‘‹ 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 @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

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