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

[Android] add step-by-step adjustment on frontlight dialog and a way to switch back to system defaults #290

Open
dcruzbosco opened this issue Oct 22, 2020 · 4 comments

Comments

@dcruzbosco
Copy link

  • KOReader version: 2020.10.1
  • Device: MiPad 1

Issue

Since version 2020.09, accessing the Frontlight dialog opens native Android brightness popup.
I am interested in knowing how I can go back to the old in-app Frontlight dialog (widget) that allows step-by-step brightness adjustment.
Is there a setting or property to be added in settings.lua for example, that would allow me to switch back to the old brightness adjustment popup?

@dcruzbosco dcruzbosco changed the title Accessing In-App Frontlight Dialog (not Android native brightness popup) [Android] Accessing In-App Frontlight Dialog (not Android native brightness popup) [Android] Oct 22, 2020
@dcruzbosco dcruzbosco changed the title [Android] Accessing In-App Frontlight Dialog (not Android native brightness popup) [Android] [Android] Accessing In-App Frontlight Dialog (not Android native brightness popup) Oct 22, 2020
@pazos
Copy link
Member

pazos commented Oct 22, 2020

Nope, you can't. The good old frontlight widget is not available for a very good reason: it was a PITA to work with and didn't support per device overrides.

step-by-step brightness adjustment.

This is doable with the android dialog too. But needs to be done :)

edit: I mean add a couple of +/- buttons to https://github.com/koreader/android-luajit-launcher/blob/master/app/src/org/koreader/launcher/MainActivity.kt#L261-L317

@dcruzbosco
Copy link
Author

Thanks! This helps a lot!

@pazos
Copy link
Member

pazos commented Oct 22, 2020

@dcruzbosco: I'm going to update the ticket. Patches welcome!

@pazos pazos changed the title [Android] Accessing In-App Frontlight Dialog (not Android native brightness popup) [Android] add step-by-step adjustment on frontlight dialog Oct 22, 2020
@pazos pazos transferred this issue from koreader/koreader Jan 18, 2021
@pazos pazos changed the title [Android] add step-by-step adjustment on frontlight dialog [Android] add step-by-step adjustment on frontlight dialog and a way to switch back to system defaults May 22, 2024
@pazos
Copy link
Member

pazos commented May 22, 2024

Including, from koreader/koreader#11837

The Moon Reader app solves this issue by having a toggle checkbox next to the slider that enables/disables forced brightness override. Might be a good solution beyond the workaround of just not touching the in-app brightness slider.

We can do the same for devices using the generic driver. Actually in that driver 0 means system defaults.

Expected behaviour:

Checking the box should disable the widget slider.
Unchecking the box should enable the widget slider (and adjust to current brightness)
Gestures should uncheck the box if checked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants