Skip to content

[PB700] Simplify rotation suspension fix#14897

Merged
Frenzie merged 3 commits into
koreader:masterfrom
jannickfahlbusch:pb700/rotationFix2
Jan 29, 2026
Merged

[PB700] Simplify rotation suspension fix#14897
Frenzie merged 3 commits into
koreader:masterfrom
jannickfahlbusch:pb700/rotationFix2

Conversation

@jannickfahlbusch

@jannickfahlbusch jannickfahlbusch commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

#14895 is a bit inefficient and does not take the option to lock the orientation in place.

This change reuses the existing functionality and in turn respects the other settings as well. koreader/koreader-base#2261 is a prerequsite to deduplicate the G-sensor translation. I'll update the submodule reference here once that change is merged


This change is Reviewable

koreader#14895 is a bit inefficient and does not take the option to lock the orientation in place.

This change reuses the existing functionality and in turn respects the other settings as well
jannickfahlbusch added a commit to jannickfahlbusch/koreader-base that referenced this pull request Jan 29, 2026
Frenzie pushed a commit to koreader/koreader-base that referenced this pull request Jan 29, 2026
@Frenzie Frenzie added this to the 2026.01 milestone Jan 29, 2026
@Frenzie Frenzie merged commit da34d0e into koreader:master Jan 29, 2026
4 checks passed
@Frenzie

Frenzie commented Jan 29, 2026

Copy link
Copy Markdown
Member

It also looks nicer than before. 👍

@jannickfahlbusch jannickfahlbusch deleted the pb700/rotationFix2 branch January 29, 2026 22:27
benoit-pierre pushed a commit to benoit-pierre/koreader that referenced this pull request Jan 31, 2026
0xstillb pushed a commit to 0xstillb/koreader-thai that referenced this pull request May 9, 2026
koreader#14895 is a bit inefficient and does not take the option to lock the orientation in place.

This change reuses the existing functionality and in turn respects the other settings as well
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants