From 3ac4249651c39daa68534875f0eec4dbc692dcfa Mon Sep 17 00:00:00 2001 From: Brandon Oxendine Date: Wed, 19 Aug 2026 12:20:29 +0200 Subject: [PATCH] Match the reaction and edit button sizes The reaction trigger sits in a block wrapper, so its icon formed a line box and took the leading with it, leaving the button taller than the edit one beside it. It also carried its own padding, colour and hover fill, on top of the ones the action around it already provides. The action is now a fixed square with its content centred, so its size no longer depends on the shape of whatever it wraps. --- src/components/Revision.svelte | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) diff --git a/src/components/Revision.svelte b/src/components/Revision.svelte index 9c649ab68..e67edd74e 100644 --- a/src/components/Revision.svelte +++ b/src/components/Revision.svelte @@ -1106,19 +1106,33 @@ align-items: center; gap: 0.125rem; } + /* A fixed square rather than padding around whatever is nested inside: the + reaction trigger sits in a block wrapper, so its icon forms a line box and + took the leading with it, making the button taller than the edit one. */ .body-action { display: inline-flex; align-items: center; justify-content: center; + flex: none; + width: 1.5rem; + height: 1.5rem; background: none; border: none; - padding: 0.25rem; + padding: 0; border-radius: var(--border-radius-sm); cursor: pointer; color: var(--color-text-tertiary); opacity: 0; transition: opacity 150ms ease; } + /* The trigger brings its own padding, colour and hover fill; the body action + around it already provides all three. */ + .body-action :global(.global-icon-button) { + display: flex; + padding: 0; + color: inherit; + background-color: transparent; + } .patch-body:hover .body-action, .patch-body:focus-within .body-action, .body-action:focus-visible {