Skip to content

feat(web): save 按钮改成 icon-only,与 ☆/✎ 风格统一#100

Merged
github-actions[bot] merged 2 commits into
mainfrom
judy/save-icon-button
Jun 16, 2026
Merged

feat(web): save 按钮改成 icon-only,与 ☆/✎ 风格统一#100
github-actions[bot] merged 2 commits into
mainfrom
judy/save-icon-button

Conversation

@SymbolStar

Copy link
Copy Markdown
Owner

Scott 截图反馈:viewer toolbar 里 ☆(收藏)和 ✎(编辑)都是 icon 按钮,只有「保存」是带文字的蓝底按钮,宽度被压窄时还会折成上下两行「保/存」,视觉上很突兀。

改动:

  • web/index.html:把 #btn-save-file 换成 icon-only-btn,内嵌软盘 SVG,title/aria-label='保存';去掉 primary-btn 蓝底,跟旁边两个 icon 视觉权重统一。
  • web/app.jssaveRef 的进度反馈从 textContent='保存中…' 改成 title + is-saving class,避免覆盖 SVG 子元素;网络错误/正常返回两条路径都恢复 title 和 class。

行为不变;只是外观对齐。

Scott 看截图反馈:保存按钮还是文字「保存」,跟收藏(☆)/编辑(✎)
两个 icon 按钮风格不一致,宽度被压窄时还会折成上下两行「保/存」。

- index.html: 把 btn-save-file 换成 icon-only-btn,内嵌软盘 SVG,
  title/aria-label='保存';移除 primary-btn 蓝底(与 ☆/✎ 视觉权重统一)
- app.js: saveRef 进度反馈从 textContent 改成 title + is-saving class,
  避免覆盖 SVG 内容;保留所有错误分支的状态恢复
@github-actions

github-actions Bot commented Jun 16, 2026

Copy link
Copy Markdown

🤖 bot-review (comment-only · phase 1)

Diff: 3 files changed, 17 insertions(+), 6 deletions(-) @ d932b41

Red-line checks:

  • ✅ A-7.5: no new 'forbidden' code in xiaof

Phase 2: auto-approve + auto-merge fire only when red-lines are clean, author is internal, and no needs-human path is touched. Block with no-auto-merge label or [no-auto-merge] in title.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Auto-approved by bot-review: red-line checks clean, internal author, no needs-human paths.

@github-actions github-actions Bot enabled auto-merge (squash) June 16, 2026 13:53
PR #99 合进 main 之后 ruff lint 一直 fail (I001 import block
unsorted)。我这条 PR 的 CI 顺带踩到,顺手修了。

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Auto-approved by bot-review: red-line checks clean, internal author, no needs-human paths.

@github-actions github-actions Bot merged commit 78d700c into main Jun 16, 2026
7 checks 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