Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TapKey not simulating all keycodes properly in MenuMode #231

Closed
DisasterGoneAwesome opened this issue Jul 6, 2024 · 4 comments
Closed

Comments

@DisasterGoneAwesome
Copy link

TapKey does not work for some keycodes while in MenuMode.

Better Menu Controls stopped working correctly after obse22.2. After the hotfix (22.3) OnKeyDown seems to be working again, but the mod still doesn't work properly, although some functionality works fine.

After some investigation, TabKey seems to be the culprit. While in MenuMode simulating arrow-key and enter presses does not work. On the other hand, simulating Tab or F1 works fine.

This is how I tested it DGA_TapKeyTest.zip.
Tested on a new savegame with no other mods installed. I also remapped the movement controls to the arrow keys (from wasd) and simulated keypresses work fine in GameMode.

@slowpard
Copy link

slowpard commented Jul 6, 2024

Can confirm this, tried to simulate arrow presses to autonavigate a custom menu, doesn't work.

@llde
Copy link
Owner

llde commented Aug 14, 2024

Hi @DisasterGoneAwesome
the provided testcase is in fact wrong. It use TapKey to ask for keypresses for the UI, but the UI manage this stuff differently requiring MenuTapKey. TapKey in menu works only for some keys whose handling is managed by the Input poll directly.
This is not a bug and certainly not a regression. I remade the test case using OnKeyDown for the i -> Upper Key and tested on OBSE 21.4 and the behavior is the same.
Now there is in fact a bug where MenuTapKey seems to not working.

@llde llde added this to the 22-final milestone Aug 14, 2024
@llde llde closed this as completed in 55a5635 Aug 14, 2024
@llde
Copy link
Owner

llde commented Aug 14, 2024

@DisasterGoneAwesome should be fixed in commit 55a5635

You can test the file from this action: https://github.com/llde/xOBSE/actions/runs/10390492938

@slowpard
Copy link

Confirm that MenuTapKey works as intended now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants