B-26364 adjust for comments

src/components/drivers/pdf-viewer.jsx

@@ -138,6 +138,7 @@ Object.freeze(ZOOM_STEPS)
 
 const MAX_ZOOM_INDEX = ZOOM_VALUES.length - 1
 const DEFAULT_ZOOM_INDEX = 8 // 1.0 (100%)
+const WIDTH_FIT_SCALE = 0.8 // 80% screen width
 
 export default class PDFDriver extends React.Component {
   constructor(props) {
@@ -220,7 +221,7 @@ export default class PDFDriver extends React.Component {
       scale: 1.0,
       rotation: this.props.rotationValue || 0,
     })
-    const scaleWidth = (containerWidth / viewport.width) * 0.8
+    const scaleWidth = (containerWidth / viewport.width) * WIDTH_FIT_SCALE
     return this.findClosestZoomStep(scaleWidth)
   }
 

src/components/drivers/photo-viewer.jsx

@@ -38,11 +38,10 @@ export default class PhotoViewer extends Component {
   constructor(props) {
     super(props)
 
-    const { originalHeight, originalWidth, width: viewerWidth } = props
+    const { originalWidth, width: viewerWidth } = props
 
     const bestFitZoomIndex = this.calculateBestFitZoom(
       originalWidth,
-      originalHeight,
       viewerWidth
     )
 
@@ -82,13 +81,9 @@ export default class PhotoViewer extends Component {
     return closestIndex
   }
 
-  calculateBestFitZoom(imageWidth, imageHeight, containerWidth) {
-    if (imageWidth > imageHeight) {
-      const scaleWidth = (containerWidth / imageWidth) * WIDTH_FIT_SCALE
-      return this.findClosestZoomStep(scaleWidth)
-    }
-
-    return this.findClosestZoomStep(1.0)
+  calculateBestFitZoom(imageWidth, containerWidth) {
+    const scaleWidth = (containerWidth / imageWidth) * WIDTH_FIT_SCALE
+    return this.findClosestZoomStep(scaleWidth)
   }
 
   setZoom(index) {