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

Black stripe on the buttom of the screen #11280

Open
672493 opened this issue Dec 27, 2023 · 12 comments
Open

Black stripe on the buttom of the screen #11280

672493 opened this issue Dec 27, 2023 · 12 comments

Comments

@672493
Copy link

672493 commented Dec 27, 2023

  • KOReader version:
  • Device:

Issue

Starting from this version
Screenshot_20231226_161749_org koreader launcher

there is a black stripe on the buttom of the screen when show cutout in system settings is turned on. On this version
Screenshot_20231226_161613_org koreader launcher

everything is ok

Steps to reproduce

This bug appears If show cutout in system setting is turned on

Screenshot_20231227_013143_com android settings

crash.log (if applicable)

crash.log is a file that is automatically created when KOReader crashes. It can normally be found in the KOReader directory:

  • /mnt/private/koreader for Cervantes
  • koreader/ directory for Kindle
  • .adds/koreader/ directory for Kobo
  • applications/koreader/ directory for Pocketbook

Android logs are kept in memory. Please go to [Menu] → Help → Bug Report to save these logs to a file.

Please try to include the relevant sections in your issue description.
You can upload the whole crash.log file (zipped if necessary) on GitHub by dragging and dropping it onto this textbox.

If your issue doesn't directly concern a Lua crash, we'll quite likely need you to reproduce the issue with verbose debug logging enabled before providing the logs to us.
To do so, from the file manager, go to [Tools] → More tools → Developer options, and tick both Enable debug logging and Enable verbose debug logging.
You'll need to restart KOReader after toggling these on.

If you instead opt to inline it, please do so behind a spoiler tag:

crash.log
<Paste crash.log content here>
@672493 672493 changed the title Black stripe on the button of the screen Black stripe on the buttom of the screen Dec 27, 2023
@pazos
Copy link
Member

pazos commented Dec 27, 2023

Yay, use auto or stick with the last version that works for you

@672493
Copy link
Author

672493 commented Dec 27, 2023

If I use auto that black line appears on top in the notch area) and the program does not extend to full screen also

@pazos
Copy link
Member

pazos commented Dec 27, 2023

Yes, that's the behaviour we're looking for. Top blank line. Bottom matches screen

@672493
Copy link
Author

672493 commented Dec 27, 2023

Thank you! But it does not look consistent with white screen and black font. Why not to extend the program window fullscreen, as it was a few versions ago?

@pazos
Copy link
Member

pazos commented Dec 27, 2023

You were exploiting an undefined behaviour on the application. We fixed that. The app worked the same since we added support for notches, in #4769

Now, I truly don't know how it behaved on your device, with "auto" set, on previous versions. But that was also undefined if your device has android>=13.

Anyhow, please use the old version if that is what you want. I'm using a v2022 on my main reader device and have no FOMO whatsoever ;)

@672493
Copy link
Author

672493 commented Dec 27, 2023

Thank you, everything is clear), anyway, on my device this patch works fine #10698 (comment)
Thank you

@pazos
Copy link
Member

pazos commented Dec 27, 2023

Thank you, everything is clear), anyway, on my device this patch works fine #10698 (comment) Thank you

Yup, as long as you always read in portrait mode.
It might be possible to get another patch that takes width into account and works also in landscape mode. If somebody figures out a patch like that let us know :)

@hugleo
Copy link
Contributor

hugleo commented Dec 27, 2023

For what I understood, your device doesn't have a hole, right?
For tests on the emulator, there was no black strip when I tested. However, I just tested with the previous options before I deduplicate:

if (cut != null && cut.boundingRects.isNotEmpty())

I didn't test with:

if (cut != null)

Maybe cut.boundingRects.isNotEmpty() makes some difference. I shall test again on the emulator. If that breaks on the emulator, we can fix that. Otherwise, @pazos would be a good idea to have a koreader option that lets the user choose to ignore or not the cutout area?

@672493, what is your device

@pazos
Copy link
Member

pazos commented Dec 27, 2023

@hugleo: please, no.

It is not possible ATM unless you address koreader/android-luajit-launcher#238 properly first

@hugleo
Copy link
Contributor

hugleo commented Dec 27, 2023

@hugleo: please, no.

It is not possible ATM unless you address koreader/android-luajit-launcher#238 properly first

unless you address koreader/android-luajit-launcher#238 properly first

Suffering succotash :)

It is not possible ATM unless you address koreader/android-luajit-launcher#238 properly first

Suffering succotash :)

@672493
Copy link
Author

672493 commented Dec 28, 2023

@672493, what is your device

This is Huawei p60 pro. It has a notch and curved corners, but I try to ignore it)

@hugleo
Copy link
Contributor

hugleo commented Dec 29, 2023

All right, no bugs on our side then. The black strip is intended to cover the notch area so that Koreader contents aren't cut off, such as image edges or text letters.

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

3 participants