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: wide screen ui scales better #298

Merged
merged 3 commits into from
Nov 25, 2021
Merged

fix: wide screen ui scales better #298

merged 3 commits into from
Nov 25, 2021

Conversation

dannyhw
Copy link
Member

@dannyhw dannyhw commented Nov 19, 2021

Issue: #163

Previously the menu would always scale to 70% of the screen, now if the screen is 1024 or wider it will scale to be 30% and the preview will be 70%.

What I did

I adjusted the scaling animation so it flips the size of the menu and preview on wide screens for a better experience on the web or ipads etc.

How to test

run the example app on the web

cd examples/native

# to run normally
yarn ios

# to run on an ipad
yarn ios --simulator "iPad Pro (12.9-inch) (5th generation)"

# to run on the web
yarn web

screenshots

image
image
image
image

@dannyhw dannyhw added the 6.5 label Nov 19, 2021
@dannyhw dannyhw self-assigned this Nov 19, 2021
Copy link
Contributor

@jorgeruvalcaba jorgeruvalcaba left a comment

Choose a reason for hiding this comment

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

Minor comments, but LGTM

@dannyhw
Copy link
Member Author

dannyhw commented Nov 22, 2021

@jorgeruvalcaba thanks for the review 🙇

@dannyhw dannyhw merged commit 4a2d3f0 into next-6.0 Nov 25, 2021
dannyhw added a commit to raychanks/react-native that referenced this pull request Feb 7, 2022
* fix: wide screen ui scales better

* fix: allow ipads in example

* refactor: use constants
@dannyhw dannyhw deleted the fix/web-ui-scale branch February 2, 2024 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants