From e4dad7323bbbbcec52bf54db0dd9be66e81dfd98 Mon Sep 17 00:00:00 2001
From: Sagar Vora <16315650+sagarvora@users.noreply.github.com>
Date: Sun, 1 Mar 2026 10:05:09 +0530
Subject: [PATCH] fix: regression in grid button hiding logic

---
 frappe/core/doctype/recorder/recorder.js | 2 +-
 frappe/public/js/frappe/form/grid.js     | 4 +++-
 frappe/public/scss/common/global.scss    | 8 ++++----
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/frappe/core/doctype/recorder/recorder.js b/frappe/core/doctype/recorder/recorder.js
index 735511f5b1..aee75635d0 100644
--- a/frappe/core/doctype/recorder/recorder.js
+++ b/frappe/core/doctype/recorder/recorder.js
@@ -24,7 +24,7 @@ frappe.ui.form.on("Recorder", {
 		});
 
 		let index_grid = frm.fields_dict.suggested_indexes.grid;
-		index_grid.wrapper.find(".grid-footer").toggle(true);
+		index_grid.wrapper.find(".grid-footer").toggleClass("hidden", false);
 		index_grid.toggle_checkboxes(true);
 		index_grid.df.cannot_delete_rows = true;
 		index_grid.add_custom_button(__("Add Indexes"), function () {
diff --git a/frappe/public/js/frappe/form/grid.js b/frappe/public/js/frappe/form/grid.js
index e901dc63a5..ad7b2fa8c8 100644
--- a/frappe/public/js/frappe/form/grid.js
+++ b/frappe/public/js/frappe/form/grid.js
@@ -671,9 +671,11 @@ export default class Grid {
 			this.wrapper.find(".grid-footer").addClass("hidden");
 		}
 
+		// don't be tempted to use the `.hidden` class here
+		// it is used in other logic for the same buttons and will cause conflicts
 		this.wrapper
 			.find(".grid-add-row, .grid-add-multiple-rows, .grid-upload")
-			.toggleClass("hidden", !is_editable);
+			.toggleClass("d-none", !is_editable);
 	}
 
 	setup_fields() {
diff --git a/frappe/public/scss/common/global.scss b/frappe/public/scss/common/global.scss
index 4e80af9ecc..ce736107ac 100644
--- a/frappe/public/scss/common/global.scss
+++ b/frappe/public/scss/common/global.scss
@@ -142,10 +142,6 @@ body {
 	-webkit-transform: translate(-50%, -50%);
 }
 
-.hide {
-	display: none !important;
-}
-
 .btn-link {
 	box-shadow: none !important;
 	outline: none;
@@ -159,6 +155,10 @@ body {
 	@extend .d-none;
 }
 
+.hide {
+	@extend .d-none;
+}
+
 .margin {
 	margin: var(--margin-sm);
 }