From: Raphaël Jakse <raphael.trivabble@jakse.fr>
Date: Sun, 19 Apr 2020 10:43:24 +0000 (+0200)
Subject: Correctly handle jockers and missing tiles when showing the rack
X-Git-Url: https://secure.softndesign.org/git/?a=commitdiff_plain;h=cb045696938df15788755bac23cd514362b49962;p=trivabble.git

Correctly handle jockers and missing tiles when showing the rack
---

diff --git a/trivabble.css b/trivabble.css
index 7feeaae..7e994e9 100644
--- a/trivabble.css
+++ b/trivabble.css
@@ -219,6 +219,7 @@ button:hover {
     width: 1.5em;
     height: 1.5em;
     margin:2px;
+    vertical-align:bottom;
 }
 
 #chat-messages .tile-letter {
diff --git a/trivabble.js b/trivabble.js
index 909422b..6adeac5 100644
--- a/trivabble.js
+++ b/trivabble.js
@@ -1249,18 +1249,32 @@
         myConfirm(
             _("Are you sure you want to show your rack to everybody?"),
             function () {
+                const letters = playerLetters.map(
+                    function (tilePlaceholer) {
+                        return tilePlaceholer.querySelector(".tile");
+                    }
+                ).filter(function (tile) {
+                    return tile !== null;
+                });
+
                 sendCmds([{
                     cmd: "msg",
-                    msg: _("Here is my rack:") + " " + playerLetters.map(
-                        (tile) => (
-                            tile.textContent === ""
-                                ? _("joker")
-                                : tile.querySelector(".tile-letter").textContent
-                        )
+                    msg: _("Here is my rack:") + " " + letters.map(
+                        function (tile) {
+                            return (
+                                tile.textContent === ""
+                                    ? _("joker")
+                                    : tile.querySelector(".tile-letter").textContent
+                            );
+                        }
                     ).join(", "),
                     specialMsg: {
                         type: "rack",
-                        rack: playerLetters.map((tile) => tile.querySelector(".tile-letter").textContent)
+                        rack: letters.map(
+                            function (tile) {
+                                return tile.querySelector(".tile-letter").textContent
+                            }
+                        )
                     }
                 }]);
             }