Skip to content

fix: add links to major versions within 'other versions' dropdown#170

Merged
danielroe merged 5 commits intonpmx-dev:mainfrom
azaleta:main
Jan 27, 2026
Merged

fix: add links to major versions within 'other versions' dropdown#170
danielroe merged 5 commits intonpmx-dev:mainfrom
azaleta:main

Conversation

@azaleta
Copy link
Copy Markdown
Contributor

@azaleta azaleta commented Jan 27, 2026

In the [Versions] section : the root node of grouped major version is '<a herf>'
But the [OtherVersions] section don't have the same function.
Modification as below:

  1. change root node to '<a> element'
  2. Add PackageDateTime and ProvenanceBadge on right side.

Before img:
image

After img:
image

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 27, 2026

@mouchixin is attempting to deploy a commit to the danielroe Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link
Copy Markdown

vercel Bot commented Jan 27, 2026

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

Project Deployment Review Updated (UTC)
npmx.dev Ready Ready Preview, Comment Jan 27, 2026 0:50am

Request Review

@danielroe danielroe changed the title fix(PackageVersions): OtherVersions group major version root node should be an url fix: add links to major versions within 'other versions' dropdown Jan 27, 2026
- Add visible focus states to expand/collapse buttons
- Add aria-hidden to decorative icons
- Add aria-label to Other versions and major group expand buttons
- Add i18n keys for expand/collapse other versions and major groups
- Add tests for DateTime and ProvenanceBadge in OtherVersions section
- Add accessibility tests for focus states and aria attributes
@danielroe danielroe merged commit f63b7b7 into npmx-dev:main Jan 27, 2026
7 checks passed
vinnymac pushed a commit to vinnymac/npmx.dev that referenced this pull request Jan 28, 2026
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.

3 participants