extern/egroupware_official
1
0
Fork 0
mirror of https://github.com/EGroupware/egroupware.git synced 2024-10-05 17:42:18 +02:00
Code Issues Packages Projects Releases Wiki Activity

Better way to show "no suggestions" indicator, works with local options now too

Browse Source
This commit is contained in:
nathan 2022-09-16 11:17:06 -06:00
parent 12ab9b72a1
commit 1deeee207f
1 changed files with 18 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
api/js/etemplate/Et2Select/SearchMixin.ts
View File

@ -386,6 +386,11 @@ export const Et2WithSearchMixin = <T extends Constructor<LitElement>>(superclass
`;
}
protected _noResultsTemplate()
{
return html`
<div class="no-results">${this.egw().lang("no suggestions")}</div>`;
}
/**
* Do we have the needed properties set, so we can actually do searching
@ -760,8 +765,6 @@ export const Et2WithSearchMixin = <T extends Constructor<LitElement>>(superclass
*/
public async startSearch()
{
this.menu.querySelector("slot").textContent = "";
// Stop timeout timer
clearTimeout(this._searchTimeout);
@ -787,8 +790,14 @@ export const Et2WithSearchMixin = <T extends Constructor<LitElement>>(superclass
this.remoteSearch(this._searchInputNode.value, this.searchOptions)
]).then(() =>
{
// Show / hide no results indicator
this.menu.querySelector("slot").textContent = this.menuItems.length == 0 ? this.egw().lang("No suggestions") : "";
// Show no results indicator
if(this.menuItems.filter(e => !e.classList.contains("no-match")).length == 0)
{
let target = this._optionTargetNode || this;
let temp = document.createElement("div");
render(this._noResultsTemplate(), temp);
target.append(temp.children[0]);
}
// Remove spinner
spinner.remove();
@ -826,8 +835,12 @@ export const Et2WithSearchMixin = <T extends Constructor<LitElement>>(superclass
protected _clearResults()
{
// Remove remote options that aren't used
let target = this._optionTargetNode || this;
// Remove "no suggestions"
target.querySelector(".no-results")?.remove();
// Remove remote options that aren't used
let keepers = this._selected_remote.reduce((prev, current) =>
{
return ":not([value='" + current.value + "'])";