From 30a1141997b638813a92281a68fa4aa6be5515a5 Mon Sep 17 00:00:00 2001
From: gsinghpal <gur_preet.singh@outlook.com>
Date: Tue, 12 May 2026 01:01:50 -0400
Subject: [PATCH] ui+fix(jobs): compact row buttons + remove racking inspection
 gate
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

UI: secondary row buttons in the embedded step list are now icon-only
with tooltips (Pause / Complete 1 → Next / Record / Skip / Move…).
Saves ~70% horizontal space. "Finish & Next" stays text+icon as the
primary action.

Fix: removed the racking-inspection gate from button_finish. Racking
is now a recipe step (not a separate inspection workflow), so the
"Racking inspection for ... is Inspecting — must be Done" error no
longer fires. _fp_check_racking_inspection_complete() helper is
preserved for diagnostics but no longer called from the finish path.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 fusion_plating/fusion_plating_jobs/__manifest__.py   |  2 +-
 .../fusion_plating_jobs/models/fp_job_step.py        |  6 ++++--
 .../views/fp_job_form_inherit.xml                    | 12 ++++++------
 3 files changed, 11 insertions(+), 9 deletions(-)

diff --git a/fusion_plating/fusion_plating_jobs/__manifest__.py b/fusion_plating/fusion_plating_jobs/__manifest__.py
index d6fffc88..6f34b761 100644
--- a/fusion_plating/fusion_plating_jobs/__manifest__.py
+++ b/fusion_plating/fusion_plating_jobs/__manifest__.py
@@ -3,7 +3,7 @@
 # License OPL-1 (Odoo Proprietary License v1.0)
 {
     'name': 'Fusion Plating — Native Jobs',
-    'version': '19.0.8.20.7',
+    'version': '19.0.8.20.8',
     'category': 'Manufacturing/Plating',
     'summary': 'Native plating job model — replaces mrp.production / mrp.workorder bridge.',
     'author': 'Nexa Systems Inc.',
diff --git a/fusion_plating/fusion_plating_jobs/models/fp_job_step.py b/fusion_plating/fusion_plating_jobs/models/fp_job_step.py
index ac78eb7a..13548f10 100644
--- a/fusion_plating/fusion_plating_jobs/models/fp_job_step.py
+++ b/fusion_plating/fusion_plating_jobs/models/fp_job_step.py
@@ -910,8 +910,10 @@ class FpJobStep(models.Model):
     def button_finish(self):
         # Policy B — block until QA-005 complete (when customer requires it).
         self._fp_check_contract_review_complete()
-        # Sub 8 — block until racking inspection is Done / Flagged.
-        self._fp_check_racking_inspection_complete()
+        # NOTE: racking inspection gate removed — racking is now a recipe
+        # step, not a separate inspection workflow. _fp_check_racking_
+        # inspection_complete() is kept as a helper for diagnostics but
+        # no longer enforced from button_finish.
         result = super().button_finish()
         for step in self:
             if step.state == 'done':
diff --git a/fusion_plating/fusion_plating_jobs/views/fp_job_form_inherit.xml b/fusion_plating/fusion_plating_jobs/views/fp_job_form_inherit.xml
index 3ed79544..97301d34 100644
--- a/fusion_plating/fusion_plating_jobs/views/fp_job_form_inherit.xml
+++ b/fusion_plating/fusion_plating_jobs/views/fp_job_form_inherit.xml
@@ -150,7 +150,7 @@
                                 class="btn-link text-success"
                                 invisible="state not in ('ready', 'pending')"/>
                         <button name="button_resume" type="object"
-                                string="Resume" icon="fa-play-circle"
+                                title="Resume" icon="fa-play-circle"
                                 class="btn-link text-success"
                                 invisible="state != 'paused'"/>
                         <button name="action_finish_and_advance" type="object"
@@ -164,7 +164,7 @@
                              measurements without finishing the step;
                              Skip + Move (cross-station) tucked together. -->
                         <button name="button_pause" type="object"
-                                string="Pause" icon="fa-pause"
+                                title="Pause" icon="fa-pause"
                                 class="btn-link text-warning"
                                 invisible="state != 'in_progress'"/>
                         <!-- Streaming flow: complete 1 part at a time,
@@ -172,19 +172,19 @@
                              parked or the step isn't actively running.
                              Auto-finishes when qty_at_step drains to 0. -->
                         <button name="action_complete_one_to_next" type="object"
-                                string="Complete 1 → Next" icon="fa-forward"
+                                title="Complete 1 → Next (streaming flow)" icon="fa-forward"
                                 class="btn-link text-success"
                                 invisible="state != 'in_progress' or qty_at_step &lt; 1"/>
                         <button name="action_open_input_wizard" type="object"
-                                string="Record" icon="fa-pencil-square-o"
+                                title="Record Inputs" icon="fa-pencil-square-o"
                                 class="btn-link"
                                 invisible="state in ('cancelled', 'skipped')"/>
                         <button name="button_skip" type="object"
-                                string="Skip" icon="fa-step-forward"
+                                title="Skip step" icon="fa-step-forward"
                                 class="btn-link text-muted"
                                 invisible="state not in ('pending', 'ready')"/>
                         <button name="action_open_move_wizard" type="object"
-                                string="Move…" icon="fa-exchange"
+                                title="Move… (custom qty / destination)" icon="fa-exchange"
                                 class="btn-link text-muted"
                                 invisible="state in ('done', 'cancelled', 'skipped', 'pending')"/>
                     </list>