From 7d27602c07a3c7163c725fbddd9e9a0fb2b744a5 Mon Sep 17 00:00:00 2001 From: Akhil Narang Date: Fri, 10 May 2024 17:49:07 +0530 Subject: [PATCH] fix: allow navigating to `Table MultiSelect` fields using tab It was skipped earlier as it was part of `no_value_type` and didn't have an exception added like `Table` Signed-off-by: Akhil Narang --- frappe/public/js/frappe/form/layout.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/frappe/public/js/frappe/form/layout.js b/frappe/public/js/frappe/form/layout.js index 78f535e91c1..9817f1daa9c 100644 --- a/frappe/public/js/frappe/form/layout.js +++ b/frappe/public/js/frappe/form/layout.js @@ -629,7 +629,10 @@ frappe.ui.form.Layout = class Layout { // show grid row (if exists) field.grid.grid_rows[0].show_form(); return true; - } else if (!frappe.model.no_value_type.includes(field.df.fieldtype)) { + } else if ( + field.df.fieldtype === "Table MultiSelect" || + !frappe.model.no_value_type.includes(field.df.fieldtype) + ) { this.set_focus(field); return true; }