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

fix: hide empty filter state when filter is added

Browse source
This commit is contained in:
prssanna 2020-07-31 13:55:50 +05:30
parent b4161febf9
commit 3ca1d618e8
1 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
frappe/public/js/frappe/ui/filters/filter_list.js
View file

@ -60,14 +60,19 @@ frappe.ui.FilterGroup = class {
});
this.filter_button.on('shown.bs.popover', (e) => {
let hide_empty_filters = this.filters && this.filters.length > 0;
if (!this.wrapper) {
this.wrapper = $('.filter-popover');
if (this.filters && this.filters.length > 0) {
console.log('shownn', this.filters);
if (hide_empty_filters) {
this.toggle_empty_filters(false);
this.add_filters_to_popover(this.filters);
}
this.set_filter_events();
}
hide_empty_filters && this.toggle_empty_filters(false);
});
this.filter_button.on('hidden.bs.popover', (e) => {
@ -255,15 +260,15 @@ frappe.ui.FilterGroup = class {
return $(`
<div class="filter-area">
<div class="filter-edit-area text-center">
<div class="text-muted empty-filters">No filters selected</div>
<div class="text-muted empty-filters">${__('No filters selected')}</div>
</div>
<hr class="divider"></hr>
<div class="filter-action-buttons">
<div class="text-muted add-filter">
${__("+ Add a Filter")}
${__('+ Add a Filter')}
</div>
<button class="btn btn-secondary-dark btn-xs clear-filters">
${__("Clear Filters")}
${__('Clear Filters')}
</button>
</div>
</div>`