Skip to content

Conversation

@dlom
Copy link

@dlom dlom commented Jan 17, 2026

Objective

  • Addressing the top comment in the file:
TODO: This probably doesn't belong in bevy_ui_widgets, but I am not sure where it should go.

Solution

  • I moved the module from bevy_ui_widgets into bevy_ecs (under the observer module)
  • This seems like the most logical landing spot

Testing

  • cargo run -p -- compile
  • cargo run -p -- test

@github-actions
Copy link
Contributor

Welcome, new contributor!

Please make sure you've read our contributing guide and we look forward to reviewing your pull request shortly ✨

@alice-i-cecile alice-i-cecile added A-ECS Entities, components, systems, and events X-Controversial There is active debate or serious implications around merging this PR S-Needs-SME Decision or review from an SME is required labels Jan 17, 2026
@alice-i-cecile
Copy link
Member

This is something that we've discussed, and probably won't do. It's more likely that bsn lands, or we allow observers to be defined using relations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ECS Entities, components, systems, and events S-Needs-SME Decision or review from an SME is required X-Controversial There is active debate or serious implications around merging this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants