Blur search field on escape if empty
This commit is contained in:
parent
9d28183e76
commit
8919482511
@ -232,11 +232,19 @@ describe('GlobalSearchComponent', () => {
|
|||||||
component.searchInputKeyDown(new KeyboardEvent('keydown', { key: 'Enter' }))
|
component.searchInputKeyDown(new KeyboardEvent('keydown', { key: 'Enter' }))
|
||||||
expect(primaryActionSpy).toHaveBeenCalled()
|
expect(primaryActionSpy).toHaveBeenCalled()
|
||||||
|
|
||||||
|
component.query = 'test'
|
||||||
const resetSpy = jest.spyOn(GlobalSearchComponent.prototype as any, 'reset')
|
const resetSpy = jest.spyOn(GlobalSearchComponent.prototype as any, 'reset')
|
||||||
component.searchInputKeyDown(
|
component.searchInputKeyDown(
|
||||||
new KeyboardEvent('keydown', { key: 'Escape' })
|
new KeyboardEvent('keydown', { key: 'Escape' })
|
||||||
)
|
)
|
||||||
expect(resetSpy).toHaveBeenCalled()
|
expect(resetSpy).toHaveBeenCalled()
|
||||||
|
|
||||||
|
component.query = ''
|
||||||
|
const blurSpy = jest.spyOn(component.searchInput.nativeElement, 'blur')
|
||||||
|
component.searchInputKeyDown(
|
||||||
|
new KeyboardEvent('keydown', { key: 'Escape' })
|
||||||
|
)
|
||||||
|
expect(blurSpy).toHaveBeenCalled()
|
||||||
})
|
})
|
||||||
|
|
||||||
it('should search on query debounce', fakeAsync(() => {
|
it('should search on query debounce', fakeAsync(() => {
|
||||||
|
@ -265,7 +265,11 @@ export class GlobalSearchComponent implements OnInit {
|
|||||||
) {
|
) {
|
||||||
this.primaryButtons.first.nativeElement.click()
|
this.primaryButtons.first.nativeElement.click()
|
||||||
} else if (event.key === 'Escape' && !this.resultsDropdown.isOpen()) {
|
} else if (event.key === 'Escape' && !this.resultsDropdown.isOpen()) {
|
||||||
this.reset(true)
|
if (this.query?.length) {
|
||||||
|
this.reset(true)
|
||||||
|
} else {
|
||||||
|
this.searchInput.nativeElement.blur()
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user