romainpiel / meatspace-android

Chat for Android with gif generation and more...
https://chat.meatspac.es/
BSD 3-Clause "New" or "Revised" License
61 stars 5 forks source link

Swiping the activity out of the apps list should stop the service #18

Closed alexchally closed 10 years ago

alexchally commented 10 years ago

As a user, having to close the app in two places (and quickly) is confusing and seemingly redundant.

Thanks!

romainpiel commented 10 years ago

I'm not sure about that one. It's a bad design to kill an app from the notification bar when it's in foreground. Another solution would be to hide the notification when the app is in foreground.

romainpiel commented 10 years ago

I'm going to make sure the service is not running after the cross is pressed. Swiping the app away doesn't kill the service properly because the service is sticky (same as any music app or google hangout). What I can also do is adding a settings to only enable the service when the app is in foreground. That would solve your issue.

alexchally commented 10 years ago

That would be awesome! Thanks! On Dec 6, 2013 9:19 AM, "Romain Piel" notifications@github.com wrote:

I'm going to make sure the service is not running after the cross is pressed. Swiping the app away doesn't kill the service properly because the service is sticky (same as any music app or google hangout). What I can also do is adding a settings to only enable the service when the app is in foreground. That would solve your issue.

— Reply to this email directly or view it on GitHubhttps://github.com/RomainPiel/meatspace-android/issues/18#issuecomment-30011817 .

romainpiel commented 10 years ago

setting is ready for next release