Skip to content

♿(frontend) focus skip link on headings and skip grid dropzone#1983

Merged
Ovgodd merged 2 commits intomainfrom
enhance/skiplink-focus
Mar 20, 2026
Merged

♿(frontend) focus skip link on headings and skip grid dropzone#1983
Ovgodd merged 2 commits intomainfrom
enhance/skiplink-focus

Conversation

@Ovgodd
Copy link
Collaborator

@Ovgodd Ovgodd commented Mar 11, 2026

Purpose

Improve keyboard and screen reader navigation by moving skip link focus to page
headings instead of the whole main region, and by keeping the docs grid
dropzone out of the keyboard focus order.

Proposal

  • focus the skip link and route-change target on the first available page
    heading (h1, then h2) instead of main
  • remove the header logo heading semantics so each page keeps a single,
    meaningful main heading
  • update main page titles to use proper heading levels for the new focus
    target
  • prevent the docs grid dropzone container from receiving keyboard focus
  • Change Skiplink semantic from button to a

@Ovgodd Ovgodd requested a review from AntoLC March 11, 2026 13:46
@Ovgodd Ovgodd self-assigned this Mar 11, 2026
@Ovgodd Ovgodd force-pushed the enhance/skiplink-focus branch 4 times, most recently from 798f479 to c909355 Compare March 11, 2026 13:56
@github-actions
Copy link

github-actions bot commented Mar 11, 2026

Size Change: -983 B (-0.02%)

Total Size: 4.23 MB

Filename Size Change
apps/impress/out/_next/static/cfbf3eb0/_buildManifest.js 0 B -904 B (removed) 🏆
apps/impress/out/_next/static/ef3fe2c0/_buildManifest.js 905 B +905 B (new file) 🆕

compressed-size-action

@Ovgodd Ovgodd force-pushed the enhance/skiplink-focus branch 3 times, most recently from 5b61e43 to 968b033 Compare March 11, 2026 14:05
@Ovgodd Ovgodd marked this pull request as ready for review March 11, 2026 14:07
@Ovgodd Ovgodd force-pushed the enhance/skiplink-focus branch 4 times, most recently from 790d114 to ff9efe0 Compare March 11, 2026 15:08
@Ovgodd Ovgodd force-pushed the enhance/skiplink-focus branch from 9fc029a to 32fbc79 Compare March 16, 2026 09:30
@Ovgodd Ovgodd requested a review from AntoLC March 16, 2026 09:31
@Ovgodd Ovgodd linked an issue Mar 16, 2026 that may be closed by this pull request
@Ovgodd Ovgodd moved this from Backlog to In review in LaSuite Docs A11y Mar 16, 2026
@Ovgodd Ovgodd requested a review from AntoLC March 19, 2026 09:04
@Ovgodd Ovgodd requested a review from AntoLC March 19, 2026 13:51
@Ovgodd Ovgodd force-pushed the enhance/skiplink-focus branch 3 times, most recently from 972fe6b to a253467 Compare March 20, 2026 08:30
Ovgodd added 2 commits March 20, 2026 10:05
We land keyboard users on page headings and keep the grid dropzone untabbable.
@Ovgodd Ovgodd force-pushed the enhance/skiplink-focus branch from a253467 to bf93640 Compare March 20, 2026 09:05
@Ovgodd Ovgodd merged commit bf93640 into main Mar 20, 2026
26 of 27 checks passed
@Ovgodd Ovgodd deleted the enhance/skiplink-focus branch March 20, 2026 09:46
@github-project-automation github-project-automation bot moved this from In review to Done in LaSuite Docs A11y Mar 20, 2026
lunika added a commit that referenced this pull request Mar 21, 2026
Changed

- 💫(frontend) fix the help button to the bottom in tree #2073
- ♿️(frontend) improve version history list accessibility #2033
- ♿️(frontend) fix more options menu feedback for screen readers #2071
- ♿(frontend) focus skip link on headings and skip grid dropzone #1983
- ♿️(frontend) fix search modal accessibility issues #2054
- ♿️(frontend) add sr-only format to export download button #2088
- ♿️(frontend) announce formatting shortcuts for screen readers #2070
- ✨(frontend) add markdown copy icon for Copy as Markdown option #2096
- ♻️(backend) skip saving in database a document when payload is empty #2062

Fixed

- ♿️(frontend) fix aria-labels for table of contents #2065
- 🐛(backend) allow using search endpoint without refresh token enabled #2097
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Incorrect semantics used on the skip link

2 participants