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

Expose dirtied func in Javascript bindings #842

Closed
wants to merge 3 commits into from
Closed

Expose dirtied func in Javascript bindings #842

wants to merge 3 commits into from

Conversation

rmhartog
Copy link

Yoga currently has the concept of a 'dirtied func', which is called when a Node becomes dirty, which can be useful to trigger a repaint. This PR exposes that functionality in the Javascript bindings.

@rmhartog
Copy link
Author

Hello @davidaurelio I see that you've been actively working on some things in this project, do you think you could have a look at this? CI is failing, but that's unrelated (fixed in #843). Let me know if it's not okay to tag you like this and if there is a better way to get some eyes on these PRs 😄

@davidaurelio
Copy link
Contributor

ok, let’s merge it. Which way did you test this?

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@davidaurelio has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@rmhartog
Copy link
Author

I ported the YGDirtiedTest to JavaScript, but had not included in the PR for some reason, my apologies. It replicates the bahaviour of the YGDirtiedTest.cpp.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@davidaurelio has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@rmhartog
Copy link
Author

Noticed that the m_dirtiedFunc should also be cleaned up when the node is reset, same as the measure function, added those changes to the branch.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@davidaurelio has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

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

Successfully merging this pull request may close these issues.

3 participants