From f5e588fbc4ed95b949de3130be2237033d2a5cee Mon Sep 17 00:00:00 2001
From: Yen Colon <yendejesusc41@gmail.com>
Date: Tue, 19 Jan 2021 23:44:15 -0400
Subject: [PATCH] improving part reactions ui

---
 .../components/PartReactions/PartReactions.js | 70 ++++++++++---------
 .../PartReactionsCell/PartReactionsCell.js    |  2 +-
 2 files changed, 39 insertions(+), 33 deletions(-)

diff --git a/web/src/components/PartReactions/PartReactions.js b/web/src/components/PartReactions/PartReactions.js
index d47e878..37e343a 100644
--- a/web/src/components/PartReactions/PartReactions.js
+++ b/web/src/components/PartReactions/PartReactions.js
@@ -13,32 +13,35 @@ const PartReactions = ({ reactions }) => {
   }
 
   return (
-    <>
-      <div className="bg-gray-100 p-4 min-h-md rounded-lg shadow-lg">
-        <Tabs
-          value={tab}
-          onChange={onTabChange}
-          variant="scrollable"
-          scrollButtons="off"
-          textColor="primary"
-          indicatorColor="primary"
-        >
-          {emotes.map((emote, i) => (
-            <Tab
-              label={`${emote.emoji} ${emote.count}`}
-              key={`${emote.emoji}-${i}}`}
-              style={{ minWidth: 100 }}
-            />
-          ))}
-        </Tabs>
-        <ul>
-          {reactions
-            .filter((reaction) => reaction.emote === emotes[tab].emoji)
-            .map((reactionPart, i) => (
-              <li
-                className="flex flex-row p-2 items-center"
-                key={`${reactionPart.emote}-${i}}`}
-              >
+    <div className="bg-gray-100 p-4 min-h-md rounded-lg shadow-lg">
+      <Tabs
+        value={tab}
+        onChange={onTabChange}
+        variant="scrollable"
+        scrollButtons="off"
+        textColor="primary"
+        indicatorColor="primary"
+      >
+        <Tab label="All" style={{ minWidth: 100 }} />
+        {emotes.map((emote, i) => (
+          <Tab
+            label={`${emote.emoji} ${emote.count}`}
+            key={`${emote.emoji}-${i}}`}
+            style={{ minWidth: 100 }}
+          />
+        ))}
+      </Tabs>
+      <ul>
+        {reactions
+          .filter((reaction) =>
+            tab === 0 ? true : reaction.emote === emotes[tab - 1].emoji
+          )
+          .map((reactionPart, i) => (
+            <li
+              className="flex flex-row justify-between p-3 items-center"
+              key={`${reactionPart.emote}-${i}}`}
+            >
+              <div className="flex items-center justify-center">
                 <div className="w-8 h-8 overflow-hidden rounded-full border border-indigo-300 shadow flex-shrink-0">
                   <ImageUploader
                     className=""
@@ -48,7 +51,7 @@ const PartReactions = ({ reactions }) => {
                   />
                 </div>
                 <div className="ml-4 font-roboto">
-                  <div className="text-gray-800 font-bold text-md mb-1">
+                  <div className="text-gray-800 font-normal text-md mb-1">
                     <Link
                       to={routes.user({
                         userName: reactionPart.user?.userName,
@@ -58,11 +61,14 @@ const PartReactions = ({ reactions }) => {
                     </Link>
                   </div>
                 </div>
-              </li>
-            ))}
-        </ul>
-      </div>
-    </>
+              </div>
+              <div>
+                <span>{reactionPart.emote}</span>
+              </div>
+            </li>
+          ))}
+      </ul>
+    </div>
   )
 }
 
diff --git a/web/src/components/PartReactionsCell/PartReactionsCell.js b/web/src/components/PartReactionsCell/PartReactionsCell.js
index d0f4bc9..2cf4c0c 100644
--- a/web/src/components/PartReactionsCell/PartReactionsCell.js
+++ b/web/src/components/PartReactionsCell/PartReactionsCell.js
@@ -22,5 +22,5 @@ export const Empty = () => <div>Empty</div>
 export const Failure = ({ error }) => <div>Error: {error.message}</div>
 
 export const Success = ({ partReactionsByPartId }) => {
-  return <PartReactions reactions={partReactionsByPartId} /> //JSON.stringify(partReactions)
+  return <PartReactions reactions={partReactionsByPartId} />
 }