Skip to content

[4882] Accessibility: Skip navigation & focus visibility#4883

Merged
ZJvandeWeg merged 2 commits intomainfrom
4882-accessibility-1-and-4
Apr 15, 2026
Merged

[4882] Accessibility: Skip navigation & focus visibility#4883
ZJvandeWeg merged 2 commits intomainfrom
4882-accessibility-1-and-4

Conversation

@n-lark
Copy link
Copy Markdown
Contributor

@n-lark n-lark commented Apr 14, 2026

Description

See details here.

Related Issue(s)

Resolves #4882

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated
  • For blog PRs, an Art Request has been created (instructions)

@n-lark n-lark requested review from allthedoll and knolleary April 14, 2026 18:42
@n-lark n-lark self-assigned this Apr 14, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 14, 2026

Deploy Preview for flowforge-website ready!

Name Link
🔨 Latest commit 31b9315
🔍 Latest deploy log https://app.netlify.com/projects/flowforge-website/deploys/69de8bcf98fadd000892d214
😎 Deploy Preview https://deploy-preview-4883--flowforge-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 87 (🟢 up 1 from production)
Accessibility: 82 (no change from production)
Best Practices: 100 (no change from production)
SEO: 91 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

@allthedoll allthedoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, but have someone else test local as I'm on mobile.

@ZJvandeWeg
Copy link
Copy Markdown
Member

Don't mean to overstep, but is simple enough to merge.

@ZJvandeWeg ZJvandeWeg merged commit ca06dd5 into main Apr 15, 2026
7 checks passed
@ZJvandeWeg ZJvandeWeg deleted the 4882-accessibility-1-and-4 branch April 15, 2026 13:03
@Yndira-E
Copy link
Copy Markdown
Contributor

Hey @n-lark, quick accessibility question: when the homepage loads, the AI chat box appears to already have a focused style (blue outline) visible by default, before any user interaction.

Is this intentional? Attaching a screenshot for reference.
Screenshot 2026-04-15 at 20 14 03

@n-lark
Copy link
Copy Markdown
Contributor Author

n-lark commented Apr 15, 2026

Hey @Yndira-E good catch! That's expected behavior. The homepage auto-focuses the AI chat text area on page load, which triggers the new focus outline. Since the field genuinely has keyboard focus at that point, showing the indicator is the correct accessible behavior. If a user starts tabbing, they'll know exactly where they are.

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.

Accessibility: Skip navigation & focus visibility

4 participants