Skip to content

Add tabular-nums to all dynamic numeric displays#255

Merged
Dbochman merged 1 commit into
mainfrom
feature/tabular-nums
Mar 4, 2026
Merged

Add tabular-nums to all dynamic numeric displays#255
Dbochman merged 1 commit into
mainfrom
feature/tabular-nums

Conversation

@Dbochman

@Dbochman Dbochman commented Mar 4, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds font-variant-numeric: tabular-nums (via Tailwind's tabular-nums class) to ~30 numeric elements across 11 components
  • Prevents layout shift when numbers update by using equal-width digits
  • Covers analytics cards, SLO tool, MTTR builder, Lighthouse scores, GitHub billing, and traffic quality displays

Test plan

  • Verify analytics dashboard numbers render with consistent digit widths
  • Drag MTTR/SLO sliders and confirm numbers don't cause horizontal jitter
  • Check GitHub billing card dollar amounts align properly in columns
  • Verify Lighthouse score badges maintain consistent width

🤖 Generated with Claude Code

Prevents layout shift when numbers change by using equal-width digits
across analytics cards, SLO tool, MTTR builder, and billing components.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

🤖 Codex Review

  1. Summary: This PR adds the Tailwind tabular-nums utility to many numeric UI elements across analytics and SLO components so changing numbers keep consistent character widths. The changes are presentation-only (className additions / small span wrappers), with no logic changes.

  2. Review:

  1. Verdict: Needs Discussion (mainly to align title/scope; code quality looks fine otherwise).

Automated review by OpenAI Codex • Skip with [skip-review] in PR title or commit message

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying personal-website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 199db2c
Status: ✅  Deploy successful!
Preview URL: https://864afccf.personal-website-adg.pages.dev
Branch Preview URL: https://feature-tabular-nums.personal-website-adg.pages.dev

View logs

@Dbochman Dbochman merged commit 509fc76 into main Mar 4, 2026
3 checks passed
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.

1 participant