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

Update GlobalTransform on insertion #9081

Merged

Conversation

Shatur
Copy link
Contributor

@Shatur Shatur commented Jul 9, 2023

Objective

GlobalTransform after insertion will be updated only on Transform or hierarchy change.

Fixes #9075

Solution

Update GlobalTransform after insertion too.


Changelog

  • GlobalTransform is now updated not only on Transform or hierarchy change, but also on insertion.

@Shatur Shatur force-pushed the update-global-transform-insert branch from 80c2204 to 1a585be Compare July 9, 2023 10:17
@Selene-Amanita Selene-Amanita added C-Bug An unexpected or incorrect behavior A-Transform Translations, rotations and scales labels Jul 9, 2023
@Selene-Amanita Selene-Amanita 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 Jul 9, 2023
@james7132 james7132 added this pull request to the merge queue Jul 10, 2023
Merged via the queue into bevyengine:main with commit c71ae26 Jul 10, 2023
@Shatur Shatur added this to the 0.11.1 milestone Jul 11, 2023
@Shatur Shatur deleted the update-global-transform-insert branch July 11, 2023 17:07
ickshonpe added a commit to ickshonpe/bevy that referenced this pull request Jul 12, 2023
# Objective

`GlobalTransform` after insertion will be updated only on `Transform` or
hierarchy change.

Fixes bevyengine#9075

## Solution

Update `GlobalTransform` after insertion too.

---

## Changelog

- `GlobalTransform` is now updated not only on `Transform` or hierarchy
change, but also on insertion.
cart pushed a commit that referenced this pull request Aug 10, 2023
# Objective

`GlobalTransform` after insertion will be updated only on `Transform` or
hierarchy change.

Fixes #9075

## Solution

Update `GlobalTransform` after insertion too.

---

## Changelog

- `GlobalTransform` is now updated not only on `Transform` or hierarchy
change, but also on insertion.
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 C-Bug An unexpected or incorrect behavior 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
Development

Successfully merging this pull request may close these issues.

Transform propagation after adding GlobalTransform
4 participants