Skip to content

fix(account): clarify two-step verification toggle label#8792

Merged
wangsijie merged 2 commits into
masterfrom
wangsijie-fix-two-step-verification-label
May 13, 2026
Merged

fix(account): clarify two-step verification toggle label#8792
wangsijie merged 2 commits into
masterfrom
wangsijie-fix-two-step-verification-label

Conversation

@wangsijie
Copy link
Copy Markdown
Contributor

Summary

  • Use a neutral "2-step verification" label for the account security MFA toggle so the switch clearly represents the feature state.

Fixes #8791

Testing

  • Unit tests

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 12, 2026 02:14
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 12, 2026

COMPARE TO master

Total Size Diff 📉 -1.38 KB

Diff by File
Name Diff
.changeset/clarify-two-step-toggle-label.md 📉 -124 Bytes
packages/account/src/pages/Security/MfaSection/index.tsx 📉 -150 Bytes
packages/phrases-experience/src/locales/ar/account-center.ts 📉 -75 Bytes
packages/phrases-experience/src/locales/cs/account-center.ts 📉 -68 Bytes
packages/phrases-experience/src/locales/de/account-center.ts 📉 -70 Bytes
packages/phrases-experience/src/locales/en/account-center.ts 📉 -64 Bytes
packages/phrases-experience/src/locales/es/account-center.ts 📉 -71 Bytes
packages/phrases-experience/src/locales/fr/account-center.ts 📉 -77 Bytes
packages/phrases-experience/src/locales/it/account-center.ts 📉 -71 Bytes
packages/phrases-experience/src/locales/ja/account-center.ts 📉 -68 Bytes
packages/phrases-experience/src/locales/ko/account-center.ts 📉 -58 Bytes
packages/phrases-experience/src/locales/pl-pl/account-center.ts 📉 -69 Bytes
packages/phrases-experience/src/locales/pt-br/account-center.ts 📉 -72 Bytes
packages/phrases-experience/src/locales/pt-pt/account-center.ts 📉 -72 Bytes
packages/phrases-experience/src/locales/ru/account-center.ts 📉 -99 Bytes
packages/phrases-experience/src/locales/th/account-center.ts 📉 -121 Bytes
packages/phrases-experience/src/locales/tr-tr/account-center.ts 📉 -65 Bytes
packages/phrases-experience/src/locales/uk-ua/account-center.ts 📉 -97 Bytes
packages/phrases-experience/src/locales/zh-cn/account-center.ts 📉 -55 Bytes
packages/phrases-experience/src/locales/zh-hk/account-center.ts 📉 -55 Bytes
packages/phrases-experience/src/locales/zh-tw/account-center.ts 📉 -58 Bytes

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR resolves UX ambiguity on the Account Security page by changing the 2-step verification toggle label from an action-based phrase (“Turn on/off …”) to a neutral, state-independent feature name (“2-step verification”), aligning the text with the toggle’s on/off meaning (Fixes #8791).

Changes:

  • Replaced the conditional “Turn on/off 2-step verification” toggle title with a neutral “2-step verification” label.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@wangsijie wangsijie requested a review from gao-sun as a code owner May 12, 2026 02:21
@github-actions github-actions Bot added size/s and removed size/xs labels May 12, 2026
@wangsijie wangsijie merged commit 32c40b1 into master May 13, 2026
42 of 44 checks passed
@wangsijie wangsijie deleted the wangsijie-fix-two-step-verification-label branch May 13, 2026 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

UX Ambiguity: Confusing labeling for 2-Step Verification toggle

4 participants