FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Avoid excessive requestLayout calls by hshristov · Pull Request #2270 · NativeScript/NativeScript · GitHub

Avoid excessive requestLayout calls - #2270

Merged
hshristov merged 1 commit into
masterfrom
hhristov/layout-improvements
Jun 8, 2016
Merged

hshristov merged 1 commit into
masterfrom
hhristov/layout-improvements

Conversation

Copy link
Copy Markdown
Contributor

Added View.isLayoutRequired - to check if Layout call is needed after a measure call. Use in list-view.
Measure pass now get all its properties from nativeLayoutParams property instead of using instance dependency properties (e.g. perf improvement).
List-view now layouts cells only if there is need to.
List-view now measure rows with the specified rowHeight.

Added View.isLayoutRequired - to check if Layout call is needed after a measure call. Use in list-view.
Measure pass now get all its properties from nativeLayoutParams property instead of using instance dependency properties (e.g. perf improvement).
List-view now layouts cells only if there is need to.
List-view now measure rows with the specified rowHeight.
ns-bot added the cla: yes label Jun 8, 2016

Copy link
Copy Markdown
Contributor

👍

hshristov merged commit 2260801 into master Jun 8, 2016
hshristov deleted the hhristov/layout-improvements branch June 8, 2016 12:41
hshristov added this to the 2.1 milestone Jul 1, 2016
hshristov self-assigned this Jul 1, 2016

lock Bot commented Aug 28, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock Bot locked and limited conversation to collaborators Aug 28, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL