From 8b0a1630a14496a73f1db3b989bed1e187ef1600 Mon Sep 17 00:00:00 2001
From: LeRatierBretonnier <sylvain@lahiette.com>
Date: Sun, 29 Mar 2026 19:15:50 +0200
Subject: [PATCH] =?UTF-8?q?feat:=20remplace=20les=20inputs=20modificateur/?=
 =?UTF-8?q?aspect=20par=20des=20selects=20-4=20=C3=A0=20+4?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- roll.mjs : passe modifierChoices dans le contexte du dialogue
- roll-dialog.hbs : <select> pour modifier et aspectModifier (-4..+4)
- roll.less : select inclus dans la règle width:100% du form-two-col

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---
 module/documents/roll.mjs |  6 ++++++
 styles/roll.less          |  2 +-
 templates/roll-dialog.hbs | 12 ++++++++++--
 3 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/module/documents/roll.mjs b/module/documents/roll.mjs
index f8a0428..035b93c 100644
--- a/module/documents/roll.mjs
+++ b/module/documents/roll.mjs
@@ -42,6 +42,11 @@ export class CelestopolRoll extends Roll {
       ? game.i18n.localize(SYSTEM.WOUND_LEVELS[woundLevelId]?.label ?? "")
       : null
 
+    const modifierChoices = [-4, -3, -2, -1, 0, 1, 2, 3, 4].map(v => ({
+      value: v,
+      label: v > 0 ? `+${v}` : `${v}`,
+    }))
+
     const dialogContext = {
       actorName:          options.actorName,
       statLabel:          options.statLabel,
@@ -53,6 +58,7 @@ export class CelestopolRoll extends Roll {
       defaultDifficulty:  options.difficulty ?? "normal",
       destGaugeFull,
       defaultRollMoonDie: options.rollMoonDie ?? false,
+      modifierChoices,
     }
 
     const content = await foundry.applications.handlebars.renderTemplate(
diff --git a/styles/roll.less b/styles/roll.less
index f188421..4edd4f5 100644
--- a/styles/roll.less
+++ b/styles/roll.less
@@ -187,7 +187,7 @@
         text-align: left;
       }
 
-      input { width: 100%; box-sizing: border-box; }
+      input, select { width: 100%; box-sizing: border-box; }
     }
   }
 
diff --git a/templates/roll-dialog.hbs b/templates/roll-dialog.hbs
index 8e3c9a5..42fba9f 100644
--- a/templates/roll-dialog.hbs
+++ b/templates/roll-dialog.hbs
@@ -40,11 +40,19 @@
     <div class="form-two-col">
       <div class="form-row-line">
         <label for="modifier">{{localize "CELESTOPOL.Roll.modifier"}}</label>
-        <input type="number" id="modifier" name="modifier" value="0" min="-10" max="10">
+        <select id="modifier" name="modifier">
+          {{#each modifierChoices as |choice|}}
+          <option value="{{choice.value}}" {{#if (eq choice.value 0)}}selected{{/if}}>{{choice.label}}</option>
+          {{/each}}
+        </select>
       </div>
       <div class="form-row-line">
         <label for="aspectModifier">{{localize "CELESTOPOL.Roll.aspect"}}</label>
-        <input type="number" id="aspectModifier" name="aspectModifier" value="0" min="-4" max="4">
+        <select id="aspectModifier" name="aspectModifier">
+          {{#each modifierChoices as |choice|}}
+          <option value="{{choice.value}}" {{#if (eq choice.value 0)}}selected{{/if}}>{{choice.label}}</option>
+          {{/each}}
+        </select>
       </div>
     </div>