Skip to content

Remove the attribute which fixes the app to portrait mode. #63

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

VarunBarad
Copy link
Contributor

@VarunBarad VarunBarad commented Nov 21, 2019

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

The app works only in portrait mode before this. There was a flag which was set so that the app never switched to landscape mode even when user rotates the phone.

Probably because originally we were not saving webview's state and then restoring it. This might have caused the webview to reload the app from scratch and not preserve where user was in the app.

Since the state saving and restoring was added some time ago. This now works smoothly(ish) even after we remove that flag which sets it to portrait-only.

One problem that happens is that user is initially greeted with the splash-screen on rotating the device until the webpage loads.

Related Tickets & Documents

#62

Screenshots/Recordings (if there are UI changes)

There are UI changes but the Android screen capture doesn't correctly pickup the orientation change so not posting any. Since there are no UI changes inside the app, it has just started functioning in landscape mode too now.

[optional] What gif best describes this PR or how it makes you feel?

Dog happily typing on laptop

@pr-triage pr-triage bot added the PR: draft label Nov 21, 2019
@VarunBarad VarunBarad marked this pull request as ready for review November 21, 2019 10:06
Base automatically changed from master to main January 18, 2021 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant