Skip to content

Pancake and v8 bug fixes#364

Merged
bmorcelli merged 5 commits into
bmorcelli:mainfrom
H4W9:Pancake_V8
Jul 7, 2026
Merged

Pancake and v8 bug fixes#364
bmorcelli merged 5 commits into
bmorcelli:mainfrom
H4W9:Pancake_V8

Conversation

@H4W9

@H4W9 H4W9 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Pancake bug fixes

  • fix exit when tapping the first 2 items in the SD browse menu
  • Partition size editor bottom labels are misaligned
  • Fix the short/long press for pancake

Marauder V8 bug fixes

  • fix exit when tapping the first 2 items in the SD browse menu
  • Fix the short/long press for pancake

I believe the Marauder v6.1 and other similar boards require this fix too, but I can't test them, so I left it only for the ones I could verify.
Could simply change (in sd_functions.cpp line #325)
#if defined(PANCAKE) || defined(MARAUDERV8)
to
#if defined(HAS_TOUCH) && defined(DONT_USE_INPUT_TASK) && !defined(E_PAPER_DISPLAY)
to include them.

H4W9 added 3 commits July 7, 2026 09:17
- fix exit when tapping the first 2 items in the SD menu
- Partition size editor bottom labels misaligned
Fix short/long press for pancake
same fix for v8

@bmorcelli bmorcelli left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

couple of changes and it is done

Comment thread src/partitioner.cpp
Comment thread src/sd_functions.cpp Outdated
// Long Press Detection
LongPressDetected = false;
#ifndef E_PAPER_DISPLAY
#if defined(PANCAKE) || defined(MARAUDERV8)

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This issue will happen to all devices sharing Touch with TFT, that are under DONT_USE_INPUT_TASK flag.. so you can generalize it usinf this flag.

when this flag is used, it runs the InputHandler function every 5 runs of check(...), it doesn't happen when using inputHandlingTask.. which runs almost every 10ms...

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

So I just replace this line with
#if defined(HAS_TOUCH) && defined(DONT_USE_INPUT_TASK) && !defined(E_PAPER_DISPLAY)

H4W9 added 2 commits July 7, 2026 12:47
Apply fix to all where needed
Spread the three hints into left/centre/right columns so they line up cleanly across the width (matches the touch footer layout).
@bmorcelli
bmorcelli merged commit 229f9e6 into bmorcelli:main Jul 7, 2026
70 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants