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

[Yoga] Minimize number of nodes that have MeasureFunc set on them. #369

Merged
merged 1 commit into from
Jun 19, 2017

Conversation

appleguy
Copy link
Member

@appleguy appleguy commented Jun 18, 2017

NOTE: Yoga support is still highly experimental and is not
planned to become a supported / documented mode of the framework.
We recommend that most apps use ASStackLayoutSpec and the other
specs: http://texturegroup.org/docs/layout2-layoutspec-types.html

This has one important benefit: fixing the stretching behavior of spacer nodes. In addition, it should help efficiency of Yoga and certainly minimize calls to layoutThatFits:.

In the Yoga layout mode of ASDKgram, note how the timestamp is now correctly at the right-hand side of the screen, as allowed by a flexible spacer node placed in the photo header's horizontal stack:

image

@appleguy appleguy self-assigned this Jun 18, 2017
@TextureGroup TextureGroup deleted a comment Jun 18, 2017
This has one important benefit: fixing the stretching behavior of spacer nodes.

In addition, it should help efficiency of Yoga and certainly minimize calls
to layoutThatFits:.

Next up for Yoga is a mostly-red diff, deleting the non-Contiguous code branches.
Copy link
Contributor

@maicki maicki left a comment

Choose a reason for hiding this comment

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

Looks good to me. Great that we can also reduce having measure func's on nodes.

@TextureGroup TextureGroup deleted a comment Jun 19, 2017
@appleguy appleguy merged commit 486410d into master Jun 19, 2017
@appleguy appleguy deleted the YogaMeasureFunc branch June 19, 2017 01:18
bernieperez pushed a commit to AtomTickets/Texture that referenced this pull request Apr 25, 2018
…extureGroup#369)

This has one important benefit: fixing the stretching behavior of spacer nodes.

In addition, it should help efficiency of Yoga and certainly minimize calls
to layoutThatFits:.

Next up for Yoga is a mostly-red diff, deleting the non-Contiguous code branches.
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.

None yet

2 participants