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

refactor: assign to dropdown is searchable by data-name

Browse source
This commit is contained in:
Shivam Mishra 2019-07-12 14:58:47 +05:30
parent b225b36f74
commit 16ff984cab
2 changed files with 11 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
frappe/public/js/frappe/list/list_sidebar.js
View file

@ -22,7 +22,7 @@ frappe.views.ListSidebar = class ListSidebar {
this.sidebar = $('<div class="list-sidebar overlay-sidebar hidden-xs hidden-sm"></div>')
.html(sidebar_content)
.appendTo(this.page.sidebar.empty());
this.setup_reports();
this.setup_list_filter();
this.setup_views();
@ -253,10 +253,17 @@ frappe.views.ListSidebar = class ListSidebar {
let $elements = dropdown.find('li');
$dropdown_search.on('keyup',()=> {
let text_filter = $search_input.val().toLowerCase();
// Replace trailing and leading spaces
text_filter = text_filter.replace(/^\s+|\s+$/g, '')
let text;
for (var i = 0; i < $elements.length; i++) {
text = $elements.eq(i).find(text_class).text();
if (text.toLowerCase().indexOf(text_filter) > -1) {
let text_element = $elements.eq(i).find(text_class);
let text = text_element.text().toLowerCase();
// Search data-name since label for current user is 'Me'
let name = text_element.data('name').toLowerCase();
if (text.search(text_filter) || name.search(text_filter)) {
$elements.eq(i).css('display','');
} else {
$elements.eq(i).css('display','none');

2
frappe/public/js/frappe/list/list_sidebar_group_by.js
View file

@ -135,7 +135,7 @@ frappe.views.ListGroupBy = class ListGroupBy {
return `<li class="group-by-item" data-value="${value}">
<a class="badge-hover" href="#" onclick="return false;">
<span class="group-by-value">${label}</span>
<span class="group-by-value" data-name="${field.name}">${label}</span>
<span class="badge pull-right group-by-count">${field.count}</span>
</a>
</li>`;