Skip to content

docs(ui): mark item G verified — scroll reset + copy flash, close v1.18 wave - #54

Merged
argszero merged 1 commit into
mainfrom
docs/ui-ue-round3-g-wrap
Aug 17, 2026
Merged

argszero merged 1 commit into
mainfrom
docs/ui-ue-round3-g-wrap

Conversation

@argszero

Copy link
Copy Markdown
Owner

Summary

Wave wrap-up for item G (其他细节) of the UI/UE round-3 wave (rant 2026-08-17T18:06:09, version v1.18).

Item G has two acceptance points, and both were already implemented in the codebase — this PR verifies them with a DOM-stub smoke test and documents the confirmation (docs-only change):

  1. 视图切换后滚动位置复位 (scroll reset on view switch) — switchView already sets $("#main").scrollTop = 0 after rendering; .main is the scroll container (flex: 1; overflow-y: auto). Verified: switch → #main.scrollTop resets to 0, repeatedly.
  2. 复制成功按钮短暂变「已复制 ✓」 (copy button flash) — copyKey already implements flash(ok): on success the button turns into「已复制 ✓」(disabled), restores after 1.2s; degraded path shows「请 Ctrl+C」(in place since v1.16). Verified: click → 已复制 ✓ + disabled → restore after 1.2s.

What this PR changes

No production code change — the rant's G items were already satisfied; this documents and closes the wave.

Verification

  • DOM-stub smoke test (5 assertions): G1 scroll reset ×2, G2 flash / disabled / restore ✓
  • node --check ✓ · cargo test ✓ · cargo fmt --check ✓

Rant (verbatim)

G. 其他细节

  • 侧边栏视图切换后滚动位置复位(main 回到顶部)
  • 复制成功时按钮短暂变「已复制 ✓」(当前只有 toast + key 闪烁)

Docs

  • docs/user-stories.md v1.18 entry completed for G
  • ui/README.md: "其他细节约定" section added

@argszero
argszero merged commit a544a80 into main Aug 17, 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.

1 participant