Skip to content

fix(i18n): 反馈浮钮避开主操作区 + 英文设置页网络依据双语(dev-board#574 #575) - #799

Merged
zeweihan merged 1 commit into
masterfrom
claude/en-feedback-keep-clear-574
Sep 10, 2026
Merged

fix(i18n): 反馈浮钮避开主操作区 + 英文设置页网络依据双语(dev-board#574 #575)#799
zeweihan merged 1 commit into
masterfrom
claude/en-feedback-keep-clear-574

Conversation

@zeweihan

Copy link
Copy Markdown
Owner

内容

验证

  • app-e2e 全量:修复前(仅新门禁)140 通过 1 失败(elementFromPoint 命中 SPAN "Feedback");修复后 142 通过 0 失败,J12 工具名断言真执行。
  • node --test tests/feedback-widget/keep-clear.test.mjs 7/7。
  • NetworkRegionServiceTest 红(英文期望拿到中文)→ 绿。
  • 真渲染 en/zh × 1440x900/1280x720,发送键中心 elementFromPoint 均命中按钮自身。

补丁边界内(仅 frontend/ backend/src),随 v0.38.2 发(dev-board#571)。

🤖 Generated with Claude Code

- #574 英文空会话占位折行把发送键推进右缘反馈浮钮范围,点击被浮钮吃掉。
  改为主操作区打 data-awd-keep-clear 声明、浮钮只沿竖直方向让路
  (utils/keepClear.js 纯几何 + FeedbackWidget 低频重判,让路不持久化)。
- app-e2e J12 点发送前断言 elementFromPoint 命中按钮自身,被盖住判红而非 skip;
  工具名断言只在消息确实发出后执行。修复前 140/1 红,修复后 142/0。
- #575 NetworkRegionService.detectionBasis 改 LangText 双语,补中英单测。
- feedback-optimizer.md 补 keep-clear 契约地雷。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@zeweihan
zeweihan enabled auto-merge (squash) September 10, 2026 07:16
@zeweihan
zeweihan merged commit 120fd7f into master Sep 10, 2026
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 10, 2026
@zeweihan
zeweihan deleted the claude/en-feedback-keep-clear-574 branch September 10, 2026 07:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant