[no ci] prepare new Version

Reviewed-on: #42

README.md

@@ -42,6 +42,8 @@ services:
     restart: always
     ports:
       - "8080:8080"
+    environment:
+      DICE_LIMIT: 30    # OPTIONAL: amount of dice allowed to roll (default: 30)
 ```
 Run the container with:
 ```bash

pom.xml

@@ -4,7 +4,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>de.arindy</groupId>
     <artifactId>dice-tower</artifactId>
-    <version>1.2.1</version>
+    <version>1.2.8-SNAPSHOT</version>
 
     <properties>
         <compiler-plugin.version>3.13.0</compiler-plugin.version>

src/main/kotlin/de/arindy/dicetower/ChatOverlayResource.kt

@@ -21,14 +21,16 @@ class ChatOverlayResource {
         @QueryParam("modsAllowed") modsAllowed: Boolean = false,
         @QueryParam("vipAllowed") vipAllowed: Boolean = false,
         @QueryParam("subsAllowed") subsAllowed: Boolean = false,
+        @QueryParam("allAllowed") allAllowed: Boolean = false,
         @QueryParam("cmd") cmd: String? = "roll",
         @QueryParam("theme") theme: String? = "default",
         @QueryParam("faceColor") faceColor: String? = "#ff0202",
         @QueryParam("numberColor") numberColor: String? = "#ffffff",
         @QueryParam("clearAfter") clearAfter: Long? = -1,
-        @QueryParam("timeout") timeout: Long? = -1
+        @QueryParam("timeout") timeout: Long? = -1,
+        @QueryParam("showResults") showResults: Boolean = true
     ): TemplateInstance {
-        return Templates.chatoverlay(channel, scale ?: 7, maxDice ?: 20, modsAllowed, vipAllowed, subsAllowed, cmd ?: "roll", theme ?: "default", faceColor ?: "#ff0202", numberColor ?: "#ffffff", clearAfter ?: 10, timeout ?: 60)
+        return Templates.chatoverlay(channel, scale ?: 7, maxDice ?: 20, modsAllowed, vipAllowed, subsAllowed, allAllowed, cmd ?: "roll", theme ?: "default", faceColor ?: "#ff0202", numberColor ?: "#ffffff", clearAfter ?: 10, timeout ?: 60, showResults)
     }
 
     @GET

src/main/kotlin/de/arindy/dicetower/DiceResource.kt

@@ -14,12 +14,19 @@ import jakarta.ws.rs.sse.OutboundSseEvent
 import jakarta.ws.rs.sse.Sse
 import jakarta.ws.rs.sse.SseBroadcaster
 import jakarta.ws.rs.sse.SseEventSink
+import org.eclipse.microprofile.config.inject.ConfigProperty
+import java.util.Optional
 import java.util.UUID
 
