search

LibreShift / red-moon

Android screen filter app for night time phone use.
GNU General Public License v3.0
647 stars 81 forks source link

Notification regression #209

Closed SZenglein closed 6 years ago

SZenglein commented 6 years ago

Hey, I'm currently still on version 3.0 since in my opinion the notification in newer versions is a regression. It doesn't show the current profile anymore. It permanently shows the warning to disable red moon to install apps instead. Also, changing the profile while paused unpauses without updating the notification.

Could you please add an option to show the current profile again and disable the warning?

smichel17 commented 6 years ago

Hi, thanks for the report. What version of Android are you using? The current profile should be shown, so that sounds like a bug.

The behavior when changing profiles is also a bug, but a separate one; could you split it into its own issue, please?

I'm in the midst of finals right now so I won't be doing any coding for about two weeks, but I'll be back to look into this after that.

SZenglein commented 6 years ago

7.1.2

smichel17 commented 6 years ago

For clarity, this is how it is currently intended to look:

Notification

("Lower Dim" being one of my profiles)

SZenglein commented 6 years ago

Okay, I now understand what the problem is. I didn't notice the small indicator because it doesn't change when pressing next.

This is probably the same as #210. I'll update 210 and this issue should be closed