Skip to content

Use const/let in more places#11467

Merged
gaearon merged 1 commit intofacebook:masterfrom
raphamorim:master
Nov 6, 2017
Merged

Use const/let in more places#11467
gaearon merged 1 commit intofacebook:masterfrom
raphamorim:master

Conversation

@raphamorim
Copy link
Copy Markdown
Contributor

@raphamorim raphamorim commented Nov 6, 2017

  • Convert ReactDOM to const/let
  • Convert ReactDOMComponentTree to const/let
  • Convert ReactDOMComponentTree to const/let
  • Convert getNodeForCharacterOffset to const/let
  • Convert getTextContentAccessor to const/let
  • Convert inputValueTracking to const/let
  • Convert setInnerHTML to const/let
  • Convert setTextContent to const/let
  • Convert validateDOMNesting to const/let

// the target node

// SVG temp container for IE lacking innerHTML
let reusableSVGContainer;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is intentionally outside of the function. We don't want to allocate one per call.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sense. Updated ad3223a

* Convert ReactDOM to const/let
* Convert ReactDOMComponentTree to const/let
* Convert ReactDOMComponentTree to const/let
* Convert getNodeForCharacterOffset to const/let
* Convert getTextContentAccessor to const/let
* Convert inputValueTracking to const/let
* Convert setInnerHTML to const/let
* Convert setTextContent to const/let
* Convert validateDOMNesting to const/let
@gaearon gaearon merged commit bb3c22c into facebook:master Nov 6, 2017
@gaearon
Copy link
Copy Markdown
Collaborator

gaearon commented Nov 6, 2017

Thanks.

raphamorim added a commit to raphamorim/react that referenced this pull request Nov 12, 2017
* Convert ReactDOMFiberTextarea to const/let
* Convert ReactDOMSelection to const/let
* Convert setTextContent to const/let
* Convert validateDOMNesting to const/let
raphamorim added a commit to raphamorim/react that referenced this pull request Nov 12, 2017
raphamorim added a commit to raphamorim/react that referenced this pull request Nov 12, 2017
raphamorim added a commit to raphamorim/react that referenced this pull request Nov 12, 2017
* Convert ReactDOMFiberTextarea to const/let
* Convert ReactDOMSelection to const/let
* Convert setTextContent to const/let
* Convert validateDOMNesting to const/let
raphamorim added a commit to raphamorim/react that referenced this pull request Nov 13, 2017
* Convert ReactDOMFiberTextarea to const/let
* Convert ReactDOMSelection to const/let
* Convert setTextContent to const/let
* Convert validateDOMNesting to const/let
gaearon pushed a commit that referenced this pull request Nov 19, 2017
* Use const/let in more places (#11467)

* Convert ReactDOMFiberTextarea to const/let
* Convert ReactDOMSelection to const/let
* Convert setTextContent to const/let
* Convert validateDOMNesting to const/let

* Replace Object.assign by Object Spread

* Convert ReactDOMFiberOption to Object Spread
* Convert ReactDOMFiberTextarea to Object Spread
* Convert validateDOMNesting to Object Spread
raphamorim added a commit to raphamorim/react that referenced this pull request Nov 23, 2017
* Convert BeforeInputEventPlugin to const/let
* Convert BrowserEventConstats to const/let
* Convert ChangeEventPlugin to const/let
* Convert DOMEventPluginOrder to const/let
* Convert EnterLeaveEventPlugin to const/let
* Convert FallbackCompositionState to const/let
* Convert getEventCharCode to const/let
* Convert getEventKey to const/let
Ethan-Arrowood pushed a commit to Ethan-Arrowood/react that referenced this pull request Dec 8, 2017
* Convert ReactDOM to const/let
* Convert ReactDOMComponentTree to const/let
* Convert ReactDOMComponentTree to const/let
* Convert getNodeForCharacterOffset to const/let
* Convert getTextContentAccessor to const/let
* Convert inputValueTracking to const/let
* Convert setInnerHTML to const/let
* Convert setTextContent to const/let
* Convert validateDOMNesting to const/let
Ethan-Arrowood pushed a commit to Ethan-Arrowood/react that referenced this pull request Dec 8, 2017
* Use const/let in more places (facebook#11467)

* Convert ReactDOMFiberTextarea to const/let
* Convert ReactDOMSelection to const/let
* Convert setTextContent to const/let
* Convert validateDOMNesting to const/let

* Replace Object.assign by Object Spread

* Convert ReactDOMFiberOption to Object Spread
* Convert ReactDOMFiberTextarea to Object Spread
* Convert validateDOMNesting to Object Spread
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants