Skip to content

fix: separate donut labels and align section spacing - #163

Merged
f0rr0 merged 5 commits into
nextfrom
f0rr0/chart-label-spacing
Sep 23, 2026
Merged

f0rr0 merged 5 commits into
nextfrom
f0rr0/chart-label-spacing

Conversation

@f0rr0

@f0rr0 f0rr0 commented Sep 23, 2026 •

Copy link
Copy Markdown
Owner

Donut callouts could overlap the ring or each other, especially “Other activity” and adjacent small slices. Place labels outside the ring with straight connectors and spacing based on the existing 24px text rhythm.

Normalize section headings to a 32px row with a 16px content gap while retaining 44px control targets. Move the Work page’s refresh control into the existing space above the feed so an invisible control no longer creates an empty header row.

Token breakdowns select the longest available period by default (Past year with the current configuration), falling back to 30 or 7 days when needed. Verified the initial selection in the browser.

Give cumulative token totals 15% headroom with rounded axis ticks (80B and 160B for the current data), and add the homepage’s “All open source” link to the configured GitHub profile’s repositories sorted by stars. Verified both in the browser.

Validation: 20 focused tests, lint and typecheck; browser checks across 320, 390, 640, 768 and 1440px for all three chart periods; desktop/mobile visual checks for chart and Work layouts, including the visible refresh state. Local Work verification uses a fixture from the public production feed.

@vercel

vercel Bot commented Sep 23, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
f0rr0-github-io Ready Ready Preview Sep 23, 2026 9:06pm UTC

@f0rr0
f0rr0 merged commit f9e77b1 into next Sep 23, 2026
4 checks passed
@f0rr0
f0rr0 deleted the f0rr0/chart-label-spacing branch September 23, 2026 21:07

This branch was successfully deployed

1 active deployment
Preview — 0ee0cba0 Deployed Sep 23, 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.

1 participant