Skip to content

Conversation

@abiduzz420
Copy link
Contributor

Prerequisites

Changes added

  • ReactPortal.js shifted to shared directory
  • Implemented createPortal() method in ReactNoop.js
  • This createPortal() is used to write test for support portals in ReactIncrementalPerf-test.internal.js

Please let me know if I need to make any further modifications or changes.

@abiduzz420
Copy link
Contributor Author

abiduzz420 commented Nov 30, 2017

This PR solves the issue from #11299 in ReactIncrementalPerf-test.internal.js

@gaearon gaearon merged commit f57d963 into facebook:master Nov 30, 2017
@gaearon
Copy link
Collaborator

gaearon commented Nov 30, 2017

Thanks!

@abiduzz420
Copy link
Contributor Author

Thanks @gaearon Dan for helping me out :)

@gaearon gaearon mentioned this pull request Nov 30, 2017
26 tasks
@SomeshThakur
Copy link

Awesome Work Uzair ! :)

@leimonio
Copy link

leimonio commented Dec 2, 2017

Great job @abiduzz420

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.

4 participants