Skip to content

Implement short hop aerial macro in CSS menu - #2711

Open
EIREXE wants to merge 1 commit into
HDR-Development:pre-releasefrom
EIREXE:short_hop_aerial_macro
Open

EIREXE wants to merge 1 commit into
HDR-Development:pre-releasefrom
EIREXE:short_hop_aerial_macro

Conversation

@EIREXE

@EIREXE EIREXE commented Aug 10, 2026

Copy link
Copy Markdown

Reimplements the previously commented out short hop aerial macro from vanilla ultimate and makes it optionally toggleable from within the small menu in the CSS.

Some work had to be done in lua.rs because otherwise the bits indicating the macro was enabled would get cleared on when saving from the vanilla options menu.

@BrianAllred BrianAllred added enhancement New feature or request semver:patch labels Aug 10, 2026
@github-actions

Copy link
Copy Markdown

Download the artifacts for this pull request:

@SuddyN SuddyN left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code looks good. However, if memory serves, we previously had an issue with the size of the buttons struct we were able to use online, due to networking constraints. The specific issue is unfortunately undocumented institutional knowledge, but I'll do some research and ask around. In the meantime, please test this online on official hardware, and document any crashes or desyncs.

@EIREXE

EIREXE commented Aug 11, 2026

Copy link
Copy Markdown
Author

This code looks good. However, if memory serves, we previously had an issue with the size of the buttons struct we were able to use online, due to networking constraints. The specific issue is unfortunately undocumented institutional knowledge, but I'll do some research and ask around. In the meantime, please test this online on official hardware, and document any crashes or desyncs.

I do not have my switch on hand (I'm on vacation) if anyone can try it please be my guest.

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

Labels

enhancement New feature or request semver:patch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants