Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add methods for calculating the size and postion of UI nodes #7930

Merged
merged 14 commits into from
Mar 9, 2023
Merged

Add methods for calculating the size and postion of UI nodes #7930

merged 14 commits into from
Mar 9, 2023

Conversation

ickshonpe
Copy link
Contributor

Objective

Users are often confused about how to calculate the size and position of UI nodes.

Solution

Implement methods on Node to calculate its size and position given its GlobalTransform.

Changelog

  • Added methods logical_rect and physical_rect that calculate the geometry of a UI node and return it as a bevy_math::Rect

@ickshonpe ickshonpe changed the title methods to calculate the size and postion of UI nodes Add methods for calculating the size and postion of UI nodes Mar 6, 2023
@alice-i-cecile alice-i-cecile added A-UI Graphical user interfaces, styles, layouts, and widgets C-Usability A simple quality-of-life change that makes Bevy easier to use labels Mar 6, 2023
@alice-i-cecile
Copy link
Member

This will be super useful for tests too.

@ickshonpe
Copy link
Contributor Author

ickshonpe commented Mar 6, 2023 •
edited
Loading

For some reason it pulled in my changes from the UI example fix PR. Fixed, I think.

@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Mar 8, 2023
james7132
james7132 approved these changes Mar 9, 2023
View reviewed changes
crates/bevy_ui/src/ui_node.rs Show resolved Hide resolved
ickshonpe and others added 2 commits March 9, 2023 10:15
@ickshonpe @mockersf
Update crates/bevy_ui/src/ui_node.rs
b219cb9 
Co-authored-by: François <mockersf@gmail.com>
@ickshonpe @mockersf
Update crates/bevy_ui/src/ui_node.rs
f0b355b 
Co-authored-by: François <mockersf@gmail.com>
@mockersf mockersf added this pull request to the merge queue Mar 9, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 9, 2023
@mockersf mockersf added this pull request to the merge queue Mar 9, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 9, 2023
@mockersf mockersf added this pull request to the merge queue Mar 9, 2023
Merged via the queue into bevyengine:main with commit f554700 Mar 9, 2023
Shfty pushed a commit to shfty-rust/bevy that referenced this pull request Mar 19, 2023
@ickshonpe @mockersf @Shfty
Add methods for calculating the size and postion of UI nodes (bevyeng…
b06d0ea 
…ine#7930)

Co-authored-by: François <mockersf@gmail.com>
Shfty pushed a commit to shfty-rust/bevy that referenced this pull request Mar 19, 2023
@ickshonpe @mockersf @Shfty
Add methods for calculating the size and postion of UI nodes (bevyeng…
677b4eb 
…ine#7930)

Co-authored-by: François <mockersf@gmail.com>
This was referenced Jun 28, 2023
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@james7132 james7132 james7132 approved these changes

@alice-i-cecile alice-i-cecile alice-i-cecile approved these changes

@mockersf mockersf mockersf approved these changes

@Pietrek14 Pietrek14 Pietrek14 approved these changes

Assignees
No one assigned
Labels
A-UI Graphical user interfaces, styles, layouts, and widgets C-Usability A simple quality-of-life change that makes Bevy easier to use S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@ickshonpe @alice-i-cecile @james7132 @mockersf @Pietrek14