Fix some linting rules

facebook · Nov 19, 2016 · dd4850e · dd4850e
1 parent 4a066ec
commit dd4850e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/src/renderers/dom/shared/eventPlugins/__tests__/SimpleEventPlugin-test.js b/src/renderers/dom/shared/eventPlugins/__tests__/SimpleEventPlugin-test.js
@@ -96,7 +96,7 @@ describe('SimpleEventPlugin', function() {
   it('does not trigger captured click events when target is a disabled element', function() {
     var element = ReactTestUtils.renderIntoDocument(
       <div onClickCapture={onClick}>
-        <button disabled={true}></button>
+        <button disabled={true} />
       </div>
     );
     var button = ReactDOM.findDOMNode(element).querySelector('button');

diff --git a/src/renderers/shared/shared/ReactTreeTraversal.js b/src/renderers/shared/shared/ReactTreeTraversal.js
@@ -111,10 +111,9 @@ function shouldIgnoreElement(inst) {
 }
 
 function traverseTwoPhase(inst, fn, arg) {
-  // Do not traverse an tree that originates with a disabled
-  // element
+  // Do not traverse an tree that originates with a disabled element
   if (shouldIgnoreElement(inst)) {
-    return
+    return false;
   }
 
   var path = [];
@@ -128,7 +127,7 @@ function traverseTwoPhase(inst, fn, arg) {
   // walking from parent to child
   for (i = path.length; i-- > 0;) {
     // Are we currently, our about to be, within a disabled tree
-    disabled = disabled || shouldIgnoreElement(path[i])
+    disabled = disabled || shouldIgnoreElement(path[i]);
     // If so, remove the current element from traversion if it
     // is interactive
     if (disabled && isInteractive(path[i])) {