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

Implement incognito mode #791

Merged
merged 9 commits into from
Nov 27, 2019
Merged

Conversation

David-Development
Copy link
Member

Fixes #773

Note the incognito icon on the bottom.

@David-Development
Copy link
Member Author

As I didn't really liked the idea of having an overlay at the bottom, I tried using a bottom navigation bar instead. This also helps me use the app as I can't reach the read/unread and star/unstar button on my OnePlus anymore without moving the phone in my hand. The new design looks like this. Let me know what you guys think! @tobiasKaminsky @jancborchardt @stefan-niedermann

@tobiasKaminsky
Copy link
Member

I am quite unsure about it…

  • the three buttons are maybe less used than the back button, so maybe switch it?
  • what if you want to open the article? Then you still have to go all the way up…
  • it is not common at all to have this at bottom

@stefan-niedermann
Copy link
Member

I also am very unsure... to be honest it just feels odd. The back-arrow is now nearly directly at the same position like the native android-back-triangle. It's a nice idea, and i know, mozilla is trying to do the same thing on firefox for android currently, but i would downvote this.

@David-Development
Copy link
Member Author

Thank you for your feedback guys!

what if you want to open the article? Then you still have to go all the way up…

True, maybe we could add an floating action bar button that lets the user open the article

it is not common at all to have this at bottom

isn't it? I saw some other readers going in the same direction.. and the bottom bar navigation is quite common now, isn't it? --> https://material.io/components/app-bars-bottom/#usage

The back-arrow is now nearly directly at the same position like the native android-back-triangle.

Yes, valid point. It looks kind of odd on the emulator when I tested it..

But then again.. I feel like we need some changes here because screens are getting bigger and bigger and especially for the detail view, I'm using the buttons in the action-bar quite often and it's annoying to move the phone position around...

@jancborchardt
Copy link
Member

Hmm, so reading the original issue, it seems the solution proposed is actually quite good?

Solution: Add a menu item – maybe a slider – which lets you choose whether to download external content at all. Options could be: Always / Only on WLAN / Never.

And that would just be in the settings. No indicator needed at all, nor a bottom bar.


(Regarding the bottom bar, I find it odd as well as said by others above already. :) Also I’m not sure what the square next to the star is … I’m guessing it’s a checkbox for having read it? – My point is if I am unsure then many people will have no clue. ;)

@David-Development
Copy link
Member Author

@jancborchardt Thank you for your feedback! :) I reverted the changes again and added a simple menu bottom in the Detail-View. That should be enough. Thanks again everyone for the feedback! 👍

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.

[Feature Request] Add option to never download external content / images.
4 participants