Skip to content

🎨 Palette: Improve landing page and project navigation accessibility - #45

Open
ISTEC-FOR340 wants to merge 1 commit into
mainfrom
palette-semantic-navigation-a11y-13122572610622220354
Open

🎨 Palette: Improve landing page and project navigation accessibility#45
ISTEC-FOR340 wants to merge 1 commit into
mainfrom
palette-semantic-navigation-a11y-13122572610622220354

Conversation

@ISTEC-FOR340

Copy link
Copy Markdown
Owner

πŸ’‘ What: Improved accessibility and semantic structure of the landing page and project navigation.

🎯 Why: The previous implementation used non-semantic div elements with onclick listeners for navigation, which made the site inaccessible to keyboard and screen reader users. Additionally, focus indicators were missing or clipped.

β™Ώ Accessibility Improvements:

  • Semantic <a> tags for all navigation.
  • High-contrast :focus-visible indicators.
  • Valid HTML nesting (no nested interactives).
  • Standardized lang="es" for Spanish-speaking audience.
  • Unclipped focus rings for glassmorphism components.

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

This commit enhances the user experience and accessibility by:
1. Converting non-semantic `div` cards into semantic `<a>` tags for native navigation and keyboard support.
2. Replacing nested `<button>` elements with `<span>` to maintain valid HTML within links.
3. Adding explicit `:focus-visible` styles with high-contrast focus rings for better keyboard navigation visibility.
4. Converting project "Back" buttons into semantic `<a>` tags.
5. Standardizing the `lang="es"` attribute across all HTML files for better screen reader support in the target language.
6. Removing redundant navigation logic from `script.js`.
7. Ensuring focus rings are not clipped by glassmorphism cards by updating `overflow` properties.

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