Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(suite-native): revise and unify spacing of account-related screens #15234

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

yanascz
Copy link
Contributor

@yanascz yanascz commented Nov 5, 2024

  • Horizontal screen padding unified to 16px for relevant account screens
  • Removed invalid margin around graph on the account detail screen

Related Issue

Follow-up for #14065

Screenshots:

Simulator Screenshot - iPhone 16 Pro - 2024-11-05 at 14 56 06
Simulator Screenshot - iPhone 16 Pro - 2024-11-05 at 14 56 09
Simulator Screenshot - iPhone 16 Pro - 2024-11-05 at 14 56 11
Simulator Screenshot - iPhone 16 Pro - 2024-11-05 at 14 57 25
Simulator Screenshot - iPhone 16 Pro - 2024-11-05 at 14 57 31
Simulator Screenshot - iPhone 16 Pro - 2024-11-05 at 15 00 39
Simulator Screenshot - iPhone 16 Pro - 2024-11-05 at 15 01 04
Simulator Screenshot - iPhone 16 Pro - 2024-11-05 at 15 07 46

@yanascz yanascz added the mobile Suite Lite issues and PRs label Nov 5, 2024
@yanascz yanascz self-assigned this Nov 5, 2024
@yanascz yanascz requested a review from a team as a code owner November 5, 2024 14:19
Copy link

github-actions bot commented Nov 5, 2024

🚀 Expo preview is ready!

  • Project → trezor-suite-preview
  • Platforms → android, ios
  • Scheme → trezorsuitelite
  • Runtime Version → 9
  • More info

Learn more about 𝝠 Expo Github Action

@yanascz yanascz merged commit b3eb721 into develop Nov 6, 2024
18 checks passed
@yanascz yanascz deleted the fix/native/account-screens-spacing branch November 6, 2024 15:06
@STew790
Copy link
Contributor

STew790 commented Nov 11, 2024

QA OK
LGTM, I don't see anything broken.

Info
24.11.1 c214aa3

@STew790 STew790 added the QA OK Issue passed QA without any blocker label Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mobile Suite Lite issues and PRs QA OK Issue passed QA without any blocker
Projects
Status: ✅ Approved
Development

Successfully merging this pull request may close these issues.

3 participants