From e09e99308136994faa480458021d78ce40fa314e Mon Sep 17 00:00:00 2001 From: Tom Najdek Date: Mon, 10 Nov 2025 19:46:58 +0100 Subject: [PATCH] Align buttons with the top line of the field (#5604) --- chrome/content/zotero/elements/attachmentBox.js | 2 +- scss/abstracts/_mixins.scss | 9 ++++----- scss/elements/_attachmentBox.scss | 6 ++++-- scss/elements/_attachmentRow.scss | 1 + 4 files changed, 10 insertions(+), 8 deletions(-) diff --git a/chrome/content/zotero/elements/attachmentBox.js b/chrome/content/zotero/elements/attachmentBox.js index ecc166fc84..166fe065f6 100644 --- a/chrome/content/zotero/elements/attachmentBox.js +++ b/chrome/content/zotero/elements/attachmentBox.js @@ -64,7 +64,7 @@ diff --git a/scss/abstracts/_mixins.scss b/scss/abstracts/_mixins.scss index 380b9c45e7..89802fef71 100644 --- a/scss/abstracts/_mixins.scss +++ b/scss/abstracts/_mixins.scss @@ -115,7 +115,7 @@ @mixin clicky-item { display: flex; - align-items: flex-start; + align-items: baseline; gap: 4px; padding-inline-start: 4px; overflow: hidden; @@ -130,7 +130,7 @@ } .icon { - height: calc(1.3333333333 * var(--zotero-font-size)); + line-height: calc(1.3333333333 * var(--zotero-font-size)); } .label { @@ -140,9 +140,6 @@ width: 0; // Needed to allow the label to shrink for some reason flex: 1; overflow: hidden; - } - - .icon, .label { padding-block: 2px; } } @@ -177,6 +174,8 @@ width: 0; min-width: 100%; display: flex; + align-items: baseline; + toolbarbutton { margin-inline-start: 4px; } diff --git a/scss/elements/_attachmentBox.scss b/scss/elements/_attachmentBox.scss index 4412a3024a..ec2f645b34 100644 --- a/scss/elements/_attachmentBox.scss +++ b/scss/elements/_attachmentBox.scss @@ -23,8 +23,10 @@ attachment-box { @include meta-table; #index-status { + flex: 1 1 auto; padding-inline: var(--editable-text-padding-inline); margin-top: 2px; + @include comfortable { margin-top: 3px; } @@ -34,9 +36,9 @@ attachment-box { { height: 22px; width: 22px; - padding: 1px; + padding: 4px; margin-left: 4px; - @include svgicon-menu("sync", "universal", "20"); + @include svgicon-menu("sync", "universal", "16"); &:not([disabled='true']) { color: var(--fill-secondary); } diff --git a/scss/elements/_attachmentRow.scss b/scss/elements/_attachmentRow.scss index 3fbd46c2f8..338e2527e0 100644 --- a/scss/elements/_attachmentRow.scss +++ b/scss/elements/_attachmentRow.scss @@ -11,6 +11,7 @@ attachment-row { display: flex; align-items: center; gap: 4px; + align-items: baseline; .clicky-item { @include clicky-item;