+
Skip to content

Conversation

Y0SH1M4S73R
Copy link
Contributor

@Y0SH1M4S73R Y0SH1M4S73R commented Sep 11, 2025

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.

  • Individual items within a menu dropdown can be disabled
  • Submenus, which display a dropdown-within-a-dropdown when hovered
  • The 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.

@AyIong
Copy link
Collaborator

AyIong commented Sep 11, 2025

Can you add demo video please?

@Y0SH1M4S73R
Copy link
Contributor Author

Can you add demo video please?

Done

Copy link
Collaborator

@AyIong AyIong left a comment

Choose a reason for hiding this comment

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

As you there, can you add placement="bottom-start" here?

image

Currently it looks odd

@Y0SH1M4S73R
Copy link
Contributor Author

As you there, can you add placement="bottom-start" here?
image

Currently it looks odd

Handled

Copy link
Collaborator

@ZeWaka ZeWaka left a comment

Choose a reason for hiding this comment

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

otherwise looks good

Copy link
Collaborator

@ZeWaka ZeWaka left a 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

@AyIong
Copy link
Collaborator

AyIong commented Sep 13, 2025

Still a breaking change here, unsure if intentional

Yes... perhaps it is not necessary to rename props unless absolutely necessary

@Y0SH1M4S73R Y0SH1M4S73R requested a review from ZeWaka September 15, 2025 04:36
@jlsnow301 jlsnow301 merged commit 8c19326 into tgstation:main Oct 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载