Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

data-active and aria-current are not server-rendered on plain anchors

Fermée
  • Quelque chose ne va pas ?
#654 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
52/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
typescript
Domaine
full-stack

Piste de recherche

Start in src/routers/factory.tsx at the isServer guard around setupLinkClaims, then read src/utils.ts matchLink and managedPath to see what a server-side computation would need (request path, query, base instead of document.baseURI). Reproduce with a renderToString render at a matching path and confirm the anchor lacks data-active/aria-current. Done means the server HTML contains both attributes (never data-pending), verified by an SSR test; the compiler/@solidjs/web hook is the larger follow-on, so scope the router-side change first.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

Describe the bug

On next, the link-state attributes that plain anchors get from the router (data-active, aria-current="page") are never in the server-rendered HTML. They only appear after the client router hydrates.

setupLinkClaims only runs inside if (!isServer) in src/routers/factory.tsx, and in the server build of @solidjs/web, registerElementClaim() does nothing. That means:

  • Navigation styled with a[data-active] or a[aria-current="page"] shows no active link until hydration finishes, and the highlight then appears late. This is easy to see on slow connections or long streamed pages.
  • Pages that never hydrate (noScripts or HTML-only routes) never get active state. They also never get aria-current="page", so screen readers are not told which link is the current page.
Your Example Website or App

The minimal code is below. No hosted repro is needed because the behavior shows up in any renderToString output.

Steps to Reproduce the Bug or Issue
  1. Render a router app on the server at /about, with a plain <a href="/about">About</a> in a layout.
  2. Look at the HTML response.
  3. The anchor is <a href="/about">About</a>, with no data-active and no aria-current.
Expected behavior

The server HTML contains <a href="/about" data-active aria-current="page">About</a>, so the first paint already matches what the client will apply. data-pending would always be absent on the server.

Platform
  • @solidjs/router 2.0.0-next.35 (origin/next at 7c24cf1)
  • @solidjs/web and solid-js 2.0.0-rc.11
  • Not specific to any OS or browser
Additional context

The router side looks small, because matchLink in utils.ts is already a pure function shared with useLinkState. The server knows the request path, query and base, so it could compute the same active and exact result. Only the document.baseURI and window.location.origin checks in managedPath would need a server equivalent that uses the request URL.

The harder part is in the compiler. In SSR output, a static anchor is baked into the template string, so there is no runtime point where anything could add an attribute:

// generate: "ssr"
var _tmpl$ = ["<nav", "><a href=\"/about\">About</a><a", " class=\"x\">Dyn</a>", "</nav>"];
// generate: "dom"
_$claimElement(_el$2); _$claimElement(_el$3); _$claimElement(_el$4);

One possible design:

  1. In SSR mode, dom-expressions gives every a[href] an attribute hole, for example _$ssrLinkClaim(href), which returns "" or data-active aria-current="page". The hole must not change hydration keys.
  2. @solidjs/web adds a server-side registration hook for that function and calls it from ssrElement for spread anchors. With nothing registered it returns "", so apps without the router pay only one call per anchor.
  3. The router registers a server handler for each request that calls matchLink, and honors explicitLinks by checking for the link attribute.

Hydration should not be affected: Solid does not compare attributes during hydration, and the client claim keeps control of the attributes after the first navigation.

A smaller, router-only alternative is an inline script in the SSR output that marks matching anchors before hydration. That removes the late highlight, but it does not help pages that never load scripts, and those are where the missing aria-current matters most.

Langage dominant
TypeScript
Étoiles
1.3k
Forks
184
Merge moyen
16 h 46 min
PR mergées (30 j)
52

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de solidjs/solid-router

Toutes les issues de solidjs/solid-router

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.