From c327a5ca93ec690e89912a43fe5436a60237b990 Mon Sep 17 00:00:00 2001 From: Rohit Waghchaure Date: Mon, 25 May 2026 15:33:12 +0530 Subject: [PATCH] fix: job card buttons color --- erpnext/manufacturing/doctype/job_card/job_card.js | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/erpnext/manufacturing/doctype/job_card/job_card.js b/erpnext/manufacturing/doctype/job_card/job_card.js index 140733353f4..795136d2374 100644 --- a/erpnext/manufacturing/doctype/job_card/job_card.js +++ b/erpnext/manufacturing/doctype/job_card/job_card.js @@ -655,10 +655,10 @@ frappe.ui.form.on("Job Card", { }; const buttons_html = [ - show_start && btn("btn-default jcd-btn-start", "play", __("Start Job")), - show_resume && btn("btn-default jcd-btn-resume", "play", __("Resume Job")), + show_start && btn("btn-primary jcd-btn-start", "play", __("Start Job")), + show_resume && btn("btn-primary jcd-btn-resume", "play", __("Resume Job")), show_pause && btn("btn-default jcd-btn-pause", "pause", __("Pause Job")), - show_complete && btn("btn-success jcd-btn-complete", "check", __("Complete Job"), "white"), + show_complete && btn("btn-primary jcd-btn-complete", "check", __("Complete Job"), "white"), ] .filter(Boolean) .join(""); @@ -766,6 +766,14 @@ frappe.ui.form.on("Job Card", { }, 1000); } + // Demote Submit to btn-default when an action button is already primary. + const has_action_button = show_start || show_resume || show_complete; + if (frm.page.btn_primary) { + frm.page.btn_primary + .toggleClass("btn-primary", !has_action_button) + .toggleClass("btn-default", has_action_button); + } + return is_timer_running; },