From 8be89334463509146fa125506ce3d38730f6706b Mon Sep 17 00:00:00 2001
From: Mikayla Dobson <mikaylaherself@gmail.com>
Date: Sat, 27 Aug 2022 14:53:02 -0500
Subject: [PATCH] various changes to reserved card handling in player ui

---
 src/components/Card/Card.tsx          | 77 ++++++++++++++++-----------
 src/components/Card/CardRow.scss      |  1 -
 src/components/Player/AllPlayers.scss | 31 +++++++++++
 src/components/Player/Player.tsx      | 13 +----
 src/sass/helper/_variables.scss       |  9 +---
 src/util/propTypes.ts                 |  1 +
 6 files changed, 80 insertions(+), 52 deletions(-)

diff --git a/src/components/Card/Card.tsx b/src/components/Card/Card.tsx
index 3f7090d..af60a5c 100644
--- a/src/components/Card/Card.tsx
+++ b/src/components/Card/Card.tsx
@@ -1,46 +1,61 @@
 import { v4 } from 'uuid';
 import { CardProps } from '../../util/propTypes';
-import { ResourceCost } from '../../util/types';
+import { CardData, PlayerCards, ResourceCost } from '../../util/types';
 import { useCurrentPlayer } from '../../hooks/useCurrentPlayer';
 import { buyCardActions } from '../Player/ActionMethods';
 import { hasMaxReserved, reserveCard } from '../Player/ActionMethods/reserveCardActions';
 const { buyCard, tooExpensive } = buyCardActions;
 
-export default function Card({ data, state, setState }: CardProps) {
+export default function Card({ data, state, setState, reserved = false }: CardProps) {
     const currentPlayer = useCurrentPlayer(state);
-    if (!data) return <div className="card"></div>;
+    if (!data || !currentPlayer) return <div className="card"></div>;
+
+    const purchaseDisabled = (): boolean => {
+        // TO DO: check whether a card belongs to the current player,
+        // if card is tagged as reserved
+        return tooExpensive(data, state);
+    }
 
     return (
-        <div className="card" key={v4()}>
+        <div className='card' key={v4()}>
             <img src={data.image} loading="lazy" />
-            <div className="foreground">
-                <p>{data.gemValue.toUpperCase()}</p>
-                { (data.points && data.points > 0) ? <p>{data.points} {data.points === 1 ? 'point' : 'points'}</p> : null }
-                <div className="total-card-cost">
+            <div className={reserved ? `foreground-${data.gemValue}` : 'foreground'}>
+                <section className="card-top-section">
+                    <p>{data.gemValue.toUpperCase()}</p>
+                    <div className="total-card-cost">
+                    {
+                        Object.keys(data.resourceCost).map((key: keyof ResourceCost | string) => {
+                            // @ts-ignore
+                            return (data.resourceCost[key as keyof ResourceCost] > 0) && (
+                                <p key={v4()} className={`card-cost-${key}`}>
+                                    {data.resourceCost[key as keyof ResourceCost]}
+                                </p>
+                            )
+                        })
+                    }
+                    </div>
+                    { (data.points && data.points > 0) ? <p>{data.points} {data.points === 1 ? 'point' : 'points'}</p> : null }
+                </section>
+
                 {
-                    Object.keys(data.resourceCost).map((key: keyof ResourceCost | string) => {
-                        // @ts-ignore
-                        return (data.resourceCost[key as keyof ResourceCost] > 0) && (
-                            <p key={v4()} className={`card-cost-${key}`}>
-                                {data.resourceCost[key as keyof ResourceCost]}
-                            </p>
-                        )
-                    })
-                }
-                </div>
-                { state.actions.buyCard.active &&
-                    <button
-                        onClick={() => buyCard(state, setState, data)}
-                        disabled={tooExpensive(data, state)}>
-                        Buy This Card
-                    </button>
-                }
-                { state.actions.reserveCard.active &&
-                    <button
-                        onClick={() => reserveCard(state, setState, data)}
-                        disabled={hasMaxReserved(currentPlayer)}>
-                        Reserve This Card
-                    </button>
+                    (state.actions.buyCard.active || state.actions.reserveCard.active) && (
+                        <section className="card-action-section">
+                            { state.actions.buyCard.active &&
+                                <button
+                                    onClick={() => buyCard(state, setState, data)}
+                                    disabled={purchaseDisabled()}>
+                                    Buy This Card
+                                </button>
+                            }
+                            { !reserved && state.actions.reserveCard.active &&
+                                <button
+                                    onClick={() => reserveCard(state, setState, data)}
+                                    disabled={hasMaxReserved(currentPlayer)}>
+                                    Reserve This Card
+                                </button>
+                            }
+                        </section>
+                    )
                 }
             </div>
         </div>
diff --git a/src/components/Card/CardRow.scss b/src/components/Card/CardRow.scss
index 8a79b83..bfac8e9 100644
--- a/src/components/Card/CardRow.scss
+++ b/src/components/Card/CardRow.scss
@@ -56,7 +56,6 @@
                     @include map-gem-values(".card-cost");
                 }
             }
-
         }
     
         .card-count {
diff --git a/src/components/Player/AllPlayers.scss b/src/components/Player/AllPlayers.scss
index 0d9d233..f823dae 100644
--- a/src/components/Player/AllPlayers.scss
+++ b/src/components/Player/AllPlayers.scss
@@ -1,4 +1,5 @@
 @import '../../sass/helper/mixins';
+@import '../../sass/helper/variables';
 @import '../../sass/helper/placeholders';
 
 .all-players {
@@ -33,5 +34,35 @@
                 @include map-gem-values(".reserve-cost");
             }
         }
