Skip to content

Migrate ui_texture_atlas_slice to use bevy_ui_widgets::Button (Phase 1 Item 4)#25063

Merged
alice-i-cecile merged 3 commits into
bevyengine:mainfrom
taearls:24112/images-ui-texture-atlas-slice
Jul 20, 2026
Merged

Migrate ui_texture_atlas_slice to use bevy_ui_widgets::Button (Phase 1 Item 4)#25063
alice-i-cecile merged 3 commits into
bevyengine:mainfrom
taearls:24112/images-ui-texture-atlas-slice

Conversation

@taearls

@taearls taearls commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Objective

Solution

  • Migrate the ui/images/ui_texture_atlas_slice.rs example to use bevy_ui_widgets::Button.
  • I used the existing button_system fn to handle hover and pressed events.
  • Added some comments to document the functionality

Testing

  • Did you test these changes? If so, how?
    Yes, I ran the ui_texture_atlas_slice example before and after implementing my changes, verifying the functionality is preserved.

@alice-i-cecile alice-i-cecile added C-Examples An addition or correction to our examples A-UI Graphical user interfaces, styles, layouts, and widgets X-Uncontroversial This work is generally agreed upon D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jul 18, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in UI Jul 18, 2026
@kfc35 kfc35 mentioned this pull request Jul 19, 2026
Open
kfc35
kfc35 approved these changes Jul 19, 2026
View reviewed changes
Comment thread examples/ui/images/ui_texture_atlas_slice.rs Outdated
@kfc35 kfc35 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 Jul 19, 2026
@alice-i-cecile
alice-i-cecile added this pull request to the merge queue Jul 20, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 20, 2026
@taearls

taearls commented Jul 20, 2026

Copy link
Copy Markdown
Contributor Author

@alice-i-cecile it looks like the CI failed from a 503 during the android job. Maybe it just needs to be run again?

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

Reviewers

@alice-i-cecile alice-i-cecile alice-i-cecile approved these changes
+1 more reviewer
@kfc35 kfc35 kfc35 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

A-UI Graphical user interfaces, styles, layouts, and widgets C-Examples An addition or correction to our examples 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 X-Uncontroversial This work is generally agreed upon

Projects

UI
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@taearls @kfc35 @alice-i-cecile