alf45tar / PedalinoMini

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

LILYGO T-Display-S3 On-Device Button functionality #477

Closed Pedalnetics closed 6 months ago

Pedalnetics commented 7 months ago

I just programmed a LILYGO T-Display-S3 via the sponsors repo (if this is not the right place for issues, please redirect me), and it appears the on-device buttons are not functioning properly/as documented.

The boot button does not do anything after bootup; during bootup,it does properly skip steps like Wifi/Smart/etc; I can also hold it on-power-up to boot the special modes.

Once the device is booted and displaying the bank screen, the boot button no longer does anything. There is another on-device button the docs call IO14; this does move forward profiles with a single click, but none of the other functions work: double-press to go back a profile, or long-hold to change/scroll through the display pages. (The RST button does reset the device, and the boot button does nothing at this point).

alf45tar commented 7 months ago

Yes, it is the right place for issues.

T-Display-S3 has no default configuration for the moment because it is in early stage of development. You have to created yourself the Actions.

Pedalnetics commented 7 months ago

C38 There does seem to be a global action already defined for this (I didn't add these), so maybe it was commited by accident/wasn't fully working? If I change what Press does, it responds as-expected (i.e. if I change Press to Device Info, it does change the display to device info). However neither Double Press or Long Press seem to work Edit - Found it; it appears Control38 is mapped to Pedal 8 in the default config, which does not have double-press and long-press enabled at the Pedal level. Not sure what the proper/intended design/fix is;

jaylfc commented 7 months ago

Could you elaborate a little on how to setup the buttons, or best to just wait for an update? Cheers!

alf45tar commented 7 months ago
Screenshot 2023-12-15 at 21 27 27
jaylfc commented 6 months ago

Thanks, I'll give it a try :)

alf45tar commented 6 months ago

Added same default config of T-Display for T-Display-S3 in 3.2.0