The .topic-sidebar-tools .btn flattening override also hit the Reply button (btn-primary in harmony's reply-button partial), turning it grey. Exclude btn-primary so Reply keeps the theme's gold plate.
The `.topic-sidebar-tools .btn` flattening override also hit the Reply button (`btn-primary` in harmony's reply-button partial), turning it grey. Exclude `btn-primary` so Reply keeps the theme's gold plate.
Fixes #34
The topic-sidebar-tools flattening override was hitting the Reply
button (btn-primary) too, turning it grey. Exclude btn-primary so
Reply keeps the theme's gold plate.
Fixes#34
- Blockquotes: replace the 3px gold side-stripe and warm-brown fill
with a full 1px gilt border and plum tint, per the Gilt Edge Rule
- Snap pagination shadow alpha to the documented velvet-hover value
and unify the vote glyph size at 0.95rem
- Document the already-shipped code-syntax and vote-state colors and
the small type steps (0.95/0.82/0.78rem) in DESIGN.md + sidecar
archvillainette
scheduled this pull request to auto merge when all checks succeed 2026-07-19 08:41:27 +00:00
archvillainette
canceled auto merging this pull request when all checks succeed 2026-07-19 08:43:22 +00:00
gitea-bot
approved these changes 2026-07-19 08:48:41 +00:00
LGTM. :not(.btn-primary) on the sidebar-tools flattening rule is the correct root fix: the old background: ... !important shorthand was wiping the gold gradient from _controls.scss, and the exclusion keeps any future primary button in that container gold too, consistent with the theme's one-gold-surface convention.
Minor notes, non-blocking:
Commit 376caf4 bundles unrelated design churn (design.json colors, DESIGN.md type ramp, blockquote restyle, icon/shadow tweaks) not mentioned in the PR body — fine per our reuse-the-branch convention, just noting the diff is mostly that.
crypt-black tonalRamp now has 9 entries while all others have 8; double-check the design-detector doesn't assume a fixed ramp length.
LGTM. `:not(.btn-primary)` on the sidebar-tools flattening rule is the correct root fix: the old `background: ... !important` shorthand was wiping the gold gradient from `_controls.scss`, and the exclusion keeps any future primary button in that container gold too, consistent with the theme's one-gold-surface convention.
Minor notes, non-blocking:
- Commit 376caf4 bundles unrelated design churn (design.json colors, DESIGN.md type ramp, blockquote restyle, icon/shadow tweaks) not mentioned in the PR body — fine per our reuse-the-branch convention, just noting the diff is mostly that.
- `crypt-black` tonalRamp now has 9 entries while all others have 8; double-check the design-detector doesn't assume a fixed ramp length.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
gitea-bot
The
.topic-sidebar-tools .btnflattening override also hit the Reply button (btn-primaryin harmony's reply-button partial), turning it grey. Excludebtn-primaryso Reply keeps the theme's gold plate.Fixes #34
LGTM.
:not(.btn-primary)on the sidebar-tools flattening rule is the correct root fix: the oldbackground: ... !importantshorthand was wiping the gold gradient from_controls.scss, and the exclusion keeps any future primary button in that container gold too, consistent with the theme's one-gold-surface convention.Minor notes, non-blocking:
376caf4bundles unrelated design churn (design.json colors, DESIGN.md type ramp, blockquote restyle, icon/shadow tweaks) not mentioned in the PR body — fine per our reuse-the-branch convention, just noting the diff is mostly that.crypt-blacktonalRamp now has 9 entries while all others have 8; double-check the design-detector doesn't assume a fixed ramp length.