Skip to content

Reimplement React.Children #9339

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.

Sign up for GitHub

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

Closed
koba04 wants to merge 21 commits into from
Closed

Reimplement React.Children #9339

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
da48ae0
Remove to use PooledClass for traverseAllChildren
koba04 Apr 5, 2017
efb7107
move traverseAllChildren into ReactChildren
koba04 Apr 12, 2017
d5b4b57
Use mapChildren to forEach and toArray
koba04 Apr 14, 2017
437120d
Remove React.Children.mapIntoWithKeyPrefixInternal
koba04 Apr 17, 2017
a2bcddc
Run prettier
koba04 Apr 17, 2017
e6060f6
Remove unnecessary arguments
koba04 Apr 17, 2017
ed041b4
Remove an unused argument from ReactChildren.count
koba04 Apr 18, 2017
1ead882
Test for a single child and an array with one item
koba04 Apr 20, 2017
ab49e3a
Add component stack to invalid-object-child error (cherry-pick #9415)
koba04 Apr 21, 2017
2e982b9
Add test for single child witout key
koba04 Apr 21, 2017
892dd52
Improve component type check in getComponentKey (cherry-pick #9464)
koba04 Apr 26, 2017
1e5ab7c
Add Flow types
koba04 May 8, 2017
a6cdc05
Type for getReactElementKey
koba04 May 8, 2017
8a30669
Remove comments for types
koba04 May 8, 2017
8879aef
Run prettier
koba04 May 8, 2017
d57fab8
Do not allocate results with forEach
koba04 May 9, 2017
4ceb58f
Remove unnecessary traverseContext
koba04 May 9, 2017
87aefbf
Record fiber tests
koba04 May 9, 2017
d920c9b
Rebase #9903
koba04 Jun 12, 2017
96b675c
Fix flow errors with some $FlowFixMe
koba04 Jun 15, 2017
61981da
Fix $FlowFixMe
koba04 Jun 29, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix $FlowFixMe
  • Loading branch information
@koba04
koba04 committed Jun 29, 2017
commit 61981da8e40238d821a5b06552519f0e9719dd31
6 changes: 2 additions & 4 deletions src/isomorphic/children/ReactChildren.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,16 +68,15 @@ function getReactElementKey(element, index) {
}

function traverseAllChildren<T: any, I>(
children: T,
children: T | null,
nameSoFar: string,
callback: (context: I, children: T, nameSoFar: string) => void,
callback: (context: I, children: T | null, nameSoFar: string) => void,
traverseContext: I,
) {
var type = typeof children;

if (type === 'undefined' || type === 'boolean') {
// All of the above are perceived as null.
// $FlowFixMe
children = null;
}

Expand Down Expand Up @@ -144,7 +143,6 @@ function traverseAllChildren<T: any, I>(
child = step != null ? (step.value: ReactElement) : null;
nextName = nextNamePrefix + getReactElementKey(child, ii++);
subtreeCount += traverseAllChildren(
// $FlowFixMe
child,
nextName,
callback,
Expand Down