Skip to content

Added improvements for Marauder V7 compatibility async WiFi scan, live network counter, and keyboard input improvements#2251

Open
tmalayales-dev wants to merge 1 commit intoBruceDevices:mainfrom
tmalayales-dev:main
Open

Added improvements for Marauder V7 compatibility async WiFi scan, live network counter, and keyboard input improvements#2251
tmalayales-dev wants to merge 1 commit intoBruceDevices:mainfrom
tmalayales-dev:main

Conversation

@tmalayales-dev
Copy link
Copy Markdown

@tmalayales-dev tmalayales-dev commented Mar 6, 2026

This PR adds fixes and improvements tested on Marauder V7 hardware.
Changes include:

Switched WiFi scan from blocking to async so buttons respond instantly during scanning

Added live "Found: N" counter at the bottom of the scan screen that updates in real time

OK button now stops the scan and opens the results list, Esc goes back to previous menu

Physical buttons now work correctly during touch input without being blocked by touch state

Fixed first button press being ignored due to debounce timer starting too late

Esc now cancels immediately without going through debounce delay

Touch state is now cleared after OK/CLOSE tap to prevent phantom inputs on next screen

Fixed non-standard variable length array in keyboard code to compile cleanly on all targets

Removed unused captured variable in handshake code, fixes compiler warning

added a live wifi scanning menu that shows live wifi near the device

added a close button in the keyboard for nontouch device (tested only on mrauder v7)

Tested on:

Marauder V7
ESP32 WROOM32
ILI9341 320x240 display
NRF24 module
PN532
RF Transmitter,Receiver
IR

@bmorcelli bmorcelli marked this pull request as draft March 6, 2026 19:27
@tmalayales-dev tmalayales-dev reopened this Mar 6, 2026
@tmalayales-dev tmalayales-dev marked this pull request as ready for review March 6, 2026 19:56
@bmorcelli
Copy link
Copy Markdown
Member

Some points to take care in your next PR:

  • You have deleted many files that are not meant to be changed/deleted (.github//, media .vscode/*), that are not part of your suggestionand should not be touched
  • You have deleted many things on platformio.ini that are not meant to be deleted (those commented environment names are used elsewhere)
  • You have changed things on tft_espi library that will probably affect other environments, we change display definitions on board/[board]/platformio.ini or pins_arduino.h file, not in the library itself
  • Changes must not affect other devices, and for that many display drawings are made over relative positioning, depending on many things

And last, but not least, PRs must use the "dev" brench as source...

The suggestion of making the Scan asynchronously is, in fact, amazing... Would be really a good thing specially on the Wardriving...

But the way it is now, it can't be merged, I'm sorry..

@tmalayales-dev
Copy link
Copy Markdown
Author

Some points to take care in your next PR:

  • You have deleted many files that are not meant to be changed/deleted (.github//, media .vscode/*), that are not part of your suggestionand should not be touched
  • You have deleted many things on platformio.ini that are not meant to be deleted (those commented environment names are used elsewhere)
  • You have changed things on tft_espi library that will probably affect other environments, we change display definitions on board/[board]/platformio.ini or pins_arduino.h file, not in the library itself
  • Changes must not affect other devices, and for that many display drawings are made over relative positioning, depending on many things

And last, but not least, PRs must use the "dev" brench as source...

The suggestion of making the Scan asynchronously is, in fact, amazing... Would be really a good thing specially on the Wardriving...

But the way it is now, it can't be merged, I'm sorry..

i have deleted main files because i tried to make it fit the launcher but it doesnt work thats why many deleted files so like can you implement the wifi your self to all device i dont have device with touch so i cannot add it to device with touch and i cannot test so can you implement this by yourself because now when ever we scan wifi it shows all device that was in range and if its out of range it still shows in the after scan screen so if you could do it that would be great also i have added a close button in keyboard for non touch device can you add that too because it is very usefull like now non touch device cant exit the keyboard page so the close button will be help full so please consider my recomendation and if you can add this it will be great

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