+private const val LIMIT = 30
+
 @Path("/dice/{id}")
 @ApplicationScoped
 final class DiceResource(@Context val sse: Sse) {
 
+    @ConfigProperty(name = "dice.limit")
+    private lateinit var diceLimit: Optional<Int>
+
     private var eventBuilder: OutboundSseEvent.Builder = sse.newEventBuilder()
     private var sseBroadcasters: MutableMap<String, SseBroadcaster> = HashMap()
 
@@ -28,18 +35,31 @@ final class DiceResource(@Context val sse: Sse) {
     fun parseCommand(@PathParam("id") id: String, data: RollPayload) {
         data.room = id.split(":")[0]
         data.user = id.split(":")[1]
-
         val results = ArrayList<Results>()
+        if (data.results == null) {
+            var numberOfDice = 0
             data.command.split(" ", "&", "and").filter { it.isNotEmpty() }.map { it.trim() }.toTypedArray<String>().forEach { command ->
                 val dice = command.split("d")
-            val result = IntArray(dice[0].toInt())
+                var amount = dice[0].toInt()
+                val limit = diceLimit.orElse(LIMIT)
+                if (limit < numberOfDice + amount) {
+                    amount = limit - numberOfDice
+                }
+                numberOfDice += amount
+                if (amount > 0) {
+                    val result = IntArray(amount)
                     val sides = dice[1].split("+", "-")
                     val modifier = if (dice[1].contains("+")) sides[1].toInt() else if (dice[1].contains("+")) -1 * sides[1].toInt() else 0
-            repeat(dice[0].toInt()) { index ->
+                    repeat(amount) { index ->
                         result[index] = (Math.random() * sides[0].toInt() + 1).toInt()
                     }
                     results.add(Results(sides[0].toInt(), modifier, result.sum() + modifier, result.map { Roll(it) }.toTypedArray()))
                 }
+            }
+        } else {
+            results.addAll(data.results)
+        }
+
         val map = results.map { r ->
             "${r.rolls.size}d${r.sides}@${
                 if (r.sides == 100) r.rolls.map { roll -> Roll(roll.value / 10 * 10) }
@@ -48,13 +68,13 @@ final class DiceResource(@Context val sse: Sse) {
         }.toTypedArray()
 
         data.roll = map + results.filter { it.sides == 100 }.map { r -> "${r.rolls.size}d10@${r.rolls.map { roll -> Roll(roll.value % 10) }.joinToString(",")}"}.toTypedArray()
+        if (data.roll.all { it.trim().isNotEmpty() }) {
+            results(data.room!!, Result(data.name, data.user!!, data.faceColor, null))
+        }
         sseBroadcasters[id]?.broadcast(
             eventBuilder.id((UUID.randomUUID()).toString())
                 .mediaType(MediaType.APPLICATION_JSON_TYPE).data(data).build()
         )
-        if (data.roll.all { it.trim().isNotEmpty() }) {
-            results(data.room!!, Result(data.name, data.user!!, data.faceColor, null))
-        }
         Thread.sleep(1000)
         results(data.room!!, Result(data.name, data.user!!, data.faceColor, results.toTypedArray()))
     }

src/main/kotlin/de/arindy/dicetower/OverlayResource.kt

@@ -20,7 +20,7 @@ class OverlayResource {
     @GET
     @Path("/results")
     @Produces(MediaType.TEXT_HTML)
-    fun results(@PathParam("diceid") room: String, @QueryParam("name") name: String?, @QueryParam("user") user: String?): TemplateInstance {
-        return Templates.results(room, name ?: "all", user ?: "all")
+    fun results(@PathParam("diceid") room: String, @QueryParam("name") name: String?): TemplateInstance {
+        return Templates.results(room, name ?: "all")
     }
 }

src/main/kotlin/de/arindy/dicetower/RollPayload.kt

@@ -6,7 +6,7 @@ import io.quarkus.runtime.annotations.RegisterForReflection
 data class RollPayload(
     val name: String,
     var command: String,
-    var predetermined: Boolean? = false,
+    val results: Array<DiceResource.Results>?,
     val faceColor: String = "white",
     val numberColor: String = "white",
     val theme: String = "default",

src/main/kotlin/de/arindy/dicetower/Templates.kt

@@ -9,7 +9,7 @@ object Templates {
     external fun overlay(diceid: String, scale: Int?, clearAfter: Long?): TemplateInstance
 
     @JvmStatic
-    external fun results(room: String, name: String?, user: String?): TemplateInstance
+    external fun results(room: String, name: String?): TemplateInstance
 
     @JvmStatic
     external fun index(version: String): TemplateInstance
@@ -25,11 +25,13 @@ object Templates {
         modsAllowed: Boolean,
         vipAllowed: Boolean,
         subsAllowed: Boolean,
+        allAllowed: Boolean,
         cmd: String?,
         theme: String?,
         faceColor: String?,
         numberColor: String?,
         clearAfter: Long?,
-        timeout: Long?
+        timeout: Long?,
+        showResults: Boolean?
     ): TemplateInstance
 }

src/main/resources/META-INF/resources/app.js

@@ -14,6 +14,17 @@ function url() {
     return window.location.protocol + '//' + window.location.hostname + (window.location.port?.length > 0 ? ':' + window.location.port : '');
 }
 
+function register() {
+    let httpRequest = new XMLHttpRequest();
+    httpRequest.open('POST', url() + '/dice/' + document.getElementById('room').value + '/register')
+    httpRequest.setRequestHeader('Content-Type', 'application/json')
+    httpRequest.send(JSON.stringify({
+        name: document.getElementById('name').value,
+        overlay: document.getElementById('overlayId').value,
+        id: document.getElementById('room').value + ':' + localStorage.getItem('userId')
+    }))
+}
+
 function start(event = undefined) {
     if ((!event || event.keyCode === 13) && document.getElementById('name').value.length > 0 && document.getElementById('room').value.length > 0) {
         document.getElementById('overlayId').value = url() + '/overlay/' + document.getElementById('room').value + ':' + localStorage.getItem('userId') + '?scale=10&clearAfter=30';
@@ -61,14 +72,7 @@ function start(event = undefined) {
 
         localStorage.setItem(document.getElementById('name').value + '-started', "true")
 
-        let httpRequest = new XMLHttpRequest();
-        httpRequest.open('POST', url() + '/dice/' + document.getElementById('room').value + '/register')
-        httpRequest.setRequestHeader('Content-Type', 'application/json')
-        httpRequest.send(JSON.stringify({
-            name: document.getElementById('name').value,
-            overlay: document.getElementById('overlayId').value,
-            id: document.getElementById('room').value + ':' + localStorage.getItem('userId')
-        }))
+        register();
         if (document.getElementById('gm').checked) {
             document.getElementById('resultSwitch').checked = true;
             document.getElementById('resultFrame').src = document.getElementById('resultsId').value;
@@ -98,6 +102,7 @@ function start(event = undefined) {
                     newOverlay.appendChild(newInput);
                     overlays.appendChild(newOverlay);
 
+                    if (!document.getElementById(data.id + '-diceFrame')) {
                         let dice = document.createElement('iframe');
                         dice.id = data.id + '-diceFrame'
                         dice.style.width = "50%";
@@ -110,6 +115,8 @@ function start(event = undefined) {
                         dice.style.left = "50%";
                         dice.src = data.overlay;
                         document.getElementById('results-dice').appendChild(dice)
+
+                    }
                 }
 
                 configurePopover();
@@ -125,6 +132,7 @@ function rollEasy(dice) {
 
 function roll(event) {
     if ((!event || event.keyCode === 13) && document.getElementById('command').value?.length > 0) {
+        register()
         let httpRequest = new XMLHttpRequest();
         httpRequest.open('POST', url() + '/dice/' + document.getElementById('room').value + ':' + localStorage.getItem(`userId`))
         httpRequest.setRequestHeader('Content-Type', 'application/json')
@@ -207,11 +215,7 @@ document.addEventListener("DOMContentLoaded", async () => {
     })
 
     document.getElementById('resultDiceSwitch').addEventListener('change', function () {
-        if (!this.checked) {
-            document.getElementById('results-dice').hidden = true
-        } else {
-            document.getElementById('results-dice').hidden = false
-        }
+        document.getElementById('results-dice').hidden = !this.checked;
     })
 
     document.getElementById('chatOverlayLink').href = url() + '/chatoverlay'

src/main/resources/META-INF/resources/manifest.json

@@ -0,0 +1,18 @@
+{
+  "name": "Dice-Tower",
+  "icons": [
+    {
+      "src": "192.png",
+      "type": "image/png",
+      "sizes": "192x192"
+    },
+    {
+      "src": "512.png",
+      "type": "image/png",
+      "sizes": "512x512"
+    }
+  ],
+  "start_url": "/",
+  "display_override": ["window-controls-overlay", "minimal-ui"],
+  "display": "standalone"
+}

src/main/resources/META-INF/resources/style.css

@@ -32,6 +32,22 @@ button:active {
     background: #222222;
 }
 
+#install {
+    padding: 10px;
+    border: #666666 3px solid;
+    border-radius: 10px;
+    background: #666666;
+    color: #fff
+}
+
+#install:hover {
+    background: #444444;
+}
+
+#install:active {
+    background: #222222;
+}
+
 input {
     margin: 10px;
 }

src/main/resources/META-INF/resources/vendor/color-picker.js

@@ -123,15 +123,15 @@ class ColorPicker extends HTMLElement {
 				<div class="bg-cover" id="controls">
 					<label for="hue">
 						<abbr title="Hue: The color tint.">H</abbr>
-						<input type="range" id="hue" min="0" max="360" value="0">
+						<input type="range" id="hue" min="0" max="360" value="0" step="0.01">
 					</label>
 					<label for="saturation">
 						<abbr title="Saturation: The amount of gray in the color.">S</abbr>
-						<input type="range" id="saturation" min="0" max="100" value="0">
+						<input type="range" id="saturation" min="0" max="100" value="0" step="0.01">
 					</label>
 					<label for="lightness">
 						<abbr title="Lightness: How light or dark the color is.">L</abbr>
-						<input type="range" id="lightness" min="0" max="100" value="0">
+						<input type="range" id="lightness" min="0" max="100" value="0" step="0.01">
 					</label>
 				</div>
 			</div>`;
@@ -251,9 +251,9 @@ class ColorPicker extends HTMLElement {
      */
     updateColor() {
 
-        const hue = Math.round( this.hueInput.value );
-        const saturation = Math.round( this.saturationInput.value );
-        const lightness = Math.round( this.lightnessInput.value );
+        const hue = parseFloat( this.hueInput.value );
+        const saturation = parseFloat( this.saturationInput.value );
+        const lightness = parseFloat( this.lightnessInput.value );
         const hsla = [ hue, saturation, lightness ];
 
         const rgba = this.HSLAToRGBA( hsla );
@@ -324,6 +324,10 @@ class ColorPicker extends HTMLElement {
      */
     setColor( color = 'hsla(60, 100%, 50%, 0.5)' ) {
 
+        if ( /^[0-9A-F]{3,6}$/i.test( color ) ) {
+            color = '#' + color;
+        }
+
         const rgba = this.colorToRGBA( color );
         const hsla = this.rgbToHSLA( rgba );
 
@@ -484,7 +488,8 @@ class ColorPicker extends HTMLElement {
         }
 
         // Calculate hue
-        let hue;
+        let hue = 0;
+        if ( max !== min ) {
             if ( max === normalizedR ) {
                 hue = ( ( normalizedG - normalizedB ) / ( max - min ) ) % 6;
             } else if ( max === normalizedG ) {
@@ -492,6 +497,7 @@ class ColorPicker extends HTMLElement {
             } else {
                 hue = ( ( normalizedR - normalizedG ) / ( max - min ) + 4 );
             }
+        }
 
         // Convert hue to degrees
         hue *= 60;
@@ -500,7 +506,7 @@ class ColorPicker extends HTMLElement {
             hue += 360;
         }
 
-        return [ Math.round( hue ), Math.round( saturation * 100 ), Math.round( lightness * 100 ), a ];
+        return [ parseFloat(hue.toFixed(2)), parseFloat((saturation * 100).toFixed(2)), parseFloat((lightness * 100).toFixed(2)), a ];
 
     }
 

src/main/resources/templates/chatoverlay.html

@@ -31,9 +31,7 @@
     //maxDice
 
     ComfyJS.onCommand = async (user, command, message, flags) => {
-        if ((
-                flags.broadcaster || ({modsAllowed} && flags.mod) || ({vipAllowed} && flags.vip) || ({subsAllowed} && flags.subscriber
-                )) && '{cmd}' === command && !shouldWait() && message.match(/^\d+d(4|6|8|10|12|20|100)$/) && (+message.split('d')[0] <= {maxDice})
+        if ((flags.broadcaster || {allAllowed} || ({modsAllowed} && flags.mod) || ({vipAllowed} && flags.vip) || ({subsAllowed} && flags.subscriber)) && '{cmd}' === command && !shouldWait() && message.match(/^\d+d(4|6|8|10|12|20|100)$/) && (+message.split('d')[0] <= {maxDice})
         ) {
             toggleWait(true);
 
@@ -45,7 +43,9 @@
                     })
                     document.getElementById('results').innerHTML = '<strong>' + user + '</strong> rolls <strong>' + message + '</strong>:<br/> [' + values.map(value => value === 1 ? '<strong style="text-shadow: 2px 2px 10px red">' + value + '</strong>' : value === result.sides ? '<strong style="text-shadow: 2px 2px 10px green">' + value + '</strong>' : value).join(' + ') + '] = <strong>' + result.total + '</strong> '
                 })
+                if({showResults}) {
                     document.getElementById('results').showPopover()
+                }
                 setTimeout(() => {
                     diceBox.clearDice();
                     document.getElementById('results').hidePopover()

src/main/resources/templates/chatoverlayconfig.html

@@ -52,6 +52,10 @@
                 <input type="checkbox" id="subsAllowed">
                 <span class="checkmark"></span>
             </label>
+            <label class="checkbox" id="allAllowed-container">Allow everyone to roll (yes, everyone!!)
+                <input type="checkbox" id="allAllowed">
+                <span class="checkmark"></span>
+            </label>
         </div>
         <div>
             <label for="scale">Dice-Scale </label>
@@ -64,6 +68,10 @@
             <input type="number" id="clearAfter" style="width: 50px; margin-top: 20px" value="10"/>
             <label for="timeout">Command-timeout (in seconds)</label>
             <input type="number" id="timeout" style="width: 50px; margin-top: 20px" value="60"/>
+            <label class="checkbox" id="showResults-container">Show Results Overlay
+                <input type="checkbox" id="showResults" checked>
+                <span class="checkmark"></span>
+            </label>
         </div>
 
         <div id="dice-box" style="width: 850px; height: 400px">
@@ -244,7 +252,9 @@
                     "&timeout=" + document.getElementById('timeout').value +
                     "&modsAllowed=" + document.getElementById('modsAllowed').checked +
                     "&vipAllowed=" + document.getElementById('vipAllowed').checked +
-                    "&subsAllowed=" + document.getElementById('subsAllowed').checked
+                    "&subsAllowed=" + document.getElementById('subsAllowed').checked +
+                    "&allAllowed=" + document.getElementById('allAllowed').checked +
+                    "&showResults=" + document.getElementById('showResults').checked
         }
     })
 </script>

src/main/resources/templates/index.html

@@ -4,7 +4,7 @@
     <meta charset="UTF-8">
     <title>Dice-Tower</title>
     <meta name="version" content="{version}">
-
+    <link rel="manifest" href="manifest.json" />
     <link rel="stylesheet" href="/vendor/w3css/4/w3.css">
     <link rel="stylesheet" href="/vendor/font-awesome/css/fontawesome.css">
     <link rel="stylesheet" href="/vendor/font-awesome/css/all.css">
@@ -33,6 +33,17 @@
     if (!localStorage.getItem("userId")) {
         localStorage.setItem("userId", self.crypto.randomUUID());
     }
+    addEventListener("beforeinstallprompt", (event) => {
+        event.preventDefault()
+        let install = document.createElement('button');
+        install.id = 'install'
+        install.style.position = 'absolute'
+        install.style.top = '25px'
+        install.style.right = '25px'
+        install.innerHTML = 'Install Dice-Tower'
+        install.onclick = () => event.prompt()
+        document.body.appendChild(install)
+    });
 </script>
 <div class="w3-container w3-content"
      style="height: 95vh; display: flex; flex-direction: column; justify-content: space-between;  padding: 25px">
@@ -78,7 +89,7 @@
             <button style="border: transparent; border-radius: 100%; font-size: large; font-weight: bold; height: 50px; width: 50px"
                     onclick="addDice()">+
             </button>
-            <button style="font-size: large; font-weight: bold;" onclick="rollEasy('d2')">D2</button>
+            <!--<button style="font-size: large; font-weight: bold;" onclick="rollEasy('d2')">D2</button><-->
             <button style="font-size: large; font-weight: bold;" onclick="rollEasy('d4')">D4</button>
             <button style="font-size: large; font-weight: bold;" onclick="rollEasy('d6')">D6</button>
             <button style="font-size: large; font-weight: bold;" onclick="rollEasy('d8')">D8</button>
@@ -191,11 +202,8 @@
             <h2 style="text-align: center">How-To</h2>
             <ul>
                 <li>
-                    Join a room by entering your character name and the name of the room.<br/>
-                    <strong>If you are a GM, make sure to join the room first or let all other players rejoin to get all
-                        Overlay-URLs.</strong>
+                    Join a room by entering your character name and the name of the room
                 </li>
-
                 <li>Open your Dice-Overlay either in a new Tab or as a browser source in OBS</li>
                 <ul>
                     <li>You can configure your Overlay with query parameters (for more information hover over the link)

src/main/resources/templates/overlay.html

@@ -40,7 +40,7 @@
                 foreground: data.numberColor
             }
         }).then(() => {
-            diceBox.roll(data.roll.join('&'));
+            diceBox.roll(data.roll.filter(it => it.split('@')[0].split('d')[1] !== "2").join('&'));
         })
     })
 </script>

src/main/resources/templates/results.html

@@ -17,9 +17,9 @@
     const evtSource = new EventSource(url() + '/dice/{room}/results');
     evtSource.addEventListener('message', function (event) {
         let data = JSON.parse(event.data);
-        if ("{name}" === "all" && "{user}" === "all" || "{name}" === data.name && "{user}" === data.user || "{name}" === "all" && "{user}" === data.user || "{name}" === data.name && "{user}" === "all") {
-            let name = document.getElementById(data.user + '-' + data.name) ?? document.createElement('div');
-            name.id = data.user + '-' + data.name;
+        if ("{name}" === "all" || "{name}" === data.name) {
+            let name = document.getElementById(data.name) ?? document.createElement('div');
+            name.id = data.name;
             name.replaceChildren(...[]);
             let node = document.createElement('p');
             let resultText = ''