Skip to content
This repository was archived by the owner on Feb 2, 2023. It is now read-only.

Conversation

Adlai-Holler
Copy link
Contributor

All hail the pixel grid.

@appleguy
Copy link
Contributor

Yes. However, we actually also need to do something like this for all ASLayouts. Under-specified Stack layouts can crash due to setting NaN and/or Inf on CALayer, and they also routinely set really uneven floating point numbers. Any idea on what layer we could inject this to properly apply that effect without messing up more manual calculations that folks do based on calculatedLayout? I think ASLayout itself will just need to return the rounded values. Wonder if the snapshot tests would pass with this.

cc @rcancro, @Adlai-Holler, @nguyenhuy

This patch is fine to merge and I'm ready to take it, but since it would probably be superseded / replaced by the other change, I'm wondering if we can quickly find the right spot to drop that in.

@appleguy
Copy link
Contributor

Superseded by #945. Thanks a lot for bringing this up @Adlai-Holler !

@appleguy appleguy closed this Dec 17, 2015
yxztj pushed a commit to iftechio/AsyncDisplayKit that referenced this pull request Sep 29, 2018
…rchive#937)

Fixed removing node from supernode after layout transition when automaticallyManagesSubnodes is disabled. In case if developer prefer to manage subnodes by himself then he wants to be sure that stack will not change automatically.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants