From 8a1a09b150a61d70d1b15786b68c9bfc6ee6c36a Mon Sep 17 00:00:00 2001
From: gsinghpal <gur_preet.singh@outlook.com>
Date: Tue, 2 Jun 2026 01:00:06 -0400
Subject: [PATCH] =?UTF-8?q?fix(fusion=5Fplating=5Fshopfloor):=20scan=20but?=
 =?UTF-8?q?tons=20=E2=80=94=20single=20icon=20+=20clearer=20labels?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The QrScanner component renders its own fa-qrcode icon; the board passed
it label '📷 Camera', so the camera button showed two icons (QR + camera
emoji). Drop the emoji → one icon.

Also clarify the two scan paths (they do different things):
- "Scan QR"  = camera scan of the printed job sticker (primary path)
- "Enter Code" = manual / hardware scanner-gun text drawer (no camera)
Reordered so the camera (sticker) scan reads first. Other QrScanner call
sites already pass plain/no labels — this was the only double-icon.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../static/src/xml/plant_kanban.xml               | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/fusion_plating/fusion_plating_shopfloor/static/src/xml/plant_kanban.xml b/fusion_plating/fusion_plating_shopfloor/static/src/xml/plant_kanban.xml
index 58d25027..413fe49d 100644
--- a/fusion_plating/fusion_plating_shopfloor/static/src/xml/plant_kanban.xml
+++ b/fusion_plating/fusion_plating_shopfloor/static/src/xml/plant_kanban.xml
@@ -23,13 +23,18 @@
                                     <button t-att-class="modeClass('manager')"
                                             t-on-click="() => this.setMode('manager')">Manager</button>
                                 </div>
-                                <!-- Text/wedge scan drawer toggle. Camera path
-                                     is the QrScanner inline below — it
-                                     opens its own modal + decoder. -->
+                                <!-- "Scan QR" = the QrScanner camera path (the
+                                     primary way to scan a printed job sticker).
+                                     The component renders its own fa-qrcode
+                                     icon, so the label must be plain text — an
+                                     emoji here would double up the icon.
+                                     "Enter Code" = the manual / hardware-scanner-
+                                     gun text drawer (a wedge gun types the code;
+                                     no camera). -->
+                                <QrScanner cssClass="'toolbar-btn'" label="'Scan QR'"/>
                                 <button class="toolbar-btn"
                                         t-att-class="state.showScan ? 'toolbar-btn active' : 'toolbar-btn'"
-                                        t-on-click="toggleScan">⌨️ Scan Code</button>
-                                <QrScanner cssClass="'toolbar-btn'" label="'📷 Camera'"/>
+                                        t-on-click="toggleScan">⌨️ Enter Code</button>
                                 <button class="toolbar-btn handoff" t-on-click="onHandOff">🔓 Hand Off</button>
                             </div>
                         </div>