🎨 Palette: [Enhance Header Navigation Accessibility] - #7
Conversation
Added a prominent "View Resume" button below the bio text in the Hero section. The button: - Opens Google Drive resume in a new tab - Features FileText icon from lucide-react - Uses primary color with hover effects (scale, opacity, shadow) - Is fully responsive with proper spacing for mobile/desktop
…Dev22603/dev22603 into claude/add-resume-buttons-QEFK0
Added MathEx - Math Equation Editor Library to personal projects. A React component library for math equation editors with 13 function categories and full TypeScript support.
- Add npm link for MathEx project - Fix LSTM project: change github key to kaggle for Kaggle link - Standardize all projects to have github, live, npm, kaggle keys (null if unavailable) - Update ProjectCard and ProjectModal to display npm and kaggle links with icons - Remove ExternalLink icon from project card header
Added `aria-label` attributes to the icon-only 'X' close buttons in the ProjectModal and ExperienceModal components to ensure screen reader accessibility. Also logged the finding to `.Jules/palette.md`. Co-authored-by: Dev22603 <92785712+Dev22603@users.noreply.github.com>
…5347359185212 🎨 Palette: Add ARIA labels to modal close buttons
- Convert logo wrapper to a native `<button>` for keyboard focusability and interactivity. - Add `aria-expanded` and `aria-controls` to the mobile menu toggle button. - Add `id="mobile-menu"` to the mobile menu container. - Ensure focus-visible states on buttons. Co-authored-by: Dev22603 <92785712+Dev22603@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. |
✅ Deploy Preview for dev-bachani ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
2 participants
💡 What: Improved the accessibility of the Header navigation components by utilizing native button elements and ARIA attributes.
🎯 Why: The logo acted as a scroll-to-top trigger but was implemented using a
divandspanwithcursor-pointer, making it completely inaccessible to keyboard users and screen readers. The mobile menu toggle lacked ARIA states to indicate if the menu was open or closed.♿ Accessibility:
<button>withfocus-visiblestyles and anaria-label="Scroll to top".aria-expandedandaria-controlsto the mobile hamburger menu button.id="mobile-menu"to the mobile menu container.PR created automatically by Jules for task 2350090967123088350 started by @Dev22603