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
Text wrapping behavior broken in some scenarios with yoga-web-wasm #584
Comments
Thanks for reporting this! @amitdahan Would you be able to look into this? |
Hey, I'll definitely find some time to debug this, but I'm (maybe incorrectly?) assuming this is some subtle difference in the Yoga distribution we're using? It seems So I assume it's fair that there would be some changes, I'll try to understand what's expected or a bug, or maybe we can fix something within Ink to account for this. For now, do you think we should revert? |
I don't think it's significant enough to revert, it's probably something minor, but would still love to understand what has changed and why 😉 Appreciate your help with this, whenever you have time! |
Hi,
In our codebase we have some tests that check that text wrapping works as expected in boxes with a certain
paddingX
, and I noticed that after upgrading to 4.1.0 these tests broke. I was able to do a bisect of the ink dependency and found that this commit introduced the issue.Here are the screenshots of before (ink 4.0.0) and after (ink 4.1.0 and 4.2.0)
Before:
After:
This is the code generating these
It seems like 4.0.0 is respecting the
paddingX={2}
property and wrapping the text correctly but 4.1.0+ only has one space at the end. Also, if I removeflexGrow={1}
wrapping doesn't work in in 4.0.0 as well.You can download the example repo here and flip the version from 4.0.0 to 4.2.0 to see the behavior.
The text was updated successfully, but these errors were encountered: