From 33ea19fef60f01e165fbd590bffb5ae9244d97f7 Mon Sep 17 00:00:00 2001
From: Mitchell McCaffrey <mitchemmc@gmail.com>
Date: Fri, 20 Mar 2020 17:56:34 +1100
Subject: [PATCH] Moved tokens to be relatively positioned and scaled to the
 current map

---
 src/components/AddMapButton.js | 45 ++++++++++++++---
 src/components/Map.js          | 88 +++++++++++++++++++++++-----------
 src/components/ProxyToken.js   | 25 +++++++---
 src/components/Token.js        | 13 +----
 src/components/Tokens.js       |  6 ++-
 src/routes/Game.js             | 25 +++++-----
 6 files changed, 136 insertions(+), 66 deletions(-)

diff --git a/src/components/AddMapButton.js b/src/components/AddMapButton.js
index 5a67a02..7d8820e 100644
--- a/src/components/AddMapButton.js
+++ b/src/components/AddMapButton.js
@@ -1,5 +1,13 @@
-import React, { useRef, useState } from "react";
-import { IconButton, Box, Button, Image, Flex, Label, Input } from "theme-ui";
+import React, { useRef, useState, useEffect } from "react";
+import {
+  IconButton,
+  Box,
+  Button,
+  Image as UIImage,
+  Flex,
+  Label,
+  Input
+} from "theme-ui";
 
 import Modal from "./Modal";
 
@@ -22,11 +30,26 @@ function AddMapButton({ handleMapChange }) {
     setIsAddModalOpen(false);
   }
 
+  const [imageLoaded, setImageLoaded] = useState(false);
+
   const mapDataRef = useRef(null);
