-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
return old behaviour of getLocalBounds for cacheAsBitmap and generateTexture #6477
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #6477 +/- ##
=======================================
Coverage 80.14% 80.14%
=======================================
Files 57 57
Lines 2825 2825
=======================================
Hits 2264 2264
Misses 561 561
Continue to review full report at Codecov.
|
Work in progress, I'm fixing stuff |
d735264
to
c508761
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Can you port this to v4 too? |
#6365 broke cacheAsBitmap because our mixin relies on different behaviour of getLocalBounds()
The thing is, old behaviour of
getLocalBounds()
can be called withgetLocalBounds(undefined, true)
I wanted to make it right - work with temp parent like in renderer, and restore transform after the fact just to be sure.
But right now its easier to just patch it back.
broken: https://pixijs.io/examples/#/demos-basic/cacheAsBitmap.js
WIP: https://pixijs.io/examples/?v=dev-cacheAsBitmap#/demos-basic/cacheAsBitmap.js