+
+        .card {
+            width: 100%;
+
+            img {
+                display: none;
+            }
+
+            @each $gem in $gemlist {
+                @include map-gem-values('.foreground');
+                .foreground-#{$gem} {
+                    display: flex;
+                    flex-direction: column;
+                    align-items: center;
+                    padding: 8px;
+
+                    p {
+                        display: inline;
+                        padding: 8px;
+                    }
+                    
+                    .total-card-cost {
+                        display: flex;
+                        flex-flow: row wrap;
+                        justify-content: center;
+                        @include map-gem-values(".card-cost");
+                    }
+                }
+            }
+        }
     }
 }
\ No newline at end of file
diff --git a/src/components/Player/Player.tsx b/src/components/Player/Player.tsx
index 5baeebb..f3ce8f4 100644
--- a/src/components/Player/Player.tsx
+++ b/src/components/Player/Player.tsx
@@ -29,18 +29,7 @@ export default function Player({ player, state, setState }: PlayerProps) {
         dynamic && setReservedView(
             <div className="reserved-card-view">
             <p>Reserved cards:</p>
-            {
-                dynamic.reservedCards?.map((data: CardData) => (
-                    <div className="reserved-card">
-                        <p className={`mini-card ${data.gemValue}`} key={v4()}>{data.gemValue} card: {data.points || 0} points</p>
-                        <div className="reserved-card-cost">
-                        {
-                            Object.entries(data.resourceCost).map(([key, value]) => value > 0 && <p className={`reserve-cost-${key}`} key={v4()}>{value}</p>)
-                        }
-                        </div>
-                    </div>
-                ))
-            }
+                { dynamic.reservedCards?.map((data: CardData) => <Card key={v4()} data={data} state={state} setState={setState} reserved={true} />) }
             </div>
         )
     }, [dynamic, setState])
diff --git a/src/sass/helper/_variables.scss b/src/sass/helper/_variables.scss
index 47b029f..4e85eae 100644
--- a/src/sass/helper/_variables.scss
+++ b/src/sass/helper/_variables.scss
@@ -1,8 +1 @@
-$emerald-images: 
-    "src/assets/img/SUCCULENT-angele-kamp-unsplash.jpg",
-    "src/assets/img/SUCCULENT-annie-spratt-unsplash.jpg",
-    "src/assets/img/SUCCULENT-calle-macarone-unsplash.jpg",
-    "src/assets/img/SUCCULENT-edgar-castrejon-unsplash.jpg",
-    "src/assets/img/SUCCULENT-jacalyn-beales-unsplash.jpg",
-    "src/assets/img/SUCCULENT-tim-mossholder-unsplash.jpg"
-;
\ No newline at end of file
+$gemlist: 'ruby', 'sapphire', 'onyx', 'emerald', 'diamond', 'gold';
\ No newline at end of file
diff --git a/src/util/propTypes.ts b/src/util/propTypes.ts
index de669f6..d8c4ef9 100644
--- a/src/util/propTypes.ts
+++ b/src/util/propTypes.ts
@@ -7,6 +7,7 @@ export interface StateProps {
 
 export interface CardProps extends StateProps {
     data: CardData
+    reserved?: boolean
 }
 
 export interface CardRowProps extends StateProps {