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

fix: canvas text empty shapes #3042

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

harperKKK
Copy link

fix canvas render error when "\r" occur in text
TypeError: Cannot read properties of undefined (reading 'it')
at CVTextElement.buildNewText (http://127.0.0.1:5500/build/player/lottie.js:13949:55)
at ITextElement.validateText (http://127.0.0.1:5500/build/player/lottie.js:11977:12)
at CVTextElement.renderInnerContent (http://127.0.0.1:5500/build/player/lottie.js:13998:10)
at CVTextElement.renderFrame (http://127.0.0.1:5500/build/player/lottie.js:13262:12)
at CanvasRendererBase.renderFrame (http://127.0.0.1:5500/build/player/lottie.js:14459:28)
at AnimationItem.renderFrame (http://127.0.0.1:5500/build/player/lottie.js:2059:21)
at AnimationItem.gotoFrame (http://127.0.0.1:5500/build/player/lottie.js:2045:10)
at AnimationItem.checkLoaded (http://127.0.0.1:5500/build/player/lottie.js:2016:12)
at AnimationItem.waitForFontsLoaded (http://127.0.0.1:5500/build/player/lottie.js:1997:12)
at AnimationItem.configAnimation (http://127.0.0.1:5500/build/player/lottie.js:1981:12)

@harperKKK
Copy link
Author

data.json

@harperKKK
Copy link
Author

line_break.aep.zip

@harperKKK harperKKK force-pushed the fix-canvas-empty-shape branch from 6c1ac57 to 4b31178 Compare November 9, 2023 10:34
@lxrcan
Copy link

lxrcan commented Feb 10, 2024

not sure what youre asking.

it loads as a black rectangle so i dont knw if that is what its supposed to be or not. is that the render frame error you refer to?

there is a few Kanji characters in there that might be causing issues if its having trouble parsing the file?

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 this pull request may close these issues.

2 participants