Mostly prettify

Author: M-i-k-e-l

lib/components/Keyboard/KeyboardInput/KeyboardAccessoryView.js

@@ -1,6 +1,14 @@
 import React, {Component} from 'react';
 import PropTypes from 'prop-types';
-import {StyleSheet, Platform, NativeModules, NativeEventEmitter, DeviceEventEmitter, processColor, BackHandler} from 'react-native';
+import {
+  StyleSheet,
+  Platform,
+  NativeModules,
+  NativeEventEmitter,
+  DeviceEventEmitter,
+  processColor,
+  BackHandler
+} from 'react-native';
 import KeyboardTrackingView from '../KeyboardTracking/KeyboardTrackingView';
 import CustomKeyboardView from './CustomKeyboardView';
 import KeyboardUtils from './utils/KeyboardUtils';
@@ -28,15 +36,15 @@ class KeyboardAccessoryView extends Component {
     manageScrollView: PropTypes.bool,
     requiresSameParentToManageScrollView: PropTypes.bool,
     addBottomView: PropTypes.bool,
-    allowHitsOutsideBounds: PropTypes.bool,
+    allowHitsOutsideBounds: PropTypes.bool
   };
   static defaultProps = {
     iOSScrollBehavior: -1,
     revealKeyboardInteractive: false,
     manageScrollView: true,
     requiresSameParentToManageScrollView: false,
     addBottomView: false,
-    allowHitsOutsideBounds: false,
+    allowHitsOutsideBounds: false
   };
 
   constructor(props) {
@@ -133,7 +141,7 @@ class KeyboardAccessoryView extends Component {
   render() {
     return (
       <KeyboardTrackingView
-        ref={r => this.trackingViewRef = r}
+        ref={r => (this.trackingViewRef = r)}
         style={styles.trackingToolbarContainer}
         onLayout={this.onContainerComponentHeightChanged}
         scrollBehavior={this.getIOSTrackingScrollBehavior()}
@@ -160,13 +168,11 @@ const styles = StyleSheet.create({
   trackingToolbarContainer: {
     ...Platform.select({
       ios: {
-        position: 'absolute',
-        bottom: 0,
-        left: 0,
-        right: 0,
-      },
-    }),
-  },
+        ...StyleSheet.absoluteFillObject,
+        top: undefined
+      }
+    })
+  }
 });
 
 export default KeyboardAccessoryView;

lib/components/Keyboard/KeyboardInput/KeyboardRegistry.js

@@ -3,14 +3,14 @@ import _ from 'lodash';
 import EventEmitterManager from './utils/EventEmitterManager';
 
 /*
-* Tech debt: how to deal with multiple registries in the app?
-*/
+ * Tech debt: how to deal with multiple registries in the app?
+ */
 
