feat(lua): deliver EVT_TOUCH_RELEASE event

[wimalopaan]

first try to solve #5585

Resolves #5585

[wimalopaan]

Looks like the LUA constant EVT_TOUCH_RELEASE isn't generated (from keys.h???).

[philmoz]

radio/src/lua/api_general.cpp - section starting at line 3194

[wimalopaan]

LUA constant EVT_TOUCH_RELEASE added

[wimalopaan]

@pfeerick I think this requires also some minor changes in libGUI ?

[pfeerick]

Possibly. That will be a seperate PR though - source is here

• https://github.com/EdgeTX/edgetx-sdcard/tree/master/sdcard/c480x272/WIDGETS/LibGUI
• https://github.com/EdgeTX/edgetx-sdcard/tree/master/sdcard/c480x320/WIDGETS/LibGUI

@philmoz Happy with the state of this PR?

[wimalopaan]

Possibly. That will be a seperate PR though - source is here

• https://github.com/EdgeTX/edgetx-sdcard/tree/master/sdcard/c480x272/WIDGETS/LibGUI
• https://github.com/EdgeTX/edgetx-sdcard/tree/master/sdcard/c480x320/WIDGETS/LibGUI

Ok, I'll make a PR to implement a momentaryButton.

[philmoz]

EVENT_TOUCH_RELEASE and _MSK_KEY_RELEASE are not needed.

The existing EVENT_TOUCH_BREAK and _MSK_KEY_BREAK values should be covering the touch release state; but the code is not working.

I suggest removing EVENT_TOUCH_RELEASE and using EVENT_TOUCH_BREAK in the event_cb() function and also delete these non-functional lines from the LuaEventHandler::onClicked() function.

    } else {
      es->event = EVT_TOUCH_BREAK;

[wimalopaan]

EVENT_TOUCH_RELEASE and _MSK_KEY_RELEASE are not needed.

The existing EVENT_TOUCH_BREAK and _MSK_KEY_BREAK values should be covering the touch release state; but the code is not working.

I suggest removing EVENT_TOUCH_RELEASE and using EVENT_TOUCH_BREAK in the event_cb() function and also delete these non-functional lines from the LuaEventHandler::onClicked() function.

    } else {
      es->event = EVT_TOUCH_BREAK;

Thanks, I'll have a look at that.

[pfeerick]

LGTM on TX16S - EventDemo and LibGUI Demos both appear to be be fully functional still. Thanks for the complimentary LibGUI PR, that looks good also.