alf45tar / PedalinoMini

Wireless and Bluetooth MIDI Foot Controller
GNU General Public License v3.0
488 stars 87 forks source link

Default "else" catchall config does not set FACTORY_DEFAULT_PIN mode for Long Press #488

Open Pedalnetics opened 9 months ago

Pedalnetics commented 9 months ago

Firmware 3.2.0, Config.h, Line ~990 where it's setting up the default Actions for the onboard button for Profile+, Profile-, and Device Info, it's never setting the PEDALS-1 button to be PED_PRESS_1_2_L, therefor the LONG press to show device info is not working and you can't get to see the IP address of the device.

I fixed it in my local code by adding the following, not sure if it's the right fix or not but it did what I needed: pedals[PEDALS-1].pressMode = PED_PRESS_1_2_L;

alf45tar commented 9 months ago

Not sure if I got it. The long press works for me in my T-Display and T-Display-S3 board. Your suggested fix is exactly the 990 line.

Pedalnetics commented 9 months ago

Sorry, mixed up my lines. Correct, the ttgo stuff is getting caught in the If on line 962 then setting that default pedal mode for pedals-1 on 990 (and/or the if on 866 & mode on 895) For other boards not caught on 866 or 962, they hit the else on 1056, and need that fix around 1084 I believe;

Context: My Heltec is coming up factory default without Long Press enabled on the onboard button even though the default Actions are there to try and show Device Info with a long press. Copying that line from 990 down to 1084 solved it for me.

Pedalnetics commented 9 months ago

If there is a way I can privately fork the sponsors repo and submit pull requests to it, I'd be happy to submit some of these fixes/other features. Along with this I have the heltec v3 board working, some display pedal-tag enhancements related to the rectangle-block drawing, and have some other features I'll be coding like a sequence-reset command. but as it sits, I can't pull-request those back since I'm working off the newer sponsors code base