Skip to content

Commit 4c42eaf

Browse files
authored
Merge pull request #1332 from sivakumar-kailasam/keyboard-shortcut-for-search
Focus on search on Shift+S
2 parents 6b25e6d + f22c152 commit 4c42eaf

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

app/controllers/application.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,8 @@ export default Controller.extend(EKMixin, {
1010
session: service(),
1111

1212
keyboardActivated: true,
13-
focusSearch: on(keyDown('KeyS'), keyPress('KeyS'), function(event) {
13+
14+
focusSearch: on(keyDown('KeyS'), keyPress('KeyS'), keyDown('shift+KeyS'), function(event) {
1415
if (event.ctrlKey || event.altKey || event.metaKey) {
1516
return;
1617
}

tests/acceptance/search-test.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,10 @@ module('Acceptance | search', function(hooks) {
7575
await blur('[data-test-search-input]');
7676
await triggerKeyDown('KeyS');
7777
assert.dom('[data-test-search-input]').isFocused();
78+
79+
await blur('[data-test-search-input]');
80+
await triggerKeyDown('shift+KeyS');
81+
assert.dom('[data-test-search-input]').isFocused();
7882
});
7983

8084
test('check search results are by default displayed by relevance', async function(assert) {

0 commit comments

Comments
 (0)