vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

refactor: clear active filter on 'x' button click and remove redundant clear filter option

Browse source
This commit is contained in:
KerollesFathy 2026-01-11 22:50:33 +00:00
parent aa4b61de8e
commit 0f1e1c48af
1 changed files with 7 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
frappe/public/js/frappe/list/list_filter.js
View file

@ -20,6 +20,13 @@ export default class ListFilter {
[],
__("Saved Filters")
);
// Clear active filter on clicking 'x' button
const filter_x_btn = $(".filter-x-button");
filter_x_btn.on("click", () => {
this.active_filter = null;
this.update_active_filter_label("Saved Filters");
});
}
render_saved_filters() {
@ -45,7 +52,6 @@ export default class ListFilter {
});
this.append_create_new_item($menu);
this.append_clear_selected_filter($menu);
}
apply_saved_filter(filter_name, filter_label) {
@ -87,20 +93,6 @@ export default class ListFilter {
});
$menu.append($create_item);
}
append_clear_selected_filter($menu) {
const clear_filters = {
name: "clear_selected",
filter_name: "Clear Selected Filter",
};
const $clear_item = this.filter_template(clear_filters, true);
$clear_item.find(".filter-label").on("click", (e) => {
this.list_view.filter_area.clear();
this.active_filter = null;
this.update_active_filter_label("Saved Filters");
});
$menu.append($clear_item);
}
show_create_filter_dialog() {
const fields = [