Skip to content

Add menu_divider widget.#23788

Merged
alice-i-cecile merged 3 commits intobevyengine:mainfrom
viridia:menu_divider
Apr 14, 2026
Merged

Add menu_divider widget.#23788
alice-i-cecile merged 3 commits intobevyengine:mainfrom
viridia:menu_divider

Conversation

@viridia
Copy link
Copy Markdown
Contributor

@viridia viridia commented Apr 13, 2026

Objective

Menu dividers have been requested by the editor ui design folks.

Solution

A new menu_divider bsn function

Testing

Manual testing

Showcase

menu_divider

@kfc35 kfc35 added C-Feature A new feature, making something new possible A-UI Graphical user interfaces, styles, layouts, and widgets A-Editor Graphical tools to make Bevy games D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Apr 13, 2026
@github-project-automation github-project-automation bot moved this to Needs SME Triage in UI Apr 13, 2026
Copy link
Copy Markdown
Contributor

@kfc35 kfc35 left a comment

Choose a reason for hiding this comment

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

Looks good to me, also ran and tested it

Just FYI that there are some other minor change for using InputFocusVisible included in this PR which also look good to me as well.

@alice-i-cecile alice-i-cecile added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Apr 13, 2026
@viridia
Copy link
Copy Markdown
Contributor Author

viridia commented Apr 13, 2026

Yeah I added those in because the divider doesn't look so good when there's distracting focus highlights. So they are, in a sense, related :)

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Apr 13, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Apr 13, 2026
@viridia
Copy link
Copy Markdown
Contributor Author

viridia commented Apr 13, 2026

@alice-i-cecile I expected a conflict, just pushed a merged version.

@kfc35
Copy link
Copy Markdown
Contributor

kfc35 commented Apr 14, 2026

@viridia looks like you have to resolve conflicts one more time 😅

@viridia
Copy link
Copy Markdown
Contributor Author

viridia commented Apr 14, 2026

Yes I was expecting this.

@viridia
Copy link
Copy Markdown
Contributor Author

viridia commented Apr 14, 2026

Should be good to merge now.

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Apr 14, 2026
Merged via the queue into bevyengine:main with commit 4553e66 Apr 14, 2026
38 checks passed
@github-project-automation github-project-automation bot moved this from Needs SME Triage to Done in UI Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Editor Graphical tools to make Bevy games A-UI Graphical user interfaces, styles, layouts, and widgets C-Feature A new feature, making something new possible D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants