-
Notifications
You must be signed in to change notification settings - Fork 25
New MenuBar Features: Submenus, disable individual items, any ReactNode as display value #232
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
Conversation
Can you add demo video please? |
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still a breaking change here, unsure if intentional
Yes... perhaps it is not necessary to rename props unless absolutely necessary |
About the PR
This PR adds several features to the MenuBar - two to add functionality that native window menus usually have, and one that feels appropriate given the way the property is used.
displayText
property now accepts any ReactNode. It has not been renamed to reflect this because it would be a breaking change.Video demonstration
2025-09-11.18-18-39.mp4
Why's this needed?
There's a ui I wanted to code for tg and the main additions from this PR were something I felt were needed to get the look and feel I wanted. Other people who want to make changes to stuff like modular computer applications would probably also appreciate these changes.