From 179a994f69a4ad74a149c285a6863394136e1815 Mon Sep 17 00:00:00 2001
From: bach <bachir@figureslibres.io>
Date: Sun, 6 Sep 2026 10:52:23 +0200
Subject: [PATCH] =?UTF-8?q?Fix=20empty=20Facture/Libell=C3=A9=20cells=20ha?=
 =?UTF-8?q?ving=20nothing=20to=20click?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Facture and libellé rendered their span with no fallback text when
empty, unlike Client's existing "|| '—'" -- an empty span has zero
width/height, so there was nothing there to click at all. Added the
same "—" placeholder to both.

Also switched .figli-editable-cell to display: block with a min-height,
so the clickable area fills the whole cell's content box instead of
just hugging the text -- a lone "—" character was still a tiny target
even with a placeholder.
---
 web/modules/custom/figli_compta_ledger/css/home.css          | 5 +++++
 .../templates/figli-compta-home.html.twig                    | 4 ++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/web/modules/custom/figli_compta_ledger/css/home.css b/web/modules/custom/figli_compta_ledger/css/home.css
index 9825797..7c61261 100644
--- a/web/modules/custom/figli_compta_ledger/css/home.css
+++ b/web/modules/custom/figli_compta_ledger/css/home.css
@@ -327,7 +327,12 @@ html.gin--dark-mode #figli-home-app {
    plain text rather than a pill -- a dotted underline is enough of an
    affordance without implying a fixed set of choices the way the type
    badge's pill shape does. */
+/* display: block + min-height (not just the default inline text-only
+   box) so the whole cell is a forgiving click target even when empty --
+   the "—" placeholder alone is a tiny, easy-to-miss target otherwise. */
 #figli-home-app .figli-editable-cell {
+  display: block;
+  min-height: 1.2em;
   cursor: pointer;
   border-bottom: 1px dotted transparent;
 }
diff --git a/web/modules/custom/figli_compta_ledger/templates/figli-compta-home.html.twig b/web/modules/custom/figli_compta_ledger/templates/figli-compta-home.html.twig
index 74be3e7..3f31e9f 100644
--- a/web/modules/custom/figli_compta_ledger/templates/figli-compta-home.html.twig
+++ b/web/modules/custom/figli_compta_ledger/templates/figli-compta-home.html.twig
@@ -185,7 +185,7 @@
                 @blur="editingCell = null"
                 @keyup.enter="$event.target.blur()"
               />
-              <span v-else class="figli-editable-cell" title="Cliquer pour modifier" @click="startEditCell(item, 'facture')">{{ item.facture }}</span>
+              <span v-else class="figli-editable-cell" title="Cliquer pour modifier" @click="startEditCell(item, 'facture')">{{ item.facture || '—' }}</span>
             </td>
             <td class="figli-libelle">
               <input
@@ -198,7 +198,7 @@
                 @keyup.enter="$event.target.blur()"
               />
               <template v-else>
-                <span class="figli-editable-cell" title="Cliquer pour modifier" @click="startEditCell(item, 'libelle')">{{ item.libelle }}</span>
+                <span class="figli-editable-cell" title="Cliquer pour modifier" @click="startEditCell(item, 'libelle')">{{ item.libelle || '—' }}</span>
                 <span
                   v-if="item.type === 'entree' && reconciliationByEntree.get(item.id) && reconciliationByEntree.get(item.id).count > 0"
                   class="figli-recon-badge is-clickable"