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

Inkbook Prime Frontlight control issues (it's hard to disable it) #7275

Closed
mwoz123 opened this issue Feb 11, 2021 · 3 comments
Closed

Inkbook Prime Frontlight control issues (it's hard to disable it) #7275

mwoz123 opened this issue Feb 11, 2021 · 3 comments

Comments

@mwoz123
Copy link
Contributor

mwoz123 commented Feb 11, 2021

  • KOReader version: v2021.01-3-gf48820_21-01-26 (upgraded from most probably _202-05-19.apk; I also had _2020-11-08.apk but as far as I remember I didn't use it)
  • Device: Inkbook Prime (Android 4.4.2)

Issue

After upgrade to nightly v2021.01-3-gf48820_21-01-26 in my Inkbook Prime earned an additional lower frontlight level, which is great - thanks for that:)

But unfortunately, there're some issues in controlling disabling the frontlight - is there any way to make it better?
See video:
https://youtu.be/dx40ZwZCcyo

If not then I'm fine with it - as I wrote I now have the lower frontlight level - which I have had always wanted to have;)

Steps to reproduce:

on Inkbook Prime try to disable frontlight

@pazos
Copy link
Member

pazos commented Feb 11, 2021

Tapping bottom left toggles frontlight. Sliding left changes brightness.

If you're talking about the behaviour of the slider. That's an android dialog. Is not meant to tap it directly, but pressing the slider and then slide :)

Also see koreader/android-luajit-launcher#290 (was a koreader issue that I moved to the other repo because it needs to be implemented there :))

@mwoz123
Copy link
Contributor Author

mwoz123 commented Feb 13, 2021

If you're talking about the behaviour of the slider. That's an android dialog. Is not meant to tap it directly, but pressing the slider and then slide :)

Thanks. Knowing that works much better :D
I can now disable it as set it to low level

Tapping bottom left toggles frontlight. Sliding left changes brightness.

Toggle sets to min level now (instead disabling), sliding left also can't disable it, but it's very minor issue now as I'm happy with Android dialog ;)

@pazos Thanks:)

@pazos
Copy link
Member

pazos commented Feb 13, 2021

Toggle sets to min level now (instead disabling), sliding left also can't disable it, but it's very minor issue now as I'm happy with Android dialog ;)

That depends on the android implementation :p

We're just requesting the minimum possible level of brightness, which should be 0 on e-ink devices. If that doesn't happen is because most e-ink device implementations are broken and don't expose sane APIs.

It still possible to opt-out of the generic light controller (which works on 100% on android lcd devices) and use a custom one. See https://github.com/koreader/android-luajit-launcher/tree/master/app/src/main/java/org/koreader/launcher/device/lights

I guess you can write your own controller based on GenericController but using System.getInt and System.putInt instead of windows attributes.

That should work for 99% of legacy e-ink devices.

@pazos pazos closed this as completed Apr 18, 2021
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