From 07cbb193acbf8a7d3fb061bfd720b16b5b50881c Mon Sep 17 00:00:00 2001
From: aswanson-nr <aswanson@newrelic.com>
Date: Mon, 3 Jan 2022 16:48:14 -0800
Subject: [PATCH] feat: clean up image styling

---
 src/components/PackImg.js          |  3 +-
 src/components/PackTile.js         | 62 ++++++++++++++++--------------
 src/pages/instant-observability.js | 15 +++++---
 3 files changed, 45 insertions(+), 35 deletions(-)

diff --git a/src/components/PackImg.js b/src/components/PackImg.js
index f6b70b3e3..1505e9491 100644
--- a/src/components/PackImg.js
+++ b/src/components/PackImg.js
@@ -24,7 +24,8 @@ const PackImg = ({ className, logoUrl, packName }) => {
       <img
         css={css`
           display: block;
-          width: 100%;
+          max-width: 100%;
+          max-height: 100%;
         `}
         src={logoUrl}
         alt={packName}
diff --git a/src/components/PackTile.js b/src/components/PackTile.js
index 82f70022e..5d582f878 100644
--- a/src/components/PackTile.js
+++ b/src/components/PackTile.js
@@ -16,6 +16,8 @@ import {
 } from '../data/constants';
 import PackImg from './PackImg';
 
+const IMAGE_BREAKPOINT = 1080;
+
 const VIEWS = {
   GRID: 'Grid view',
   LIST: 'List view',
@@ -74,15 +76,17 @@ const PackTile = ({
       interactive
       css={css`
         overflow: hidden;
-        display: flex;
-        flex-direction: column;
-
+        //display: flex;
+        //flex-direction: column;
+        display: grid;
+        grid-template-rows: 1fr 3.5fr 0.5fr;
         ${view === VIEWS.LIST &&
         css`
           margin-bottom: 1em;
           display: grid;
           grid-template-columns: 1fr 3.5fr 0.5fr;
-          flex-direction: row;
+          @media (max-width: 1080px) {
+            grid-template-columns: 4.5fr 0.5fr;
         `}
 
         @media screen and (max-width: ${QUICKSTARTS_COLLAPSE_BREAKPOINT}) {
@@ -91,36 +95,38 @@ const PackTile = ({
       `}
       onClick={() => handlePackClick(id)}
     >
-      <PackImg
-        logoUrl={logoUrl}
-        packName={title || name}
+      <div
         css={css`
-          height: 100px;
-          object-fit: scale-down;
-          width: 100%;
-          padding: 5% 5%;
-          margin: 0 auto 10px;
+          height: 130px;
+        `}
+      >
+        <PackImg
+          logoUrl={logoUrl}
+          packName={title || name}
+          css={css`
+            object-fit: scale-down;
+            padding: 5% 5%;
+            margin: 0 auto 10px;
 
-          .dark-mode & {
-            background-color: white;
-          }
+            .dark-mode & {
+              background-color: white;
+            }
 
-          ${view === VIEWS.LIST &&
-          css`
-            max-height: 150px;
-            height: 100%;
+            ${view === VIEWS.LIST &&
+            css`
+              max-height: 150px;
+              height: 100%;
 
-            flex: 0 0 auto;
-            @media (max-width: 1080px) {
+              @media (max-width: ${IMAGE_BREAKPOINT}px) {
+                display: none;
+              }
+            `}
+            @media screen and (max-width: ${QUICKSTARTS_COLLAPSE_BREAKPOINT}) {
               display: none;
             }
           `}
-
-          @media screen and (max-width: ${QUICKSTARTS_COLLAPSE_BREAKPOINT}) {
-            display: none;
-          }
-        `}
-      />
+        />
+      </div>
       <div
         css={css`
           padding: 1em;
@@ -130,7 +136,7 @@ const PackTile = ({
           css`
             width: 100%;
             flex: 1 1 auto;
-            @media (max-width: 1080px) {
+            @media (max-width: ${IMAGE_BREAKPOINT}px) {
               width: 100%;
             }
           `}
diff --git a/src/pages/instant-observability.js b/src/pages/instant-observability.js
index 6cd68b600..6e625571f 100644
--- a/src/pages/instant-observability.js
+++ b/src/pages/instant-observability.js
@@ -506,13 +506,16 @@ const QuickstartsPage = ({ data, location }) => {
                 287px
               );
 
-              @media (max-width: 1350px) {
-                grid-template-columns: repeat(3, 1fr);
-              }
+              ${view === VIEWS.GRID &&
+              css`
+                @media (max-width: 1350px) {
+                  grid-template-columns: repeat(3, 1fr);
+                }
 
-              @media (max-width: 1180px) {
-                grid-template-columns: repeat(1, 1fr);
-              }
+                @media (max-width: 1180px) {
+                  grid-template-columns: repeat(1, 1fr);
+                }
+              `}
 
               ${view === VIEWS.LIST &&
               css`