fix: linting

britannica · Nov 23, 2020 · 7048da0 · 7048da0
1 parent 6feecc7
commit 7048da0
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 38 deletions.
diff --git a/src/components/MediaOverlay/Media/Media.js b/src/components/MediaOverlay/Media/Media.js
@@ -1,46 +1,46 @@
 import PropTypes from 'prop-types';
-import React, { Fragment } from 'react';
+import React, { useContext } from 'react';
 import MediaOverlayContext from '../MediaOverlay.context';
 import DefaultAudio from './DefaultAudio/DefaultAudio';
 import DefaultImage from './DefaultImage/DefaultImage';
 import DefaultInteractive from './DefaultInteractive/DefaultInteractive';
 import DefaultVideo from './DefaultVideo/DefaultVideo';
 import './Media.module.scss';
 
-const Media = () => (
-  <MediaOverlayContext.Consumer>
-    {({
-      overlayState: { assembly },
-      overlayProps: {
-        videoPlayerId,
-        generatePrerollUrl,
-        audioComponent: Audio = DefaultAudio,
-        imageComponent: Image = DefaultImage,
-        interactiveComponent: Interactive = DefaultInteractive,
-        videoComponent: Video = DefaultVideo,
-      },
-      hideSidebarAndControls,
-      overlayRef,
-      showSidebarAndControls,
-    }) => (
-      <>
-        {assembly.audio && <Audio filename={assembly.audio.filename} />}
-        {assembly.image && <Image {...assembly.image} lazyContainer={overlayRef.current} />}
-        {assembly.interactive && <Interactive {...assembly.interactive} />}
-        {assembly.video && (
-          <Video
-            assembly={assembly}
-            playerId={videoPlayerId}
-            onPlay={hideSidebarAndControls}
-            onPause={showSidebarAndControls}
-            onDisplayClick={hideSidebarAndControls}
-            generatePrerollUrl={generatePrerollUrl}
-          />
-        )}
-      </>
-    )}
-  </MediaOverlayContext.Consumer>
-);
+const Media = () => {
+  const {
+    overlayState: { assembly },
+    overlayProps: {
+      videoPlayerId,
+      generatePrerollUrl,
+      audioComponent: Audio = DefaultAudio,
+      imageComponent: Image = DefaultImage,
+      interactiveComponent: Interactive = DefaultInteractive,
+      videoComponent: Video = DefaultVideo,
+    },
+    hideSidebarAndControls,
+    overlayRef,
+    showSidebarAndControls,
+  } = useContext(MediaOverlayContext);
+
+  return (
+    <>
+      {assembly.audio && <Audio filename={assembly.audio.filename} />}
+      {assembly.image && <Image {...assembly.image} lazyContainer={overlayRef.current} />}
+      {assembly.interactive && <Interactive {...assembly.interactive} />}
+      {assembly.video && (
+      <Video
+        assembly={assembly}
+        playerId={videoPlayerId}
+        onPlay={hideSidebarAndControls}
+        onPause={showSidebarAndControls}
+        onDisplayClick={hideSidebarAndControls}
+        generatePrerollUrl={generatePrerollUrl}
+      />
+      )}
+    </>
+  );
+};
 
 Media.propTypes = {
   assemblyId: PropTypes.number,

diff --git a/src/components/MediaOverlay/MediaViewer/MediaViewer.js b/src/components/MediaOverlay/MediaViewer/MediaViewer.js
@@ -23,9 +23,9 @@ const MediaViewer = () => {
       onSwipeRight={navigatePreviousMedia}
       className={classNames(styles.MediaViewer, { controlsHidden })}
     >
-      {hasError ? <Fragment>{localeLabels.ERROR}</Fragment> : <Media />}
+      {hasError ? localeLabels.ERROR : <Media />}
       {assemblies.length > 1 && !controlsHidden && (
-        <Fragment>
+        <>
           {mediaIndex > 0 && (
             <MediaLink
               assemblyId={assemblies[mediaIndex - 1].assemblyId}
@@ -42,7 +42,7 @@ const MediaViewer = () => {
               <FontAwesomeIcon icon={faAngleRight} size="3x" />
             </MediaLink>
           )}
-        </Fragment>
+        </>
       )}
     </Taparoo>
   );