Skip to content

ScreenSaver: Fix a bunch of weird corner-cases#12028

Merged
NiLuJe merged 2 commits into
koreader:masterfrom
NiLuJe:master
Jun 14, 2024
Merged

ScreenSaver: Fix a bunch of weird corner-cases#12028
NiLuJe merged 2 commits into
koreader:masterfrom
NiLuJe:master

Conversation

@NiLuJe

@NiLuJe NiLuJe commented Jun 14, 2024

Copy link
Copy Markdown
Member
  • On devices with a broken wait_for ioctl, the workaround could apparently lead to a weird race with the framebuffer state change done on exit, which could lead to strange results when displaying "slow" cover content as the sleep screen (Power off cover image garbled on Kobo Libra H2O #12009).
    Stop touching the framebuffer state on poweroff/reboot to deal with this, as that was perfectly useless in these instances anyway ;).
  • Consistently prefer the "event" i.e., (powered off/reboot...) message over the default message when show_message is enabled. Previously, it only applied when a custom message was unset, which hasn't been technically achievable through the UI for... quite a while now (maybe... ever? :D).

Fix #12009


This change is Reviewable

NiLuJe added 2 commits June 13, 2024 23:36
Apparently, the refresh can be slow enough to race with the state
switch, resulting in it being displayed at the wrong bitdepth.

Fix koreader#12009
message

i.e., we do *NOT* want to see the default "Sleeping" message shown *in
conjunction* with the "reboot/poweroff" overlay messages.

Re: koreader#12009 (comment)
@NiLuJe NiLuJe added the bug label Jun 14, 2024
@NiLuJe NiLuJe added this to the 2024.06 milestone Jun 14, 2024
@NiLuJe

NiLuJe commented Jun 14, 2024

Copy link
Copy Markdown
Member Author

(Rebase me)

@NiLuJe

NiLuJe commented Jun 14, 2024

Copy link
Copy Markdown
Member Author

Wow, does that startup script diff look bad in the GH UI ;).

It's slightly less confusing with a patience diff, but basically, it's trivial: the two top-level orig fb/cpufreq blocks have just been moved inside the rc != halt branch.

(It's much, much clearer on Reviewable ^^).

@NiLuJe NiLuJe left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @NiLuJe)

@NiLuJe NiLuJe merged commit 57c6fb6 into koreader:master Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Power off cover image garbled on Kobo Libra H2O

2 participants