From 3ffd5ab0ee8ffa63199d1428e37206f9b59fb7a5 Mon Sep 17 00:00:00 2001
From: sct <sctsnipe@gmail.com>
Date: Wed, 3 Feb 2021 07:56:26 +0000
Subject: [PATCH] fix(ui): when PersonCard has no profilePath, correctly
 position name/role content

---
 src/components/PersonCard/index.tsx | 39 ++++++++++++++---------------
 1 file changed, 19 insertions(+), 20 deletions(-)

diff --git a/src/components/PersonCard/index.tsx b/src/components/PersonCard/index.tsx
index ce4469202..f01756878 100644
--- a/src/components/PersonCard/index.tsx
+++ b/src/components/PersonCard/index.tsx
@@ -38,34 +38,33 @@ const PersonCard: React.FC<PersonCardProps> = ({
           className={`relative ${
             canExpand ? 'w-full' : 'w-36 sm:w-36 md:w-44'
           } rounded-lg text-white shadow-lg transition ease-in-out duration-150 cursor-pointer transform-gpu ${
-            isHovered ? 'bg-gray-500 scale-105' : 'bg-gray-600 scale-100'
+            isHovered ? 'bg-gray-600 scale-105' : 'bg-gray-700 scale-100'
           }`}
         >
           <div style={{ paddingBottom: '150%' }}>
             <div className="absolute inset-0 flex flex-col items-center w-full h-full p-2">
-              {profilePath && (
-                <div className="relative flex justify-center w-full mt-2 mb-4 h-1/2">
+              <div className="relative flex justify-center w-full mt-2 mb-4 h-1/2">
+                {profilePath ? (
                   <img
                     src={`https://image.tmdb.org/t/p/w600_and_h900_bestv2${profilePath}`}
                     className="object-cover w-3/4 h-full bg-center bg-cover rounded-full"
                     alt=""
                   />
-                </div>
-              )}
-              {!profilePath && (
-                <svg
-                  className="mb-6 w-28 h-28 md:w-32 md:h-32"
-                  fill="currentColor"
-                  viewBox="0 0 20 20"
-                  xmlns="http://www.w3.org/2000/svg"
-                >
-                  <path
-                    fillRule="evenodd"
-                    d="M18 10a8 8 0 11-16 0 8 8 0 0116 0zm-6-3a2 2 0 11-4 0 2 2 0 014 0zm-2 4a5 5 0 00-4.546 2.916A5.986 5.986 0 0010 16a5.986 5.986 0 004.546-2.084A5 5 0 0010 11z"
-                    clipRule="evenodd"
-                  />
-                </svg>
-              )}
+                ) : (
+                  <svg
+                    className="h-full"
+                    fill="currentColor"
+                    viewBox="0 0 20 20"
+                    xmlns="http://www.w3.org/2000/svg"
+                  >
+                    <path
+                      fillRule="evenodd"
+                      d="M18 10a8 8 0 11-16 0 8 8 0 0116 0zm-6-3a2 2 0 11-4 0 2 2 0 014 0zm-2 4a5 5 0 00-4.546 2.916A5.986 5.986 0 0010 16a5.986 5.986 0 004.546-2.084A5 5 0 0010 11z"
+                      clipRule="evenodd"
+                    />
+                  </svg>
+                )}
+              </div>
               <div className="w-full text-center truncate">{name}</div>
               {subName && (
                 <div
@@ -80,7 +79,7 @@ const PersonCard: React.FC<PersonCardProps> = ({
                   {subName}
                 </div>
               )}
-              <div className="absolute bottom-0 left-0 right-0 h-12 rounded-b-lg bg-gradient-to-t from-gray-600" />
+              <div className="absolute bottom-0 left-0 right-0 h-12 rounded-b-lg bg-gradient-to-t from-gray-700" />
             </div>
           </div>
         </div>