Skip to content

Menu Revamp #47

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 14 commits into from
Closed

Menu Revamp #47

wants to merge 14 commits into from

Conversation

Jurorno9
Copy link
Contributor

@Jurorno9 Jurorno9 commented Jan 25, 2025 •
edited
Loading

Brand new menu system

  • menu() takes an array of GUI_Elements() instead of string
  • pagemenu() and listmenu() can draw a multiple-page menu given GUI_Element(s)
  • GUI_Element defines physical arrangement of buttons
  • GUI_Container defines a basic collection of buttons
  • GUI_Cancel and GUI_Confirm macros

To demonstrate this new menu system, /nextmap<no_args> now generates a listmenu

image

@Jurorno9 Jurorno9 marked this pull request as draft January 25, 2025 14:52
@Jurorno9 Jurorno9 added the enhancement New feature or request label Jan 25, 2025
@Jurorno9 Jurorno9 marked this pull request as ready for review January 25, 2025 19:51
@Jurorno9 Jurorno9 changed the base branch from master to menu-revamp January 26, 2025 03:45
@BalaM314 BalaM314 mentioned this pull request Jan 26, 2025
Closed
@BalaM314
Copy link
Member

moved to #48

@BalaM314 BalaM314 closed this Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Jurorno9 @BalaM314