fix: dispatch short press via MenuOpenHandler for any control binding

[codepuncher]

Short press was broken for anyone using default controls (because I forgot the defaults) — Start is bound to "System"/"Pause" by default, not "Journal" or "Tween Menu" as the old
whitelist assumed.

Reads the actual user event from ControlMap at runtime and passes a synthetic
ButtonEvent to MenuOpenHandler::ProcessButton on short press, so any binding
works correctly.

Also fixes a pre-existing bug where _mapTriggered wasn't set before the
UIMessageQueue null check, causing a spurious short press dispatch if the
queue was unavailable at hold threshold.

[Copilot]

Pull request overview

This PR updates QuickMap’s Start short-press handling to use the current ControlMap user event instead of a hard-coded Journal/Tween menu whitelist, while keeping hold-to-map behavior.

Changes:

• Caches the Start button’s user event and dispatches short presses through MenuOpenHandler.
• Marks map holds as triggered before checking UIMessageQueue.
• Updates user-facing documentation to describe custom Start bindings.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
src/InputHandler.h	Replaces cached menu name state with cached user event state.
src/InputHandler.cpp	Updates short-press dispatch and hold-threshold handling.
README.md	Updates the short-press behavior description.
docs/nexus-page.md	Updates Nexus overview wording for custom Start bindings.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Comments suppressed due to low confidence (1)

src/InputHandler.cpp:154

• Because the original Start event has already been consumed, any Start binding that MenuOpenHandler does not handle is lost here. That means rebinding Start to a non-menu action will produce only this warning instead of executing the user's binding, so the implementation still does not preserve “any control binding.”

	if (!menuControls->menuOpenHandler->CanProcess(syntheticEvent)) {
		logger::warn("MenuOpenHandler rejected short press — press consumed but no menu opened");
		syntheticEvent->~ButtonEvent();
		RE::free(syntheticEvent);
		return;

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.