-const getKeyboardsWithIDs = (keyboardIDs) => {
-  return keyboardIDs.map((keyboardId) => {
+const getKeyboardsWithIDs = keyboardIDs => {
+  return keyboardIDs.map(keyboardId => {
     return {
       id: keyboardId,
-      ...KeyboardRegistry.registeredKeyboards[keyboardId].params,
+      ...KeyboardRegistry.registeredKeyboards[keyboardId].params
     };
   });
 };
@@ -21,17 +21,17 @@ export default class KeyboardRegistry {
 
   static registerKeyboard = (componentID, generator, params = {}) => {
     if (!_.isFunction(generator)) {
-      console.error(`KeyboardRegistry.registerKeyboard: ${componentID} you must register a generator function`);//eslint-disable-line
+      console.error(`KeyboardRegistry.registerKeyboard: ${componentID} you must register a generator function`);
       return;
     }
     KeyboardRegistry.registeredKeyboards[componentID] = {generator, params, componentID};
     AppRegistry.registerComponent(componentID, generator);
   };
 
-  static getKeyboard = (componentID) => {
+  static getKeyboard = componentID => {
     const res = KeyboardRegistry.registeredKeyboards[componentID];
     if (!res || !res.generator) {
-      console.error(`KeyboardRegistry.getKeyboard: ${componentID} used but not yet registered`);//eslint-disable-line
+      console.error(`KeyboardRegistry.getKeyboard: ${componentID} used but not yet registered`);
       return undefined;
     }
     return res.generator();
@@ -54,22 +54,22 @@ export default class KeyboardRegistry {
     KeyboardRegistry.eventEmitter.emitEvent(globalID, args);
   };
 
-  static removeListeners = (globalID) => {
+  static removeListeners = globalID => {
     KeyboardRegistry.eventEmitter.removeListeners(globalID);
   };
 
   static onItemSelected = (globalID, args) => {
     KeyboardRegistry.notifyListeners(`${globalID}.onItemSelected`, args);
   };
 
-  static requestShowKeyboard = (globalID) => {
+  static requestShowKeyboard = globalID => {
     KeyboardRegistry.notifyListeners('onRequestShowKeyboard', {keyboardId: globalID});
   };
 
   /**
    * iOS only
    */
-  static toggleExpandedKeyboard = (globalID) => {
+  static toggleExpandedKeyboard = globalID => {
     KeyboardRegistry.notifyListeners('onToggleExpandedKeyboard', {keyboardId: globalID});
   };
 }

lib/components/Keyboard/KeyboardInput/TextInputKeyboardManager.ios.js

@@ -3,7 +3,6 @@ import ReactNative, {NativeModules, LayoutAnimation} from 'react-native';
 const CustomInputController = NativeModules.CustomInputController;
 
 export default class TextInputKeyboardManager {
-
   static setInputComponent = (textInputRef, {component, initialProps}) => {
     if (!textInputRef || !CustomInputController) {
       return;
@@ -14,7 +13,7 @@ export default class TextInputKeyboardManager {
     }
   };
 
-  static removeInputComponent = (textInputRef) => {
+  static removeInputComponent = textInputRef => {
     if (!textInputRef || !CustomInputController) {
       return;
     }
@@ -51,14 +50,14 @@ const springAnimation = {
   duration: 400,
   create: {
     type: LayoutAnimation.Types.linear,
-    property: LayoutAnimation.Properties.opacity,
+    property: LayoutAnimation.Properties.opacity
   },
   update: {
     type: LayoutAnimation.Types.spring,
-    springDamping: 1.0,
+    springDamping: 1.0
   },
   delete: {
     type: LayoutAnimation.Types.linear,
-    property: LayoutAnimation.Properties.opacity,
-  },
+    property: LayoutAnimation.Properties.opacity
+  }
 };

lib/components/Keyboard/KeyboardInput/__tests__/KeyboardRegistry.spec.js

@@ -12,7 +12,7 @@ describe('KeyboardRegistry - components', () => {
 
   beforeEach(() => {
     AppRegistry.registerComponent = jest.fn(AppRegistry.registerComponent);
-    console.error = jest.fn(); //eslint-disable-line
+    console.error = jest.fn();
   });
 
   it('should register the component in the keyboard registry', () => {
@@ -31,18 +31,18 @@ describe('KeyboardRegistry - components', () => {
   it('should fail if generator is not provided and produce an error', () => {
     KeyboardRegistry.registerKeyboard(mockComponent);
     expect(AppRegistry.registerComponent).not.toHaveBeenCalled();
-    expect(console.error).toHaveBeenCalledTimes(1); //eslint-disable-line
+    expect(console.error).toHaveBeenCalledTimes(1);
   });
 
   it('should only allow to register a generator function and produce an error', () => {
     KeyboardRegistry.registerKeyboard(mockComponent, MockElement);
     expect(AppRegistry.registerComponent).not.toHaveBeenCalled();
-    expect(console.error).toHaveBeenCalledTimes(1); //eslint-disable-line
+    expect(console.error).toHaveBeenCalledTimes(1);
   });
 
   it('should produce an error if component was not and return undefined', () => {
     const res = KeyboardRegistry.getKeyboard('not_existing_component');
-    expect(console.error).toHaveBeenCalledTimes(1); //eslint-disable-line
+    expect(console.error).toHaveBeenCalledTimes(1);
     expect(res).toBe(undefined);
   });
 
@@ -85,7 +85,7 @@ describe('KeyboardRegistry - components', () => {
     KeyboardRegistry.registerKeyboard(mockComponent, mockGenerator);
     KeyboardRegistry.registerKeyboard(anotherMockComponent, anotherMockGenerator);
 
-    let keyboards = KeyboardRegistry.getKeyboards([anotherMockComponent, mockComponent]);
+    const keyboards = KeyboardRegistry.getKeyboards([anotherMockComponent, mockComponent]);
     expect(keyboards).toEqual([{id: anotherMockComponent}, {id: mockComponent}]);
   });
 });
@@ -99,7 +99,7 @@ describe('KeyboardRegistry - listeners', () => {
     KeyboardRegistry.eventEmitter = {
       listenOn: jest.fn(),
       emitEvent: jest.fn(),
-      removeListeners: jest.fn(),
+      removeListeners: jest.fn()
     };
   });
 

lib/components/Keyboard/KeyboardInput/utils/EventEmitterManager.js

@@ -16,7 +16,7 @@ export default class EventEmitterManager {
 
   emitEvent(eventName, params = {}) {
     if (this.handlerCallbacks[eventName]) {
-      (this.handlerCallbacks[eventName]).forEach(callback => callback(params));
+      this.handlerCallbacks[eventName].forEach(callback => callback(params));
     }
   }
 
@@ -28,7 +28,7 @@ export default class EventEmitterManager {
     const handlers = this.handlerCallbacks[eventName];
 
     if (handlers) {
-      (handlers).forEach((handler, index) => {
+      handlers.forEach((handler, index) => {
         if (handler === listener) {
           handlers.splice(index, 1);
         }

lib/components/Keyboard/KeyboardTracking/KeyboardTrackingView.ios.js

@@ -26,7 +26,7 @@ class KeyboardTrackingView extends PureComponent {
      * you still get the keyboard events and the view just hovers when you focus the input.
      * Also, if you're not using interactive style of dismissing the keyboard
      * (or if you don't have an input inside this view) it doesn't make sense to track it anyway.
-     * (This is caused because of the usage of inputAccessory to be able to track the 
+     * (This is caused because of the usage of inputAccessory to be able to track the
      * keyboard interactive change and it introduces this bug)
      */
     trackInteractive: PropTypes.bool