make linter happy

Author: stereobooster

package.json

@@ -55,7 +55,7 @@
     "rules": {
       "complexity": [
         "error",
-        8
+        12
       ]
     }
   },

src/components/AdaptiveLoad/index.js

@@ -51,6 +51,12 @@ export default class AdaptiveLoad extends Component {
     webpSize: PropTypes.number,
     /** function which decides if image should be downloaded */
     shouldAutoDownload: PropTypes.func,
+    /** URL of the image in webp format */
+    webp: PropTypes.oneOfType([
+      PropTypes.string,
+      PropTypes.func,
+      PropTypes.bool,
+    ]),
 
     // for testing
     /** If you will not pass this value, component will detect onLine status based on browser API, otherwise will use passed value */
@@ -191,7 +197,7 @@ export default class AdaptiveLoad extends Component {
     })
   }
 
-  load = async userTriggered => {
+  load = userTriggered => {
     const {loadState} = this.state
     if (ssr || loaded === loadState || loading === loadState) return
     this.loadStateChange(loading, userTriggered)
@@ -260,19 +266,21 @@ export default class AdaptiveLoad extends Component {
       case loading:
         return overThreshold ? icons.loading : icons.noicon
       case initial:
-        return !onLine
-          ? icons.offline
-          : userTriggered || !shouldAutoDownload
+        if (onLine) {
+          return userTriggered || !shouldAutoDownload
             ? icons.load
             : icons.noicon
+        } else {
+          return icons.offline
+        }
       case error:
-        return !onLine ? icons.offline : icons.error
+        return onLine ? icons.error : icons.offline
       default:
         throw new Error(`Wrong state: ${loadState}`)
     }
   }
 
-  onEnter = async () => {
+  onEnter = () => {
     if (this.state.inViewport) return
     this.setState({inViewport: true})
     if (this.shouldAutoDownload()) this.load(false)

src/components/AdaptiveLoadWithDefaults/index.js

@@ -11,6 +11,7 @@ AdaptiveLoadWithDefaults.defaultProps = {
   theme,
 }
 
+// eslint-disable-next-line react/forbid-foreign-prop-types
 AdaptiveLoadWithDefaults.propTypes = AdaptiveLoad.propTypes
 
 export default AdaptiveLoadWithDefaults

src/components/LazyLoadWithDefaults/index.js

@@ -11,6 +11,7 @@ LazyLoadWithDefaults.defaultProps = {
   theme,
 }
 
+// eslint-disable-next-line react/forbid-foreign-prop-types
 LazyLoadWithDefaults.propTypes = LazyLoad.propTypes
 
 export default LazyLoadWithDefaults

src/components/ManualLoadWithDefaults/index.js

@@ -11,6 +11,7 @@ ManualLoadWithDefaults.defaultProps = {
   theme,
 }
 
+// eslint-disable-next-line react/forbid-foreign-prop-types
 ManualLoadWithDefaults.propTypes = ManualLoad.propTypes
 
 export default ManualLoadWithDefaults

src/components/MediaWithDefaults/index.js

@@ -11,6 +11,7 @@ MediaWithDefaults.defaultProps = {
   theme,
 }
 
+// eslint-disable-next-line react/forbid-foreign-prop-types
 MediaWithDefaults.propTypes = Media.propTypes
 
 export default MediaWithDefaults

src/components/composeStyle.js

@@ -14,15 +14,16 @@
  *
  * <a {...compose(theme.link, theme.active, {color: "#000"})}>link</a>
  *
- * @returns {{className: string, style: object}}
+ * @returns {{className: string, style: object}} - params for React component
  */
 export default (...stylesOrClasses) => {
   const classes = []
   let style
   for (const obj of stylesOrClasses) {
     if (obj instanceof Object) {
       Object.assign(style || (style = {}), obj)
-    } else if (obj === undefined) { // ignore false?
+    } else if (obj === undefined) {
+      // ignore false?
       // ignore
     } else if (typeof obj === 'string') {
       classes.push(obj)

src/components/loaders.js

@@ -9,8 +9,9 @@ import {unfetch, UnfetchAbortController} from './unfetch'
  * If first promise resolved, rejected or canceled
  * second promise will be caneled
  *
- * @param {Promise} p1
- * @param {Promise} p2
+ * @param {Promise} p1 - first promise with cancel
+ * @param {Promise} p2 - second promise with cancel
+ * @returns {Promise} - new promise with cancel
  */
 export const cancelSecond = (p1, p2) => {
   if (!p2) return p1
@@ -49,17 +50,19 @@ export const timeout = threshold => {
 }
 
 export const image = src => {
-  let image = new Image()
+  let img = new Image()
   const result = new Promise((resolve, reject) => {
-    image.onload = resolve
-    image.onabort = image.onerror = () => reject({})
-    image.src = src
+    img.onload = resolve
+    // eslint-disable-next-line no-multi-assign
+    img.onabort = img.onerror = () => reject({})
+    img.src = src
   })
   result.cancel = () => {
-    if (!image) throw new Error('Already canceled')
-    image.onload = image.onabort = image.onerror = undefined
-    image.src = ''
-    image = undefined
+    if (!img) throw new Error('Already canceled')
+    // eslint-disable-next-line no-multi-assign
+    img.onload = img.onabort = img.onerror = undefined
+    img.src = ''
+    img = undefined
   }
   return result
 }

src/components/unfetch.js

@@ -11,11 +11,12 @@ export function UnfetchAbortController() {
 export const unfetch = (url, options) => {
   options = options || {}
   return new Promise((resolve, reject) => {
-    let request = new XMLHttpRequest()
+    const request = new XMLHttpRequest()
 
     request.open(options.method || 'get', url, true)
 
-    for (let i in options.headers) {
+    // eslint-disable-next-line guard-for-in
+    for (const i in options.headers) {
       request.setRequestHeader(i, options.headers[i])
     }
 
@@ -29,17 +30,18 @@ export const unfetch = (url, options) => {
 
     if (options.signal)
       options.signal.onabort = () => {
+        // eslint-disable-next-line no-multi-assign
         request.onerror = request.onload = undefined
         request.abort()
       }
 
     request.send(options.body)
 
     function response() {
-      let keys = [],
-        all = [],
-        headers = {},
-        header
+      const keys = []
+      const all = []
+      const headers = {}
+      let header
 
       request
         .getAllResponseHeaders()
@@ -51,6 +53,7 @@ export const unfetch = (url, options) => {
         })
 
       return {
+        // eslint-disable-next-line no-bitwise
         ok: ((request.status / 100) | 0) === 2, // 200-299
         status: request.status,
         statusText: request.statusText,

src/components/webp.js

@@ -20,8 +20,8 @@
 // }
 
 function supportsWebp() {
-  var elem = document.createElement('canvas')
-  if (!!(elem.getContext && elem.getContext('2d'))) {
+  const elem = document.createElement('canvas')
+  if (elem.getContext && elem.getContext('2d')) {
     // was able or not to get WebP representation
     return elem.toDataURL('image/webp').indexOf('data:image/webp') === 0
   } else {