[www] associates model id with copy id button

[Prathik Rao (prathikr)]

fixes #1019

[vercel]

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

Project	Deployment	Actions	Updated
foundry-local	Ready	Preview	Sep 17, 2026 8:44pm UTC

[Copilot]

🟡 Changes recommended

Accessible naming and formatting issues must be addressed before approval.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This pull request associates visible model IDs with Copy ID controls and adds accessibility regression coverage.

Changes:

• Adds model ID attributes and ARIA references.
• Adds accessibility assertions.

File summaries

File	Description
www/src/routes/models/components/ModelDetailsModal.svelte	Associates model IDs with Copy ID buttons.
www/test/accessibility.test.mjs	Tests the new accessibility attributes.

Review details

Suppressed comments (4)

www/src/routes/models/components/ModelDetailsModal.svelte:465

• Because aria-labelledby takes precedence over a button's own contents in the accessible-name computation, this makes the control's name only the model ID; screen-reader users lose the "Copy ID" action even though it remains visible. Preserve the action in the accessible name—for example, reference both the model ID and a span containing "Copy ID", or use the model ID as aria-describedby.

											aria-labelledby={`model-id-${variant.name}`}

www/src/routes/models/components/ModelDetailsModal.svelte:506

• Because aria-labelledby takes precedence over a button's own contents in the accessible-name computation, this makes the control's name only the model ID; screen-reader users lose the "Copy ID" action even though it remains visible. Preserve the action in the accessible name—for example, reference both the model ID and a span containing "Copy ID", or use the model ID as aria-describedby.

								aria-labelledby={`model-id-${genericModelName}`}

www/src/routes/models/components/ModelDetailsModal.svelte:602

• Because aria-labelledby takes precedence over a button's own contents in the accessible-name computation, this makes the control's name only the model ID; screen-reader users lose the "Copy ID" action even though it remains visible. Preserve the action in the accessible name—for example, reference both the model ID and a span containing "Copy ID", or use the model ID as aria-describedby.

											aria-labelledby={`model-id-${variant.name}`}

www/src/routes/models/components/ModelDetailsModal.svelte:602

• This attribute is indented one tab farther than the adjacent Button props and the analogous attribute above. The website's lint script runs prettier --check . (www/package.json:26), so align it with the surrounding props before merging.

											aria-labelledby={`model-id-${variant.name}`}

• Files reviewed: 2/2 changed files
• Comments generated: 2
• Review effort level: Lite (auto)

Note

Copilot is running an experiment and ran this review at Lite.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.