kylefang/react-native
1
0
Fork 0
mirror of https://github.com/zhigang1992/react-native.git synced 2026-04-29 04:35:36 +08:00
Code Issues Packages Projects Releases Wiki Activity

[ListView] Defer measurement one frame after componentDidMount to fix error

Browse Source 
Summary:
When `UIManager.measure` is called from `componentDidMount` it causes the error "Attempted to measure layout but offset or dimensions were NaN". Deferring the layout by one frame solves this problem. Layout measurement is already asynchronous anyway, so I believe adding the `requestAnimationFrame` call doesn't affect the program's correctness.

Fixes #1749

Closes https://github.com/facebook/react-native/pull/1750
Github Author: James Ide <ide@jameside.com>

Test Plan:
 Load UIExplorer and no longer get a redbox that says "Attempted to measure layout but offset or dimensions were NaN".
This commit is contained in:
James Ide
2015-06-26 10:25:54 -07:00
parent 1cca4fb769
commit fe7edf0860
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
Libraries/CustomComponents/ListView/ListView.js
View File

@@ -276,7 +276,9 @@ var ListView = React.createClass({
}, },
componentDidUpdate: function() { componentDidUpdate: function() {
this._measureAndUpdateScrollProps(); this.requestAnimationFrame(() => {
this._measureAndUpdateScrollProps();
});
}, },
onRowHighlighted: function(sectionID, rowID) { onRowHighlighted: function(sectionID, rowID) {