cyb3rko / flashdim

Modern flashlight app with dim functionality on Android 13+
Apache License 2.0
459 stars 18 forks source link

On tile click doesn't update the UI #12

Closed max-ishere closed 1 year ago

max-ishere commented 1 year ago

Steps to reproduce

  1. Open the app.
  2. Set the light level to any value (e.g. 22).
  3. Open notifications while the app is open and click the tile to turn the flash off. Close the notifications.

Actual result

The level in app is the same value (e.g. 22) and the bar is filled, but the light is physically off and the tile is in the off state.

Expected result

The app shows the current flash state (e.g. off).

Notes

If you click the tile again and it sets the level to something different (e.g. default is 10) the app will show 22 (from example above), but the actual level is 10.

Importance

Honestly not that important, since you can press home (swipe up on buttonless devices) and open the app again (not a full restart) and it will get the current state correctly.

Notes

The tile updates correctly if the flashlight level or state (on/off) is changed in the app.

cyb3rko commented 1 year ago

That's how you expect the app to work, unfortunately there's currently no way to check which light level the flashlight is currently on.

This is why atm this is not possible.

max-ishere commented 1 year ago

I just found this by accident so decided to report it just in case. Don't really expect any action on this.

cyb3rko commented 1 year ago

Yea, thanks for this report. I'll keep an eye on updates of the flashlight api, maybe Google will add it one day 🙃

Then I would tackle this problem.