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

feat(suite-native): changing switch visuals to stand out more #13785

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

vytick
Copy link
Contributor

@vytick vytick commented Aug 13, 2024

After the discussion with @shenkys the visual for switch has changed mostly in dark mode.
In unchecked state we add border radius and the thumb (circle) is white in both light and dark mode

Screenshot 2024-08-13 at 13 32 36

Related Issue

Resolve

Screenshots:

Simulator.Screen.Recording.-.iPhone.15.-.2024-08-13.at.14.00.18.mp4

@vytick vytick added the mobile Suite Lite issues and PRs label Aug 13, 2024
@vytick vytick requested a review from a team as a code owner August 13, 2024 12:08
@vytick vytick force-pushed the feat/native/switch-color-update branch from aec0d5b to 7025d34 Compare August 13, 2024 12:10
suite-native/atoms/src/Switch.tsx Outdated Show resolved Hide resolved
suite-native/atoms/src/Switch.tsx Show resolved Hide resolved
@vytick vytick requested a review from PeKne August 15, 2024 10:35
@vytick
Copy link
Contributor Author

vytick commented Aug 15, 2024

/rebase

Copy link

@trezor-ci trezor-ci force-pushed the feat/native/switch-color-update branch from 063e8ea to 30cde53 Compare August 15, 2024 13:29
@vytick
Copy link
Contributor Author

vytick commented Aug 15, 2024

/rebase

Copy link

@trezor-ci trezor-ci force-pushed the feat/native/switch-color-update branch from 38f0d12 to 82ab763 Compare August 15, 2024 14:00
@vytick vytick merged commit aceb3c7 into develop Aug 15, 2024
14 checks passed
@vytick vytick deleted the feat/native/switch-color-update branch August 15, 2024 18:47
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants