Skip to content

fix(ios): trim repeated and self-evident explanatory copy - #37

Open
1395490241lhj-code wants to merge 2 commits into
mainfrom
claude/trim-explanatory-copy
Open

1395490241lhj-code wants to merge 2 commits into
mainfrom
claude/trim-explanatory-copy

Conversation

@1395490241lhj-code

Copy link
Copy Markdown
Owner

Summary

An audit of the iOS app's secondary explanatory text found many footers and captions repeating each other or explaining the obvious. This PR keeps each trust/data fact once, at the decision point, and removes the rest. Copy only: no logic, data, or navigation changes.

Removed (duplicates)

  • Settings guest footer: lead sentence "无需登录即可继续使用全部本机功能。" repeated the row's 本机功能已全部可用.
  • Settings 关于: "登录不会自动上传本机厨房数据" (the login page already states it).
  • Settings 清除全部本地数据 footer: "此操作无法撤销,且不会影响远端菜谱库。" (the alert says both). The recipe-backup warning stays.
  • Guest merge entry footer "购物清单、计划和菜谱不受影响,本次只涉及库存。" (the preview says it at the decision point).
  • Guest merge preview "可以随时取消,也可以稍后处理。" (the 取消本次合并 button sits right below).
  • Special plan "菜品引用菜谱库,删除计划不会删除菜谱。" (the delete alert says 菜谱库里的菜谱不受影响).
  • Account deletion body no longer repeats its own header 这与退出登录不同.

Removed (self-evident or wrong)

  • Reminder footer: "首次开启提醒时,会请求系统通知权限。" (kept: 补货提醒 9 点送达).
  • AI settings troubleshooting footer.
  • Prepared-dish date footer ("默认日期只是…可以按自己的判断修改").
  • Weekly rhythm "这是每周的默认安排。" (restates the title).
  • Guest merge "返回上一页即可查看处理结果…" and "仅供查看,不在此修改。", which was wrong: the review page offers 修改选择 before confirming.
  • Shopping add confirmation trimmed to "已添加「X」。"

Shortened: special plan AI provenance footer, today-rhythm reset footer, recipe/receipt photo tips.

Kept on purpose

  • Restore preview restorePreview.replacementWarning: SafeRestoreWorkflowUITests requires it before confirmation, untruncated at AX XXXL.
  • Account page sign-out/delete footers: cited by docs/contracts/ACCOUNT_DELETION_DESIGN.md, and the two destructive actions sit side by side.
  • Photo-upload privacy footers: their retention claims differ between recipe and receipt flows; this is being verified against server behaviour separately.

Tests

  • Pinned strings updated: SettingsExperienceUITests, GuestMergeUIPhase2B3UITests, and test/ios-native-auth-phase1.test.mjs. The latter now checks the row's 本机功能已全部可用, which keeps the "guest mode is complete" invariant.
  • GuestMergeSummaryUI5B2BB2AUITests: reads guestMergeConfirmSupportingCopy after scrolling it into view (lazy Form; the shorter page stopped scrolling with it off screen), and asserts the remaining trust footnote instead of the removed one.
  • PR31VisualAcceptanceUITests.testBackupScopeAndClearDataWarnings: keeps scrolling (max 3) until the footer clears the tab bar. reveal only guarantees a 40pt margin, while the tab bar is 83pt tall. A real overlap still fails.

Validation (local, xcodebuild fallback, iOS Simulator)

  • build-for-testing (KitchenManager scheme): succeeded; no new warnings in changed files.
  • Node source tests (7 iOS-native files): 179/179 pass.
  • UI tests, serial: SettingsExperience, GuestMergeUIPhase2B3, GuestMergeSummaryUI5B2BB2A, GuestMergeChoiceEditingUI5B2BB2B, SpecialPlanMenu, SpecialPlanNormalSession, ComponentMeal, and 3 PR31 tests (backup/clear-data, account deletion, special-plan confirmations). First run: 72/74. Both failures came from scroll geometry and were fixed as above. The rerun of GuestMergeSummary (21) plus the clear-data test passed 22/22.
  • Screenshots checked: settings (standard + scrolled bottom), merge preview, special plan detail, clear-data footer at AX XXXL dark.
  • Not run: full UI suite; today-rhythm sheet, photo-import empty states, shopping add confirmation, and account deletion intro have no copy assertions and were not screenshot-checked.

🤖 Generated with Claude Code

1395490241lhj-code and others added 2 commits October 23, 2026 13:22
Each trust or data fact now appears once, where the user makes the
decision; lines that restated a title, a row or the next screen are gone.

- Settings: drop the guest footer's lead sentence (the row already says
  本机功能已全部可用), the About line's login/upload repeat, the
  permission-prompt and AI troubleshooting footers, and the clear-data
  footer line its alert already states.
- Guest merge: drop the entry footer and the 稍后处理 line the preview
  already covers, the back-navigation hint, and 仅供查看,不在此修改,
  which was wrong: the review page lets you change choices before
  confirming.
- Special plan, day rhythm, prepared dates, account deletion, photo tips:
  remove repeats and shorten long footers.

Kept on purpose: the restore preview's replacement warning and the
account page's sign-out/delete footers (test- and contract-pinned).

Tests: update the pinned guest-footer strings; scroll the merge supporting
copy and the clear-data footer into view before asserting, since the
shorter pages now stop scrolling at a different spot.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Resolve PreparedComponentViews: keep main's draft-based editor (DEBUG
stale-edit fixture and 记录有更新 recheck section) and drop the date
footer, as this branch intends.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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