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`

Signed-off-by: Akhil Narang <me@akhilnarang.dev>
  • Loading branch information
akhilnarang committed May 10, 2024
1 parent 2c0b77b commit 7d27602
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 @@ -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;
}
Expand Down

0 comments on commit 7d27602

Please sign in to comment.