snark13/Xteink-X4-crosspoint-reader
1
0
Fork 0
mirror of https://github.com/daveallie/crosspoint-reader.git synced 2026-02-04 14:47:37 +03:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

refactor: Simplify reader menu activity handling by removing suppression logic

Browse Source
This commit is contained in:
Arthur Tazhitdinov 2026-02-02 18:25:29 +03:00
parent 606b20d9d0
commit 78e9c7c733
2 changed files with 1 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/activities/reader/EpubReaderActivity.cpp
View File

@ -136,19 +136,7 @@ void EpubReaderActivity::loop() {
return;
}
// Enter reader menu activity (suppressed after slider confirm/cancel).
if (suppressMenuOpenOnce) {
// If we're seeing the confirm release that closed the slider, consume it and return.
if (mappedInput.wasReleased(MappedInputManager::Button::Confirm)) {
suppressMenuOpenOnce = false;
return;
}
// If confirm is no longer pressed and no release is pending, clear suppression.
if (!mappedInput.isPressed(MappedInputManager::Button::Confirm)) {
suppressMenuOpenOnce = false;
}
}
// Enter reader menu activity.
if (mappedInput.wasReleased(MappedInputManager::Button::Confirm)) {
// Don't start activity transition while rendering
xSemaphoreTake(renderingMutex, portMAX_DELAY);
@ -382,13 +370,11 @@ void EpubReaderActivity::onReaderMenuConfirm(EpubReaderMenuActivity::MenuAction
[this](const int percent) {
// Apply the new position and exit back to the reader.
jumpToPercent(percent);
suppressMenuOpenOnce = true;
exitActivity();
updateRequired = true;
},
[this]() {
// Cancel selection and return to the reader.
suppressMenuOpenOnce = true;
exitActivity();
updateRequired = true;
}));

2
src/activities/reader/EpubReaderActivity.h
View File

@ -23,8 +23,6 @@ class EpubReaderActivity final : public ActivityWithSubactivity {
bool pendingPercentJump = false;
// Normalized 0.0-1.0 progress within the target spine item, computed from book percentage.
float pendingSpineProgress = 0.0f;
// Prevents the reader menu from reopening due to the confirm button used to exit the slider.
bool suppressMenuOpenOnce = false;
bool updateRequired = false;
const std::function<void()> onGoBack;
const std::function<void()> onGoHome;