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: PWA respects orientation lock #4143

Conversation

Choromanski
Copy link
Contributor

What type of PR is this?

  • bug

What this PR does / why we need it:

Previous is you had your phone locked into portrait it would go landscape when you rotated your phone.

Which issue(s) this PR fixes:

Fixes #4136

Special notes for your reviewer:

I have tested this on Android but have no idea how it performs on iOS

Testing

Tested it with and without rotation lock and it no longer switches orientation when it should be locked.

Copy link
Collaborator

@michael-genson michael-genson left a comment

Choose a reason for hiding this comment

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

PWA config settings continue to make no sense to me

@michael-genson michael-genson enabled auto-merge (squash) September 2, 2024 23:54
@Choromanski
Copy link
Contributor Author

They are a real mystery and seem to have no real documentation

@michael-genson michael-genson merged commit 787b826 into mealie-recipes:mealie-next Sep 3, 2024
11 checks passed
boc-the-git pushed a commit to boc-the-git/mealie that referenced this pull request Sep 28, 2024
boc-the-git pushed a commit to boc-the-git/mealie that referenced this pull request Sep 28, 2024
Choromanski added a commit to Choromanski/mealie that referenced this pull request Oct 1, 2024
@Choromanski Choromanski deleted the feature/pwa-screen-orientation branch October 23, 2024 02:32
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.

[BUG] - PWA: screen changes orientation although the phone screen is locked to portrait mode
3 participants