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

OffscreenCanvas for TextMetrics is not cached #8317

Closed
coeing opened this issue May 4, 2022 · 4 comments · Fixed by #8318
Closed

OffscreenCanvas for TextMetrics is not cached #8317

coeing opened this issue May 4, 2022 · 4 comments · Fixed by #8318

Comments

@coeing
Copy link

coeing commented May 4, 2022

I am currently working on making PixiJS work in a NodeJS environment and came across this issue. Because the canvas is not cached in this location leads to the issue that the context (which is fetched with TextMetrics._context in measureFont) is not the context of the canvas that is used and therefore does not resize when setting the dimensions of the canvas in measureFont.

It may only be an issue in my special case (and I will work around it), but it seems wrong nonetheless :)

@bigtimebuddy
Copy link
Member

Oh, you're right on that @coeing, good catch.

@bigtimebuddy
Copy link
Member

Also, we have also been working to make PixiJS run in Node.js as first-class in the project. We should chat and combine forces. It is one of our most requested features.

@coeing
Copy link
Author

coeing commented May 4, 2022

Also, we have also been working to make PixiJS run in Node.js as first-class in the project. We should chat and combine forces. It is one of our most requested features.

@bigtimebuddy Very nice, looking forward to it! And yes, let me know when you have time, I am open to share my experiences, although I am not that experienced with PixiJS yet.

@bigtimebuddy
Copy link
Member

Message me on Pixi's Discord and we can find time to chat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants