From bd7df724c40ff6e1c6c137937c659fd6969f629d Mon Sep 17 00:00:00 2001
From: Oliver-Akins <Oliver.akins1999@gmail.com>
Date: Sat, 16 Dec 2023 01:30:57 -0700
Subject: [PATCH] Improve a bit of UX on the sheet to do with the skill
 dropdown

---
 .../actors/char-sheet-mvp/partials/skill.hbs  | 31 +++++++++++++------
 1 file changed, 22 insertions(+), 9 deletions(-)

diff --git a/templates/actors/char-sheet-mvp/partials/skill.hbs b/templates/actors/char-sheet-mvp/partials/skill.hbs
index d853e4b..801ab05 100644
--- a/templates/actors/char-sheet-mvp/partials/skill.hbs
+++ b/templates/actors/char-sheet-mvp/partials/skill.hbs
@@ -1,11 +1,24 @@
 <div class="skill">
-	<span>
-		{{localize (concat "dotdungeon.skill." name)}}
-	</span>
-	<select aria-label="{{localize "dotdungeon.aria.skill-dropdown" skill=name}}">
-		<option value="">Novice</option>
-		<option value="locked">Locked</option>
-		<option value="+2">Trained</option>
-		<option value="+4">Expert</option>
-	</select>
+	<button class="skill__button" data-stat="{{stat}}" data-skill="{{name}}">
+		{{!--
+		This would be nice to get working at some point, but it's seeming not
+		super plausible at the moment due to context weirdness (probably because
+		of my "with" helper)
+
+		<img src="icons/dice/{{dotdungeon-objectValue system (concat "system.skills." stat "." name)}}black.svg">
+		--}}
+		R
+	</button>
+	<label class="skill__label">
+		{{localize (concat "dotdungeon.actor.pc.skill." name)}}
+		<select
+			class="skill__select"
+			aria-label="{{localize "dotdungeon.aria.skill-dropdown" skill=name}}"
+		>
+			<option value="">Novice (+0)</option>
+			<option value="locked">Locked</option>
+			<option value="+2">Trained (+2)</option>
+			<option value="+4">Expert (+4)</option>
+		</select>
+	</label>
 </div>
\ No newline at end of file