Skip to content

Commit

Permalink
fix: allow navigating to Table MultiSelect fields using tab
Browse files Browse the repository at this point in the history
It was skipped earlier as it was part of `no_value_type` and didn't have an exception added like `Table`

(cherry picked from commit 7d27602)
Signed-off-by: Akhil Narang <me@akhilnarang.dev>
  • Loading branch information
akhilnarang committed May 10, 2024
1 parent 4fe1043 commit 1e5faff
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion frappe/public/js/frappe/form/layout.js
Original file line number Diff line number Diff line change
Expand Up @@ -578,7 +578,10 @@ frappe.ui.form.Layout = class Layout {
// show grid row (if exists)
field.grid.grid_rows[0].show_form();
return true;
} else if (!in_list(frappe.model.no_value_type, field.df.fieldtype)) {
} else if (
field.df.fieldtype === "Table MultiSelect" ||
!in_list(frappe.model.no_value_type, field.df.fieldtype)
) {
this.set_focus(field);
return true;
}
Expand Down

0 comments on commit 1e5faff

Please sign in to comment.