Skip to content

Use normal weight and primary text for worker details stats - #3962

Merged
rosanusi merged 2 commits into
mainfrom
worker-ui-fix
Sep 30, 2026
Merged

rosanusi merged 2 commits into
mainfrom
worker-ui-fix

Conversation

@rosanusi

@rosanusi rosanusi commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Restyles the stat text on the worker details page (/namespaces/{namespace}/workers/{workerInstanceKey}):

  • Removes font-semibold from the five stat numbers and from the CPU / Memory usage labels, so they render at normal weight.
  • Swaps the five stat numbers from text-brand to text-primary. text-brand is not used for static text anywhere else in the UI — elsewhere it is reserved for links, hover states, and icon accents.

Affected values: slots used, tasks processed, poller count, cache size, and cache hit rate.

Before

Screenshot 2026-09-18 at 12 50 40 PM

After

Screenshot 2026-09-28 at 2 56 51 PM

Drop font-semibold from the worker detail stat numbers and the CPU and
memory labels, and swap the stat numbers from text-brand to text-primary.
text-brand is not used for static text elsewhere in the UI.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
holocene Ready Ready Preview Sep 30, 2026 6:01pm UTC

Request Review

@rosanusi
rosanusi requested a review from laurakwhit September 29, 2026 20:27
@rosanusi
rosanusi marked this pull request as ready for review September 29, 2026 20:48
@rosanusi
rosanusi requested a review from a team as a code owner September 29, 2026 20:49
{translate('workers.tasks-processed')}
</dt>
<dd class="font-mono text-2xl font-semibold text-brand">
<dd class="font-mono text-2xl text-primary">

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

text-primary is already the default so it is redundant in some of these cases. Went ahead and removed here 🗑️

@rosanusi
rosanusi merged commit a66748b into main Sep 30, 2026
19 checks passed
@rosanusi
rosanusi deleted the worker-ui-fix branch September 30, 2026 18:52

This branch was successfully deployed

1 active deployment
Preview — 1e5a4d1a Deployed Sep 30, 2026 by vercel[bot]
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.

2 participants