Skip to content

fix(viewer): square the terminal toolbar icon buttons - #119

Merged
code0xff merged 2 commits into
devfrom
fix/viewer-square-toolbar-buttons
Oct 28, 2026
Merged

code0xff merged 2 commits into
devfrom
fix/viewer-square-toolbar-buttons

Conversation

@code0xff

Copy link
Copy Markdown
Owner

Purpose

The icon buttons in the terminal tab row looked off-square: they were 24.5×17.5px, and the compose (pencil) button was 22.8×15.8px because of its smaller icon.

Change

  • Toolbar buttons are a fixed h-6 w-6 square with the icon centred (21×21px at the 14px root).
  • The pencil icon uses the default h-4 w-4 like the other icons.
  • The row grows from 24.5px to 28px tall.

Verification

  • Measured in Chromium against an isolated daemon, desktop (1280) and phone (390): every button is 21.0×21.0px.
  • npm test 936 passed, tsc -b ok, dist rebuilt.

🤖 Generated with Claude Code

https://claude.ai/code/session_01K9Y6nr4J3kMp6GVQNUd9Ry

code0xff and others added 2 commits October 28, 2026 09:03
The panel toolbar's icon buttons were padded wider than tall (24.5x17.5px
at the 14px root), and the compose button's smaller pencil left it a
different size again. Each button is now a fixed 1.5rem square with its
icon centred, and the pencil uses the default icon size like the rest.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K9Y6nr4J3kMp6GVQNUd9Ry
The terminal toolbar row is 3.5px taller with square buttons. The new
baselines are the screenshots CI rendered in the pinned Linux runtime;
each diff is confined to that row and what sits below it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01K9Y6nr4J3kMp6GVQNUd9Ry
@code0xff
code0xff merged commit ac732a8 into dev Oct 28, 2026
6 checks passed
@code0xff
code0xff deleted the fix/viewer-square-toolbar-buttons branch October 28, 2026 00:16
@whackur whackur mentioned this pull request Oct 28, 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