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

Reorganize Settings Menu #704

Closed
ahmedre opened this issue Nov 15, 2016 · 6 comments
Closed

Reorganize Settings Menu #704

ahmedre opened this issue Nov 15, 2016 · 6 comments

Comments

@ahmedre
Copy link
Contributor

ahmedre commented Nov 15, 2016

No description provided.

@ahmedre ahmedre changed the title Split out Advanced Settings into a separate screen Reorganize Settings Menu Nov 23, 2016
@ahmedre
Copy link
Contributor Author

ahmedre commented Nov 23, 2016

The settings menu is getting a bit unwieldy.

Thoughts:

  1. maybe move "Advanced Options" into a separate screen
  2. maybe move "Display Options" into a separate screen
  3. move "Volume Key Navigation" under Reading preferences

"maybe" because I could be convinced either way - open to feedback.

@ahmedre
Copy link
Contributor Author

ahmedre commented Nov 23, 2016

anyone should feel free to work on this and send a PR

@ozbek
Copy link
Contributor

ozbek commented Nov 23, 2016

move "Volume Key Navigation" under Reading preferences

+1... OK, no "maybe" there, but still I voted :)

@ahmedre
Copy link
Contributor Author

ahmedre commented Nov 23, 2016

lol, everything should be considered a maybe :)

ahmedre added a commit that referenced this issue Nov 30, 2016
Split setting screen into options and advanced options issue  #704
@ozbek
Copy link
Contributor

ozbek commented Dec 11, 2016

maybe move "Display Options" into a separate screen

Regarding "Display Options", this seems to overlap with "Reading Preferences". For example, "New background" or "Lock screen orientation" could have been in "Reading Preferences", or "Show page info" or "Highlight bookmarks" could be put to "Display Options".

Only "Arabic mode" does not seem to fit to "Reading Preferences" as that's purely about the UI. Is it not time to remove that option altogether? Android supports Arabic natively now.

@ahmedre
Copy link
Contributor Author

ahmedre commented Dec 11, 2016

i am good with your PR - but just wanted to comment on the Arabic setting - i personally use this (despite my phone being in English as the default locale) - i think it probably gets a lot of use from:

  1. people who speak Arabic fluently but usually converse in English - so they leave their phones in English and just enable Arabic for Islamic apps when the option is there
  2. people who are learning Arabic

so i personally would be sad if this went away :(

ahmedre added a commit that referenced this issue Oct 14, 2017
Split setting screen into options and advanced options issue  #704
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants