Skip to content
This repository was archived by the owner on Jul 19, 2022. It is now read-only.

Add keyboard support to catalog search#303

Merged
hojberg merged 1 commit intomainfrom
catalog-search-keyboard
Jan 12, 2022
Merged

Add keyboard support to catalog search#303
hojberg merged 1 commit intomainfrom
catalog-search-keyboard

Conversation

@hojberg
Copy link
Member

@hojberg hojberg commented Jan 11, 2022

Problem

The user should be able to select search results by using the keyboard.

Solution

When searching through the catalog support keyboard navigation with up
and down arrows as well as selection with enter and indexed selection
with ; followed by an index number. This mirrors the functionality of
the Finder.

@hojberg hojberg requested a review from pchiusano January 11, 2022 20:32
, basePath = flags.basePath
, apiBasePath = ApiBasePath flags.apiBasePath
, appContext = AppContext.fromString flags.appContext
, navKey = navKey
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved the navKey out of the App model and into Env so that we can navigate from any page.

When searching through the catalog support keyboard navigation with up
and down arrows as well as selection with enter and indexed selection
with ; followed by an index number. This mirrors the functionality of
the Finder.
@hojberg hojberg force-pushed the catalog-search-keyboard branch from 8d8fe28 to 755c41c Compare January 11, 2022 20:39
@hojberg hojberg merged commit d6b308e into main Jan 12, 2022
@hojberg hojberg deleted the catalog-search-keyboard branch January 12, 2022 16:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant