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

Adds proper layout measurement for the ReactTextViewManager. #107

Merged
merged 1 commit into from
Jan 8, 2016

Conversation

rozele
Copy link
Collaborator

@rozele rozele commented Jan 8, 2016

In order to leverage the CSSNode class to its fullest, various view managers need to override the measurement algorithm to determine the proper width and height needed for a view.

For now, we needed to move the measurement of FrameworkElements to the dispatcher thread, as WinRT does not have any text formatter or other hooks for determining the size of text without mimicking the element itself. Issue #106 covers this.

In order to leverage the CSSNode class to its fullest, various view managers need to override the measurement algorithm to determine the proper width and height needed for a view.

For now, we needed to move the measurement of FrameworkElements to the dispatcher thread, as WinRT does not have any text formatter or other hooks for determining the size of text without mimicking the element itself.  Issue #106 covers this.
rozele added a commit that referenced this pull request Jan 8, 2016
Adds proper layout measurement for the ReactTextViewManager.
@rozele rozele merged commit 95fc5c4 into Hackfest Jan 8, 2016
@rozele rozele deleted the layout branch January 8, 2016 18:56
@rozele rozele removed the in progress label Jan 8, 2016
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.

1 participant