-  const [mapSource, setImageSource] = useState(null);
+  const [mapSource, setMapSource] = useState(null);
   function handleImageUpload(event) {
-    mapDataRef.current = { file: event.target.files[0], rows, columns };
-    setImageSource(URL.createObjectURL(mapDataRef.current.file));
+    const file = event.target.files[0];
+    const url = URL.createObjectURL(file);
+    let image = new Image();
+    image.onload = function() {
+      mapDataRef.current = {
+        file,
+        rows,
+        columns,
+        width: image.width,
+        height: image.height
+      };
+      setImageLoaded(true);
+    };
+    image.src = url;
+    setMapSource(url);
   }
 
   function handleDone() {
@@ -38,6 +61,12 @@ function AddMapButton({ handleMapChange }) {
 
   const [rows, setRows] = useState(defaultMapSize);
   const [columns, setColumns] = useState(defaultMapSize);
+  useEffect(() => {
+    if (mapDataRef.current) {
+      mapDataRef.current.rows = rows;
+      mapDataRef.current.columns = columns;
+    }
+  }, [rows, columns]);
 
   return (
     <>
@@ -70,7 +99,7 @@ function AddMapButton({ handleMapChange }) {
           />
 
           <Flex sx={{ flexDirection: "column" }}>
-            <Image
+            <UIImage
               my={2}
               sx={{
                 width: "500px",
@@ -104,7 +133,9 @@ function AddMapButton({ handleMapChange }) {
               </Box>
             </Flex>
             {mapSource ? (
-              <Button variant="primary">Done</Button>
+              <Button variant="primary" disabled={!imageLoaded}>
+                Done
+              </Button>
             ) : (
               <Button
                 varient="primary"
diff --git a/src/components/Map.js b/src/components/Map.js
index b0b1f01..3f39ad2 100644
--- a/src/components/Map.js
+++ b/src/components/Map.js
@@ -1,12 +1,13 @@
-import React from "react";
-import { Image, Flex, Box } from "theme-ui";
+import React, { useRef } from "react";
+import { Box, Image } from "theme-ui";
 
 import Token from "../components/Token";
 import ProxyToken from "../components/ProxyToken";
 
 const mapTokenClassName = "map-token";
+const defaultTokenSize = 48;
 
-function Map({ imageSource, tokens, onMapTokenMove, onMapTokenRemove }) {
+function Map({ mapSource, mapData, tokens, onMapTokenMove, onMapTokenRemove }) {
   function handleProxyDragEnd(isOnMap, token) {
     if (isOnMap && onMapTokenMove) {
       onMapTokenMove(token);
@@ -17,43 +18,76 @@ function Map({ imageSource, tokens, onMapTokenMove, onMapTokenRemove }) {
     }
   }
 
+  const mapRef = useRef(null);
+  const rows = mapData && mapData.rows;
+  const tokenSizePercent = (1 / rows) * 100;
+  const aspectRatio = (mapData && mapData.width / mapData.height) || 1;
+
   return (
     <>
-      <Flex
+      <Box
         className="map"
-        sx={{ justifyContent: "center", flexGrow: 1 }}
+        sx={{ flexGrow: 1, position: "relative", overflow: "hidden" }}
         bg="background"
       >
         <Box
           sx={{
-            position: "absolute",
-            top: 0,
-            left: 0,
-            right: 0,
-            userSelect: "none"
+            position: "relative",
+            overflow: "hidden",
+            top: "50%",
+            left: "50%",
+            transform: "translate(-50%, -50%)"
           }}
         >
-          {Object.values(tokens).map(token => (
-            <Box
-              key={token.id}
-              sx={{
-                position: "absolute",
-                transform: `translate(${token.x}px, ${token.y}px)`
-              }}
-            >
-              <Token
-                tokenId={token.id}
-                image={token.image}
-                className={mapTokenClassName}
-              />
-            </Box>
-          ))}
+          <Box
+            sx={{
+              width: "100%",
+              height: 0,
+              paddingBottom: `${(1 / aspectRatio) * 100}%`
+            }}
+          />
+
+          <Box
+            sx={{ position: "absolute", top: 0, right: 0, bottom: 0, left: 0 }}
+          >
+            <Image ref={mapRef} id="map" src={mapSource} />
+          </Box>
+          <Box
+            sx={{
+              position: "absolute",
+              top: 0,
+              left: 0,
+              right: 0,
+              bottom: 0
+            }}
+          >
+            {Object.values(tokens).map(token => (
+              <Box
+                key={token.id}
+                style={{
+                  left: `${token.x * 100}%`,
+                  top: `${token.y * 100}%`,
+                  width: `${tokenSizePercent}%`
+                }}
+                sx={{
+                  position: "absolute",
+                  display: "flex"
+                }}
+              >
+                <Token
+                  tokenId={token.id}
+                  image={token.image}
+                  className={mapTokenClassName}
+                />
+              </Box>
+            ))}
+          </Box>
         </Box>
-        <Image src={imageSource} sx={{ objectFit: "contain" }} />
-      </Flex>
+      </Box>
       <ProxyToken
         tokenClassName={mapTokenClassName}
         onProxyDragEnd={handleProxyDragEnd}
+        size={defaultTokenSize}
       />
     </>
   );
diff --git a/src/components/ProxyToken.js b/src/components/ProxyToken.js
index 069fdae..d264fb1 100644
--- a/src/components/ProxyToken.js
+++ b/src/components/ProxyToken.js
@@ -5,7 +5,7 @@ import interact from "interactjs";
 
 import usePortal from "../helpers/usePortal";
 
-function ProxyToken({ tokenClassName, onProxyDragEnd }) {
+function ProxyToken({ tokenClassName, onProxyDragEnd, size }) {
   const proxyContainer = usePortal("root");
 
   const [imageSource, setImageSource] = useState("");
@@ -67,8 +67,20 @@ function ProxyToken({ tokenClassName, onProxyDragEnd }) {
           let proxy = imageRef.current;
           if (proxy) {
             if (onProxyDragEnd) {
-              const x = parseFloat(proxy.getAttribute("data-x")) || 0;
-              const y = parseFloat(proxy.getAttribute("data-y")) || 0;
+              // TODO: look at cleaning this up with a props instead of
+              // hard coding an id
+              const map = document.getElementById("map");
+              const mapRect = map.getBoundingClientRect();
+
+              let x = parseFloat(proxy.getAttribute("data-x")) || 0;
+              let y = parseFloat(proxy.getAttribute("data-y")) || 0;
+              // Convert coordiantes to be relative to the map
+              x = x - mapRect.left;
+              y = y - mapRect.top;
+              // Normalize to map width
+              x = x / (mapRect.right - mapRect.left);
+              y = y / (mapRect.bottom - mapRect.top);
+
               const id = target.getAttribute("data-token-id");
               onProxyDragEnd(proxyOnMap.current, {
                 image: imageSource,
@@ -109,11 +121,10 @@ function ProxyToken({ tokenClassName, onProxyDragEnd }) {
       }}
     >
       <Image
-        p={2}
         src={imageSource}
         sx={{
-          width: "64px",
-          height: "64px",
+          width: `${size}px`,
+          height: `${size}px`,
           touchAction: "none",
           userSelect: "none",
           position: "absolute"
@@ -125,4 +136,6 @@ function ProxyToken({ tokenClassName, onProxyDragEnd }) {
   );
 }
 
+ProxyToken.defaultProps = { size: 48 };
+
 export default ProxyToken;
diff --git a/src/components/Token.js b/src/components/Token.js
index 317efb8..6b74f37 100644
--- a/src/components/Token.js
+++ b/src/components/Token.js
@@ -4,18 +4,7 @@ import { Image } from "theme-ui";
 function Token({ image, className, tokenId }) {
   // Store the token id in the html element for the drag code to pick it up
   const idProp = tokenId ? { "data-token-id": tokenId } : {};
-  return (
-    <Image
-      p={2}
-      className={className}
-      src={image}
-      sx={{
-        width: "64px",
-        height: "64px"
-      }}
-      {...idProp}
-    />
-  );
+  return <Image className={className} src={image} {...idProp} />;
 }
 
 export default Token;
diff --git a/src/components/Tokens.js b/src/components/Tokens.js
index 6fb8682..fe936e2 100644
--- a/src/components/Tokens.js
+++ b/src/components/Tokens.js
@@ -1,5 +1,5 @@
 import React from "react";
-import { Flex } from "theme-ui";
+import { Flex, Box } from "theme-ui";
 import shortid from "shortid";
 
 import * as tokens from "../tokens";
@@ -30,7 +30,9 @@ function Tokens({ onCreateMapToken }) {
         px={2}
       >
         {Object.entries(tokens).map(([id, image]) => (
-          <Token key={id} image={image} className={listTokenClassName} />
+          <Box key={id} m={2} sx={{ width: "48px", height: "48px" }}>
+            <Token image={image} className={listTokenClassName} />
+          </Box>
         ))}
       </Flex>
       <ProxyToken
diff --git a/src/routes/Game.js b/src/routes/Game.js
index effaf21..e5b3e8a 100644
--- a/src/routes/Game.js
+++ b/src/routes/Game.js
@@ -97,6 +97,19 @@ function Game() {
 
   return (
     <Flex sx={{ flexDirection: "column", height: "100vh" }}>
+      <Flex
+        sx={{ justifyContent: "space-between", flexGrow: 1, height: "100%" }}
+      >
+        <Party streams={streams} localStreamId={peerId} />
+        <Map
+          mapSource={mapSource}
+          mapData={mapDataRef.current}
+          tokens={mapTokens}
+          onMapTokenMove={handleEditMapToken}
+          onMapTokenRemove={handleRemoveMapToken}
+        />
+        <Tokens onCreateMapToken={handleEditMapToken} />
+      </Flex>
       <Box
         p={2}
         sx={{
@@ -108,18 +121,6 @@ function Game() {
       >
         <AddMapButton handleMapChange={handleMapChange} />
       </Box>
-      <Flex
-        sx={{ justifyContent: "space-between", flexGrow: 1, height: "100%" }}
-      >
-        <Party streams={streams} localStreamId={peerId} />
-        <Map
-          imageSource={mapSource}
-          tokens={mapTokens}
-          onMapTokenMove={handleEditMapToken}
-          onMapTokenRemove={handleRemoveMapToken}
-        />
-        <Tokens onCreateMapToken={handleEditMapToken} />
-      </Flex>
     </Flex>
   );
 }