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

Remove portrait mode option (fix #535) #886

Merged
merged 4 commits into from
Mar 8, 2018
Merged

Remove portrait mode option (fix #535) #886

merged 4 commits into from
Mar 8, 2018

Conversation

Neamar
Copy link
Owner

@Neamar Neamar commented Mar 4, 2018

As discussed in #535, every Android device since Lollipop has an easy way to lock / unlock rotation.

There is no need to provide such a feature at the app level anymore.

@Neamar Neamar requested a review from nmitsou March 4, 2018 12:31
@licaon-kter
Copy link
Contributor

licaon-kter commented Mar 4, 2018

There is no "per app Android switch"...

I would like this kept on, but then again I use Tap'nTurn, but if that becomes a nuissance (eg. gabm/TapAndTurn#38 or gabm/TapAndTurn#8 ) I'll come here asking for the toggle back.

Also, afaik, most launchers force portrait and offer a lanscape mode in settings.

@Neamar
Copy link
Owner Author

Neamar commented Mar 4, 2018

How often do you use landscape?

Most people I know actually never do. The only valid use case is fullscreen video, and Android is smart enough to do that for you.

I'd rather remove cruft legacy from KISS honestly.

@licaon-kter
Copy link
Contributor

licaon-kter commented Mar 4, 2018

Reading non-mobile aware sites, pdfs. It's a usecase.

This is not, imho, cruft.

@Neamar
Copy link
Owner Author

Neamar commented Mar 4, 2018

Makes sense. I won't merge for now, let's wait to get more feedback on the PR!

If more people agree, I guess we can leave the setting in the app.

@CCBMhf
Copy link

CCBMhf commented Mar 4, 2018

I also vote to keep it in.

@nmitsou
Copy link
Collaborator

nmitsou commented Mar 5, 2018

To me, it feels more natural to follow the system setting by default.
We could have a toggle to overpass it and lock rotation (but set to false on first run)

@Neamar
Copy link
Owner Author

Neamar commented Mar 8, 2018

Based on this discussion, I've reverted my previous commit.

In the end, the only thing I'm changing is the default value for new app users.

@Neamar Neamar merged commit 4f48e73 into master Mar 8, 2018
@Neamar Neamar deleted the remove-potrait branch March 8, 2018 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants