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

[davetcc]

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]

fixed