From 5c412fffee6b0705def8af938ef1a8b8d3a91de2 Mon Sep 17 00:00:00 2001
From: Kurt Hutten <kurt@kurthutten.com>
Date: Wed, 28 Oct 2020 20:47:05 +1100
Subject: [PATCH 1/3] Inconsequential changes

---
 web/src/components/PartForm/ImageUploader.js | 2 +-
 web/src/layouts/MainLayout/MainLayout.js     | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/web/src/components/PartForm/ImageUploader.js b/web/src/components/PartForm/ImageUploader.js
index 4809ec0..b88023e 100644
--- a/web/src/components/PartForm/ImageUploader.js
+++ b/web/src/components/PartForm/ImageUploader.js
@@ -9,7 +9,7 @@ import 'react-image-crop/dist/ReactCrop.css'
 import Svg from 'src/components/Svg/Svg.js'
 
 const CLOUDINARY_UPLOAD_PRESET = "CadHub_project_images";
-const CLOUDINARY_UPLOAD_URL = "https://api.cloudinary.com/v1_1/irevdev/upload/?custom_coordinates=10,10,20,20";
+const CLOUDINARY_UPLOAD_URL = "https://api.cloudinary.com/v1_1/irevdev/upload";
 
 export default function ImageUploader({ onImageUpload, imageUrl }) {
   const [isModalOpen, setIsModalOpen] = useState(false)
diff --git a/web/src/layouts/MainLayout/MainLayout.js b/web/src/layouts/MainLayout/MainLayout.js
index 8621647..ae55bd0 100644
--- a/web/src/layouts/MainLayout/MainLayout.js
+++ b/web/src/layouts/MainLayout/MainLayout.js
@@ -23,6 +23,7 @@ const MainLayout = ({ children }) => {
             <li>
               <Tooltip title="Very alpha, there's lots of work todo" >
                 <div className="ml-12 flex">
+                  {/* Because of how specific these styles are to this heading/logo and it doesn't need to be replicated else where as well as it's very precise with the placement of "pre-alpha" I think it's appropriate. */}
                   <h2 className="text-indigo-300 text-5xl font-ropa-sans py-1 tracking-wider" style={{letterSpacing: '0.3em'}}>CadHub</h2>
                   <div className="text-pink-400 text-sm font-bold font-ropa-sans" style={{paddingBottom: '2rem', marginLeft: '-1.8rem'}}>pre-alpha</div>
                 </div>

From 1bad0de25f8d20ffa74c309a0ff4b5079be22ef9 Mon Sep 17 00:00:00 2001
From: Kurt Hutten <kurt@kurthutten.com>
Date: Wed, 28 Oct 2020 21:19:26 +1100
Subject: [PATCH 2/3] A few quick tweaks to emoji component

---
 .../components/EmojiReaction/EmojiReaction.js   | 17 ++++++++++-------
 web/src/components/Parts/Parts.js               | 14 +++++++++++++-
 web/src/components/Svg/Svg.js                   |  2 +-
 web/src/index.css                               |  2 +-
 4 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/web/src/components/EmojiReaction/EmojiReaction.js b/web/src/components/EmojiReaction/EmojiReaction.js
index 83fb7b3..0f2d162 100644
--- a/web/src/components/EmojiReaction/EmojiReaction.js
+++ b/web/src/components/EmojiReaction/EmojiReaction.js
@@ -38,12 +38,15 @@ const EmojiReaction = ({ emotes, callback = () => {} }) => {
 
   return [
     <div className="flex justify-between">
-      <Fab variant="round" aria-describedby={popoverId} onClick={togglePopover}>
-        <Svg name="dots-vertical" />
-      </Fab>
+        <Fab size="medium" variant="round" aria-describedby={popoverId} onClick={togglePopover}>
+          <div className="bg-gray-200 border-2 m-px border-gray-300 text-gray-500 absolute inset-0 rounded-full flex justify-center items-center">
+            <Svg name="dots-vertical" />
+          </div>
+        </Fab>
+
       <div>
-        {emotes.map((emote) => (
-          <IconButton onClick={() => handleEmojiClick(emote.emoji)}>
+        {emotes.map((emote, i) => (
+          <IconButton key={`${emote.emoji}--${i}`} onClick={() => handleEmojiClick(emote.emoji)}>
             {emote.emoji} <span>{emote.count}</span>
           </IconButton>
         ))}
@@ -63,8 +66,8 @@ const EmojiReaction = ({ emotes, callback = () => {} }) => {
         horizontal: 'center',
       }}
     >
-      {emojiMenu.map((emoji) => (
-        <IconButton onClick={() => handleEmojiClick(emoji)}>{emoji}</IconButton>
+      {emojiMenu.map((emoji, i) => (
+        <IconButton key={`${emoji}-${i}}`} onClick={() => handleEmojiClick(emoji)}>{emoji}</IconButton>
       ))}
     </Popover>,
   ]
diff --git a/web/src/components/Parts/Parts.js b/web/src/components/Parts/Parts.js
index 8f49c7b..b5573e8 100644
--- a/web/src/components/Parts/Parts.js
+++ b/web/src/components/Parts/Parts.js
@@ -1,6 +1,7 @@
 import { useMutation, useFlash } from '@redwoodjs/web'
 import { Link, routes } from '@redwoodjs/router'
 import { Image as CloudinaryImage } from 'cloudinary-react'
+import EmojiiReaction from 'src/components/EmojiReaction/EmojiReaction'
 
 import avatar from 'src/assets/harold.jpg'
 
@@ -53,6 +54,17 @@ const PartsList = ({ parts }) => {
   }
 
   return (
+    <>
+      <EmojiiReaction emotes={[
+      {
+         emoji: '❤️',
+         count: 3,
+      },
+      {
+         emoji: '😁',
+         count: 2,
+      },
+   ]} callback={(e) => console.log(e)} />
     <div className="max-w-xs sm:max-w-sm md:max-w-2xl lg:max-w-5xl xl:max-w-6xl mx-auto grid gap-8 grid-cols-1 md:grid-cols-2 lg:grid-cols-3 xl:grid- cols-4">
       {parts.map((part) => {
         return (
@@ -82,7 +94,7 @@ const PartsList = ({ parts }) => {
             </div>
           </Link>
       )})}
-    </div>
+    </div></>
   )
 }
 
diff --git a/web/src/components/Svg/Svg.js b/web/src/components/Svg/Svg.js
index 43ec2ff..0e03c06 100644
--- a/web/src/components/Svg/Svg.js
+++ b/web/src/components/Svg/Svg.js
@@ -11,7 +11,7 @@ const Svg = ({name, className: className2, strokeWidth = 2}) => {
       <path strokeLinecap="round" strokeLinejoin="round" strokeWidth={strokeWidth} d="M15.232 5.232l3.536 3.536m-2.036-5.036a2.5 2.5 0 113.536 3.536L6.5 21.036H3v-3.572L16.732 3.732z" />
     </svg>,
     "dots-vertical": <svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 24 24" stroke="currentColor">
-      <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M12 5v.01M12 12v.01M12 19v.01M12 6a1 1 0 110-2 1 1 0 010 2zm0 7a1 1 0 110-2 1 1 0 010 2zm0 7a1 1 0 110-2 1 1 0 010 2z" />
+      <path strokeLinecap="round" strokeLinejoin="round" strokeWidth={strokeWidth} d="M12 5v.01M12 12v.01M12 19v.01M12 6a1 1 0 110-2 1 1 0 010 2zm0 7a1 1 0 110-2 1 1 0 010 2zm0 7a1 1 0 110-2 1 1 0 010 2z" />
     </svg>
   }
 
diff --git a/web/src/index.css b/web/src/index.css
index eed36d4..5b01aec 100644
--- a/web/src/index.css
+++ b/web/src/index.css
@@ -36,7 +36,7 @@
 
 body {
   /* TODO can I use a tailwind class here? */
-  background-color: #4a5568;
+  background-color: #E5E5E5;
 }
 
 button, input, label, textarea {

From 171299dd822cc99bb6261e823f9be8ced312c18e Mon Sep 17 00:00:00 2001
From: Kurt Hutten <kurt@kurthutten.com>
Date: Wed, 28 Oct 2020 21:21:09 +1100
Subject: [PATCH 3/3] Remove mistake

---
 web/src/components/Parts/Parts.js | 14 +-------------
 1 file changed, 1 insertion(+), 13 deletions(-)

diff --git a/web/src/components/Parts/Parts.js b/web/src/components/Parts/Parts.js
index b5573e8..8f49c7b 100644
--- a/web/src/components/Parts/Parts.js
+++ b/web/src/components/Parts/Parts.js
@@ -1,7 +1,6 @@
 import { useMutation, useFlash } from '@redwoodjs/web'
 import { Link, routes } from '@redwoodjs/router'
 import { Image as CloudinaryImage } from 'cloudinary-react'
-import EmojiiReaction from 'src/components/EmojiReaction/EmojiReaction'
 
 import avatar from 'src/assets/harold.jpg'
 
@@ -54,17 +53,6 @@ const PartsList = ({ parts }) => {
   }
 
   return (
-    <>
-      <EmojiiReaction emotes={[
-      {
-         emoji: '❤️',
-         count: 3,
-      },
-      {
-         emoji: '😁',
-         count: 2,
-      },
-   ]} callback={(e) => console.log(e)} />
     <div className="max-w-xs sm:max-w-sm md:max-w-2xl lg:max-w-5xl xl:max-w-6xl mx-auto grid gap-8 grid-cols-1 md:grid-cols-2 lg:grid-cols-3 xl:grid- cols-4">
       {parts.map((part) => {
         return (
@@ -94,7 +82,7 @@ const PartsList = ({ parts }) => {
             </div>
           </Link>
       )})}
-    </div></>
+    </div>
   )
 }