Skip to content

Conversation

@garthdw
Copy link
Contributor

@garthdw garthdw commented Feb 1, 2026

Move top level sections into their own component.

image

Missing sections were:

  • keywords
  • maintainers
  • compatibility
  • install scripts
  • AI skills

Move top level sections into their own component.
@vercel
Copy link

vercel bot commented Feb 1, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 1, 2026 0:41am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 1, 2026 0:41am
npmx-lunaria Ignored Ignored Feb 1, 2026 0:41am

Request Review

@whitep4nth3r
Copy link
Contributor

I made a little screen recording of the experience and encountered a couple of issues when navigating with a keyboard.

  1. Sometimes I am unable to use enter to close the section again. This happens most noticeably on the "latest version" section most obviously right at the end of the recording. When the group opens, the toggle button sometimes loses focus.
  2. For keyboard navigation, in dark mode, the focus outlines (or underlines) are not very clear in terms of colour contrast. I had trouble seeing where I was tabbed and focussed to. This is not specifically related to this PR but was very noticeable in testing.
collapsible_sections.mov

@whitep4nth3r
Copy link
Contributor

I created an issue to address focus outlines so you can ignore that in this PR :)

#647

@garthdw
Copy link
Contributor Author

garthdw commented Feb 1, 2026

Thanks @whitep4nth3r, the collapsibles under versions are done "manually" as opposed to using the new component. They also seem to have a lot more logic than a simple collapse/no collapse.

So likely there's some DOM stuff happening that's shifting focus. Certainly something we will want to look at.

@serhalp serhalp added the blocked label Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants