Skip to content

fix(settings): align 返回应用 back button to settings nav row recipe - #226

Merged
jackwener merged 1 commit into
mainfrom
yuejing/fix-back-button-align
Jun 24, 2026
Merged

fix(settings): align 返回应用 back button to settings nav row recipe#226
jackwener merged 1 commit into
mainfrom
yuejing/fix-back-button-align

Conversation

@jackwener

Copy link
Copy Markdown
Member

WAWQAQ msg `dd21e793` 2026-06-25 #5/5:

"返回应用 的样式和其他的侧边栏的样式不是一样的,这样子也不美观。就它一个居中,然后它的样式和其他的 通用 外观 这些的样式都不一样。"

Bug

`.settingsBackButton` and `.settingsNavItem` were styled completely differently — back link inline-flex / 13px / 4-8px padding / 6px radius / 4px gap, nav items flex full-width / 14px / 6-12px padding / 7px radius / 12px gap. The back link looked intrinsically sized + visually centered, mismatching the full-width left-aligned 通用 / 外观 / etc. rows below it.

Fix

Rewrite `.settingsBackButton` to inherit the `.settingsNavItem` recipe verbatim (width, height, padding, radius, gap, font, color tokens). Tighten the bottom margin from 10px to 4px so the back row joins the nav stack rhythm.

Diff

`1 file, 22 insertions(+), 8 deletions(-)`. CSS-only.

WAWQAQ msg `dd21e793` 2026-06-25: "返回应用 的样式和其他的侧边栏的样式
不是一样的,这样子也不美观。就它一个居中,然后它的样式和其他的 通用
外观 这些的样式都不一样。"

The back link used:
- `display: inline-flex` (intrinsic width — visually centered in
  the column layout instead of full-width like every nav row)
- 4px 8px padding
- 13px / weight 500
- 6px radius
- 4px gap
- `align-self: flex-start`

while every `.settingsNavItem` below it used:
- `display: flex` with `width: calc(100% - 8px)`
- 38px height
- 6px 12px padding
- 14px / weight 500
- 7px radius
- 12px gap

The mismatch read as "this row doesn't belong to the same list."

Fix: rewrite `.settingsBackButton` to match `.settingsNavItem`
exactly on width, height, padding, radius, gap, font, and color
tokens. Keep the no-drag region. Tighten the bottom margin from
10px to 4px so the back row sits in the same 2px stack rhythm as
the nav-group items below.

CSS-only. Single block changed.
@jackwener
jackwener merged commit 69a99fe into main Jun 24, 2026
@jackwener
jackwener deleted the yuejing/fix-back-button-align branch June 24, 2026 19:26
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