workaround for no-touchscreen device (#2301)

* just return un-consumed * submodule ver * submodule ver * merge from upstram next * revert submodule ver * revert submodule ver
portapack-mayhem · Oct 16, 2024 · 7d28e49 · 7d28e49
1 parent 48129cd
commit 7d28e49
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/firmware/application/ui_navigation.cpp b/firmware/application/ui_navigation.cpp
@@ -996,6 +996,7 @@ void BMPView::paint(Painter&) {
 bool BMPView::on_touch(const TouchEvent event) {
     /* the event thing were resolved by HTotoo, talked here https://discord.com/channels/719669764804444213/956561375155589192/1287756910950486027
      * the touch screen policy can be better, talked here https://discord.com/channels/719669764804444213/956561375155589192/1198926225897443328
+     * this workaround discussed here: https://discord.com/channels/719669764804444213/1170738202924044338/1295630640158478418
      */
 
     if (!nav_.is_valid()) {
@@ -1004,11 +1005,8 @@ bool BMPView::on_touch(const TouchEvent event) {
 
     switch (event.type) {
         case TouchEvent::Type::Start:
-            return true;
-
-        case TouchEvent::Type::End:
             handle_pop();
-            return true;
+            return false;
 
         default:
             break;

diff --git a/hackrf b/hackrf