Skip to content

#2003 Centre the action icon on the macro box and drop its scrollbar - #2006

Merged
TimothyLuke merged 1 commit into
TimothyLuke:masterfrom
LarryThiessen:feat-2003-macro-block-polish
Aug 23, 2026
Merged

#2003 Centre the action icon on the macro box and drop its scrollbar#2006
TimothyLuke merged 1 commit into
TimothyLuke:masterfrom
LarryThiessen:feat-2003-macro-block-polish

Conversation

@LarryThiessen

Copy link
Copy Markdown
Contributor

Fixes #2003

Two leftovers from before the macro command box sized itself to its content.

The action icon sat at a fixed position in its slot, so it drifted out of line as the box grew and shrank. Its CENTER is now anchored to the box's LEFT edge — a live anchor follows every height change with no per-resize maths. The icon slot's DoLayout is wrapped so a re-layout cannot snap the icon back to its static slot position.

The scrollbar never has anything to scroll now that the box fits its content, so it is hidden. A block is capped at 255 characters and cannot realistically reach the 24-row cap; one that did needs rethinking rather than a scrollbar.

Testing

  • luac -p clean.
  • Confirmed in a running client: icon stays centred as a block grows and shrinks; no scrollbar on the command box.

🤖 Generated with Claude Code

… scrollbar

Two leftovers from before the macro command box sized itself to its
content.

The action icon sat at a fixed position in its slot, so it drifted out
of line as the box grew and shrank. Anchor its CENTER to the box's LEFT
edge instead: a live anchor follows every height change with no
per-resize maths. The icon slot's DoLayout is wrapped so a re-layout
cannot snap the icon back to its static slot position.

The box now grows to fit its content, so its scrollbar never has
anything to scroll -- hide it. A block is capped at 255 characters and
cannot realistically reach the 24-row cap.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@TimothyLuke
TimothyLuke merged commit 41fcb17 into TimothyLuke:master Aug 23, 2026
1 check 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.

Macro block: centre the action icon on the command box, drop its dead scrollbar

2 participants