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.

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

[bevy_ui/layout] Update tests to get GlobalTransform properly #12802

Merged

Conversation

StrikeForceZero
Copy link
Contributor

@StrikeForceZero StrikeForceZero commented Mar 30, 2024

This is 2 of 5 iterative PR's that affect bevy_ui/layout

Diff to parent PR


Objective

  • Update a test in bevy_ui/layout to use the proper way to get an up to date GlobalTransform

Solution

  • Adds sync_simple_transforms, and propagate_transforms to the test schedule in bevy_ui/layout

@NthTensor NthTensor added A-UI Graphical user interfaces, styles, layouts, and widgets C-Testing A change that impacts how we test Bevy or how users test their apps A-Transform Translations, rotations and scales labels Mar 30, 2024
@StrikeForceZero StrikeForceZero force-pushed the dev/bevy_ui/update_layout_tests branch from d03ba9f to be7e6a2 Compare March 30, 2024 18:23
@msvbg
Copy link
Contributor

msvbg commented Apr 2, 2024

@StrikeForceZero I think these diffs will look nicer if you rebase on top of main now, since #12801 was merged.

Remove the need to manually update GlobalTransform
in bevy_ui/layout tests
@StrikeForceZero StrikeForceZero force-pushed the dev/bevy_ui/update_layout_tests branch from e89ac04 to 9946b8f Compare April 2, 2024 22:24
@james7132 james7132 added this pull request to the merge queue Apr 3, 2024
Merged via the queue into bevyengine:main with commit ba8d702 Apr 3, 2024
27 checks passed
chompaa pushed a commit to chompaa/bevy that referenced this pull request Apr 5, 2024
…gine#12802)

This is 2 of 5 iterative PR's that affect bevy_ui/layout

- [x] Blocked by bevyengine#12801

[Diff to parent
PR](https://github.com/StrikeForceZero/bevy/compare/dev/bevy_ui/breakup_layout_mod..dev/bevy_ui/update_layout_tests)

---

# Objective

- Update a test in bevy_ui/layout to use the proper way to get an up to
date `GlobalTransform`

## Solution

- Adds `sync_simple_transforms`, and `propagate_transforms` to the test
schedule in bevy_ui/layout

---
github-merge-queue bot pushed a commit that referenced this pull request Oct 15, 2024
… for UiSurface (#12803)

This is 3 of 5 iterative PR's that affect bevy_ui/layout

- [x] Blocked by #12801
- [x] Blocked by #12802

---

# Objective

- Add tests to `UiSurface`
- Add missing asserts in `_assert_send_sync_ui_surface_impl_safe`
- Add missing Debug field print for `camera_entity_to_taffy`

## Solution

- Adds tests to `UiSurface`
- Adds missing asserts in `_assert_send_sync_ui_surface_impl_safe`
- Adds missing impl Debug field print for `camera_entity_to_taffy`

---------

Co-authored-by: Alice Cecile <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Transform Translations, rotations and scales A-UI Graphical user interfaces, styles, layouts, and widgets C-Testing A change that impacts how we test Bevy or how users test their apps
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants