From: Carl Worth <cworth@cworth.org>
Date: Sat, 6 Jun 2020 19:30:23 +0000 (-0700)
Subject: Add a button to copy the game URL for sharing and tighten up text
X-Git-Url: https://git.cworth.org/git?p=lmno.games;a=commitdiff_plain;h=656345d535b978e92cd9da0801a28f4571aaa6f2;hp=6c0ef34689d35ec9deada19903f7e307f5ac618a

Add a button to copy the game URL for sharing and tighten up text

The way we are styling <h2> right now makes it far too wasteful of
vertical screen real estate. Instead, use the same font size/weight
but inline with the sentence.

Also, this button for copying the game URL should be handy,
particularly on phones, etc.
---

diff --git a/scribe/scribe.jsx b/scribe/scribe.jsx
index 06a8541..be0de37 100644
--- a/scribe/scribe.jsx
+++ b/scribe/scribe.jsx
@@ -79,14 +79,31 @@ events.addEventListener("game-state", event => {
  * Game and supporting classes                           *
  *********************************************************/
 
+function copy_to_clipboard(id)
+{
+  const tmp = document.createElement("input");
+  tmp.setAttribute("value", document.getElementById(id).innerHTML);
+  document.body.appendChild(tmp);
+  tmp.select();
+  document.execCommand("copy");
+  document.body.removeChild(tmp);
+}
+
 function GameInfo(props) {
   if (! props.id)
     return null;
 
   return (
     <div className="game-info">
-      <h2>{props.id}</h2>
-      Invite a friend to play by sending this URL: {props.url}
+      <span className="game-id">{props.id}</span>
+      {" "}
+      Share this link to invite a friend:{" "}
+      <span id="game-share-url">{props.url}</span>
+      {" "}
+      <button
+        className="inline"
+        onClick={() => copy_to_clipboard('game-share-url')}
+      >Copy Link</button>
     </div>
   );
 }
@@ -132,7 +149,7 @@ function PlayerInfo(props) {
 
   return (
     <div className="player-info">
-      <h2>Players</h2>
+      <span className="players-header">Players: </span>
       {props.player.name}
       {props.player.team ? ` (${props.player.team})` : ""}
       {props.first_move ? "" : " "}
diff --git a/style.css b/style.css
index 91cee83..6d5bdca 100644
--- a/style.css
+++ b/style.css
@@ -319,3 +319,14 @@ button:hover {
 .hide-button:hover {
     color: var(--danger-color-dark);
 }
+
+/*\
+|*|
+|*| Game-specific markup
+|*|
+\*/
+
+.game-id, .players-header {
+    font-size: 110%;
+    font-weight: bold;
+}