TcMenu / tcMenuLib

TcMenu modular IoT menu library supporting rotary encoder, joysticks, many displays, and remote control over Ethernet and Serial
Apache License 2.0
111 stars 10 forks source link

[Bug] After menuMgr.activateMenuItem, "down" twice needed to move the cursor down #192

Closed davetcc closed 1 year ago

davetcc commented 1 year ago

Describe the bug When the activeMenuItem method is called on menu manager it is not also updating the encoder with the right value, this results in a problem where the encoder does not match up with the currently set value. At the same time we should ensure that the keyboard support for this works properly, as it is probably wrong there too. Touch screen should be unaffected.

Confirm that the bug is in the embedded library This is an issue in tcMenu.cpp and possibly the keyboard support too.

Commerical or personal/community use Commercial - davetcc.

To Reproduce Simply call activateMenuItem on any item in the menu, it will leave the wrong value on the encoder.

Expected behavior The active item and the encoder should match.

Device Specs Not applicable, affects all boards.

davetcc commented 1 year ago

fixed