fix: update account status screen with new ui design system cp-7.73.0#28507
Conversation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit f1673bb. Configure here.
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: Key findings:
SmokeWalletPlatform is selected because the seedless onboarding tests (google-login-existing-user, apple-login-existing-user) that directly exercise the AccountStatus screen are tagged under this suite. No other test suites directly test this component. Performance Test Selection: |
|
✅ E2E Fixture Validation — Schema is up to date |
|
|
lgtm. Just want to confirm with product or design team why we have different cta's for same button in android and ios. cc @nakulmk |
|
Due to apple rejection last year with the reason there should be no password needed after apple login |





Description
The Account Status view (social login “wallet found” / “wallet not found”) was built with legacy component-library
Text,Button, and a localindex.styles.tsStyleSheet.Reason: Align this onboarding screen with
@metamask/design-system-react-nativeso typography, buttons, colors, and spacing stay consistent with the rest of the app and are easier to maintain.Solution: Replace
View/ScrollViewlayout wrappers with design systemBox, swap to design systemTextandButton(includingButtonVariant,isFullWidth, and children instead oflabel), and style the screen withuseTailwind()plus semantic tokens (bg-default, etc.). Removeindex.styles.ts. UseuseWindowDimensionsfor responsive width and compute the hero image size from available width (capped at 343px, preserving aspect ratio). Footer spacing uses platform-specific Tailwind classes (mb-4on iOS,mb-6on Android). Snapshot tests were updated to reflect the new component tree and styles.Changelog
CHANGELOG entry: Updated the social login account status screen to use MetaMask design system layout and buttons
Related issues
Fixes: #28505
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Low Risk
Primarily a UI refactor to the Account Status onboarding screen (layout/styling and component swaps) with no meaningful changes to navigation, analytics, or data handling; main risk is visual/regression differences across platforms/screensizes.
Overview
Updates the
AccountStatusonboarding view to use@metamask/design-system-react-nativecomponents (Box,Text,Button) and Tailwind styling (useTailwind) instead of the legacy component-library widgets and localStyleSheet.Adds responsive hero image sizing via
useWindowDimensions+ memoized width/height calculation, tweaks scroll/content and platform-specific footer spacing, and removesindex.styles.ts. Snapshot tests are updated to match the new component tree and style output.Reviewed by Cursor Bugbot for commit 66faf33. Bugbot is set up for automated code reviews on this repo. Configure here.