fix: remove tabindex=-1 for main skip link target

[knowler]

🧭 Context

Browsers already correctly set the “sequential focus navigation starting point” and doing this tends to create a larger usability regression with regards to where the focus starts (i.e. clicking within the document doesn’t set the focus starting point to wherever that was, but rather to the outer element with tabindex=-1).

There is a counterpoint related to screen magnifiers, however, I find that the benefit that tabindex=-1 brings to those situations is overstated/inconsistent and not the reason why tabindex=-1 was initially used as a hack in the first place.

📚 Description

Just removes tabindex=-1 on the main skip link target.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Building	Preview	Aug 13, 2026 1:23am
npmx.dev	Ready	Preview	Aug 13, 2026 1:23am

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
npmx-lunaria	Ignored		Aug 13, 2026 1:23am

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 1b4a42dd-d70d-485f-a386-22b623f19e6c

📥 Commits

Reviewing files that changed from the base of the PR and between 52e031f and b8380ab.

📒 Files selected for processing (1)

• app/app.vue

---

📝 Walkthrough

Summary by CodeRabbit

• Accessibility Improvements
  • Improved keyboard navigation by allowing the main content area to receive focus normally.

Walkthrough

The application removes tabindex="-1" from the #main-content container. No public declarations change.

Changes

Main content focus behaviour

Layer / File(s)	Summary
Remove negative tab index app/app.vue	The #main-content container no longer includes tabindex="-1".

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly describes the removal of tabindex="-1" from the main skip-link target.
Description check	✅ Passed	The description explains the accessibility rationale for removing tabindex="-1" from the main skip-link target.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch knowler/remove-tabindex-for-main-skip-link-target

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!