Skip to content
This repository was archived by the owner on Dec 24, 2022. It is now read-only.

Remove "Press back once more to exit" prompt#169

Open
ericwa wants to merge 1 commit into
Etuldan:masterfrom
ericwa:remove-exit-prompt
Open

Remove "Press back once more to exit" prompt#169
ericwa wants to merge 1 commit into
Etuldan:masterfrom
ericwa:remove-exit-prompt

Conversation

@ericwa

@ericwa ericwa commented Mar 14, 2016

Copy link
Copy Markdown

I find these back button prompts cumbersome on Android, this PR just removes the feature. I didn't see a specific mention of them in Google's UI design guide, but I'm pretty sure they would be discouraged.
Not sure if you'll want this.. :)

@ildar

ildar commented Mar 14, 2016 via email

Copy link
Copy Markdown

@ericwa

ericwa commented Mar 14, 2016

Copy link
Copy Markdown
Author

I can understand that, if you're used to the feature and mostly use apps that have it, then removing "tap again to quit" would be annoying and lead to exiting by accident.

On the other hand, for users who primarily use apps without "tap again to quit", it's a negative feature because tapping back to exit an activity becomes muscle memory, so when one app behaves differently it slows you down.

Maybe I could add a setting for this instead?

@eighthave

Copy link
Copy Markdown

I think spaRSS is a great app, but the "back again to Quit" thing always just aggravates me. The Android way to handle app quitting is to save state, and transparently resume. If there must be an quit/exit, then it should be a menu item.

I fully aware that there are use cases that cannot be addressed without a quit option, but I don't see any such use case in spaRSS.

@avengerx

Copy link
Copy Markdown

Instead of stripping out the option completely, why don't keep the default (ask to press twice to quit) but add a setting option to disable it? Then everyone can be happy.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants