From c95a3048ce285abae57427d511b2527f3ef78f82 Mon Sep 17 00:00:00 2001 From: Peter Bacon Darwin Date: Sun, 12 Mar 2017 15:05:01 +0000 Subject: [PATCH] fix(aio): `SearchBoxComponent` should send query on click The input may still have focus when the user hits ESC, causing the search to be hidden. If the user then clicks on the input again, they would expect it to reopen the results. --- aio/src/app/search/search-box/search-box.component.html | 3 ++- .../app/search/search-box/search-box.component.spec.ts | 8 ++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/aio/src/app/search/search-box/search-box.component.html b/aio/src/app/search/search-box/search-box.component.html index 5bdf87ca98..ded971e9e2 100644 --- a/aio/src/app/search/search-box/search-box.component.html +++ b/aio/src/app/search/search-box/search-box.component.html @@ -3,5 +3,6 @@ mdInput placeholder="Search" (keyup)="onSearch($event.target.value, $event.which)" - (focus)="onSearch($event.target.value)"> + (focus)="onSearch($event.target.value)" + (click)="onSearch($event.target.value)"> diff --git a/aio/src/app/search/search-box/search-box.component.spec.ts b/aio/src/app/search/search-box/search-box.component.spec.ts index bfd9116d04..f282724988 100644 --- a/aio/src/app/search/search-box/search-box.component.spec.ts +++ b/aio/src/app/search/search-box/search-box.component.spec.ts @@ -67,4 +67,12 @@ describe('SearchBoxComponent', () => { expect(search.search).toHaveBeenCalledWith('some query'); })); }); + + describe('on click', () => { + it('should call the search service on click', inject([SearchService], (search: SearchService) => { + const input = fixture.debugElement.query(By.css('input')); + input.triggerEventHandler('click', { target: { value: 'some query'}}); + expect(search.search).toHaveBeenCalledWith('some query'); + })); + }); });