Skip to content

Spin the refresh icon in place - #90

Merged
MathiasWP merged 1 commit into
mainfrom
center-refresh-spinner
Aug 24, 2026
Merged

Spin the refresh icon in place#90
MathiasWP merged 1 commit into
mainfrom
center-refresh-spinner

Conversation

@MathiasWP

Copy link
Copy Markdown
Owner

The rotating "refresh endpoints" spinner drifted off centre as it turned.

text-3 is 0.75rem, and the root font-size is 13px, so presetIcons' 1.2em box came out 11.7px. A mask that size resamples to a slightly different centre at each angle, so the icon orbited rather than spun in place. Both refresh-cw icons in the sidebar now use a whole-pixel w-12px h-12px box.

Measured in headless Chrome with this repo's generated UnoCSS — centroid of the rendered icon across 12 angles:

box 1x 2x
11.7px (text-3) 0.41 × 0.61 px drift 0.33 × 0.31 px
12px (w-12px h-12px) 0.06 × 0.04 px 0.04 × 0.05 px

pnpm check passes. Measured in Chromium rather than the WKWebView the app actually ships in, but the cause — a fractional-pixel mask box — is engine-independent.

@MathiasWP
MathiasWP merged commit 576eada into main Aug 24, 2026
4 checks passed
@MathiasWP
MathiasWP deleted the center-refresh-spinner branch August 24, 2026 16:46
@github-actions github-actions Bot mentioned this pull request Aug 24, 2026
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