Skip to content

Commit

Permalink
feat: add the reset button
Browse files Browse the repository at this point in the history
  • Loading branch information
razonyang committed Feb 21, 2024
1 parent 7373ad7 commit 581ed81
Show file tree
Hide file tree
Showing 3 changed files with 62 additions and 9 deletions.
46 changes: 42 additions & 4 deletions assets/search/js/form.ts
Original file line number Diff line number Diff line change
Expand Up @@ -35,10 +35,13 @@ export default class Form {

render() {
return `<form class="search-form">
<div class="search-input-group">
<span class="search-input-icon">${params.icons['search']}</span>
<span class="search-spinner">${params.icons['spinner']}</span>
<input type="search" name="q" class="search-input search-form-control" placeholder="${i18n.translate('input_placeholder')}" autocomplete="off" disabled/>
<div class="search-input-group-wrapper">
<div class="search-input-group">
<span class="search-input-icon">${params.icons['search']}</span>
<span class="search-spinner">${params.icons['spinner']}</span>
<input type="search" name="q" class="search-input search-form-control" placeholder="${i18n.translate('input_placeholder')}" autocomplete="off" disabled/>
<button class="search-reset-button disabled" type="reset">${params.icons['reset']}</button>
</div>
<button class="search-modal-close" type="button">${i18n.translate('cancel')}</button>
</div>
<div class="search-form-meta">
Expand Down Expand Up @@ -200,6 +203,31 @@ export default class Form {
this.submit()
})
})

this.ele.addEventListener('reset', (e) => {

Check warning on line 207 in assets/search/js/form.ts

View workflow job for this annotation

GitHub Actions / lint

'e' is defined but never used
const lang = document.documentElement.getAttribute('lang') ?? ''
this.language?.querySelectorAll('.search-dropdown-item').forEach((ele) => {
if (ele.getAttribute('data-value') !== lang) {
ele.classList.remove('active')
} else {
this.language.classList.add('active')
this.language.setAttribute('data-value', lang)
this.language.querySelector('.search-dropdown-label').innerHTML = ele.innerHTML
ele.classList.add('active')
}
})
for (let name of ['years', 'taxonomies']) {

Check failure on line 219 in assets/search/js/form.ts

View workflow job for this annotation

GitHub Actions / lint

'name' is never reassigned. Use 'const' instead
this.ele.querySelectorAll(`.search-${name}.active`).forEach((ele) => {
ele.querySelectorAll('.search-dropdown-item.active').forEach((item) => {
item.classList.remove('active')
})
ele.classList.remove('active')
})
}
setTimeout(() => {
this.submit()
}, 1)
})

this.spinner.show()
engine.init().then(() => {
Expand Down Expand Up @@ -245,6 +273,14 @@ export default class Form {
}
}

private showResetBtn() {
this.ele.querySelector('button[type="reset"]')?.classList.remove('disabled')
}

private hideResetBtn() {
this.ele.querySelector('button[type="reset"]')?.classList.add('disabled')
}

private submit() {
const query = this.getQuery()
this.updatePage(query)
Expand All @@ -255,11 +291,13 @@ export default class Form {
const taxonomies = this.getTaxonomies()

if (query === '' && Object.values(taxonomies).filter((item) => item.length > 0).length == 0) {
this.hideResetBtn()
this.renderer.clean()
this.renderer.renderHistories()
return
}

this.showResetBtn()
this.spinner.show()
engine.search(query, sorting, lang, years, taxonomies).then(({ results, time }) => {
this.renderer.render(query, results, time)
Expand Down
24 changes: 19 additions & 5 deletions assets/search/scss/form/index.scss
Original file line number Diff line number Diff line change
Expand Up @@ -4,14 +4,19 @@
margin-bottom: 0.5rem;
}

.search-input-group-wrapper {
display: flex;
}

.search-input-group {
display: flex;
flex: 1;
position: relative;
}

.search-input-icon,
.search-spinner {
.search-spinner,
.search-reset-button {
background: inherit;
border: none;
color: var(--search-primary);
Expand All @@ -22,13 +27,17 @@
justify-content: center;
align-items: center;
padding: 0 0.5rem;
left: 0;

&.disabled {
display: none;
}
}

.search-input-icon,
.search-spinner {
left: 0;
}

.search-spinner {
svg {
background: var(--search-bg);
Expand All @@ -49,15 +58,20 @@
}
}

.search-reset-button {
border: 0;
background: transparent;
right: 0;
}

.search-input {
border-radius: 0.25rem;
border: 1px solid var(--search-primary);
padding: 0.75rem 0.5rem 0.75rem 2.25rem;
padding: 0.75rem 1.5rem 0.75rem 2.25rem;
flex: 1;

&::-webkit-search-cancel-button {
cursor: pointer;
padding: 0.125rem;
display: none;
}
}

Expand Down
1 change: 1 addition & 0 deletions layouts/partials/search/assets/js-resource.html
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,7 @@
"expand" (partial "icons/icon" (dict "vendor" "bootstrap" "name" "chevron-expand" "size" "1em"))
"year" (partial "icons/icon" (dict "vendor" "bootstrap" "name" "calendar-check" "size" "1em"))
"taxonomies" (partial "icons/icon" (dict "vendor" "bootstrap" "name" "tags" "size" "1em"))
"reset" (partial "icons/icon" (dict "vendor" "bootstrap" "name" "x" "size" "1.5em"))
}}
{{/* Include the following icons when necessary. */}}
{{- if default true .Site.Params.search.index_all_pages }}
Expand Down

0 comments on commit 581ed81

Please sign in to comment.