Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GS] Make the keyboard shortcut customizable #77351

Open
Tracked by #83354
ryankeairns opened this issue Sep 14, 2020 · 9 comments
Open
Tracked by #83354

[GS] Make the keyboard shortcut customizable #77351

ryankeairns opened this issue Sep 14, 2020 · 9 comments
Labels
enhancement New value added to drive a business result Feature:Navigational Search Global search bar old Used to help sort old issues on GH Projects which don't support the Created search term. Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)

Comments

@ryankeairns
Copy link
Contributor

The keyboard shortcut has changed more than once since we've started working on this feature. It has been challenging to find a solution that does not conflict with one of the major browsers or screen readers. Given the wide range of possible conflicts - many which very per user - our belief is that this should be customizable via Advanced Settings.

In addition to the advanced setting, we also have on our roadmap the creation of a global keyboard shortcut registry to avoid conflict within our own applications as shortcut usage continues to grow.

@ryankeairns ryankeairns added the REASSIGN from Team:Core UI Deprecated label for old Core UI team label Sep 14, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core-ui (Team:Core UI)

@myasonik
Copy link
Contributor

Is this worth adding as an advanced setting now just to be usurped by the global keboard shortcut registry down the road?

One is significantly easier than the other but it's still duplicate work that will cannibalize itself which seems unfortunate.

@ryankeairns
Copy link
Contributor Author

Based upon experience and feedback thus far (and its not even shipped), I think it would be wise to add the advanced setting rather than wait for the shortcut registry.

@ryankeairns ryankeairns changed the title [GS] Make the navigational search shortcut customizable [GS] Make the keyboard shortcut customizable Oct 20, 2020
@ryankeairns ryankeairns added the enhancement New value added to drive a business result label Oct 20, 2020
@formgeist
Copy link
Contributor

+1 for adding a custom shortcut option in advanced settings. Europeans most likely won't be able to do / easily because the keyboard shortcut is Shift + 7.

@ryankeairns
Copy link
Contributor Author

This feels like a necessary item to prioritize in the near term. I've moved it to the 'planned' category, as such.

cc:/ @alexfrancoeur @myasonik

@myasonik
Copy link
Contributor

myasonik commented Dec 3, 2020

@ryankeairns Are you thinking we tackle just customizing this specific keyboard shortcut or tackle the wider issue of registering and customizing keyboard shortcuts across Kibana broadly?

The former would be quicker and easier. But long-term we'll still need to solve the broad issue which will cause some duplicate work.

The latter we'll probably need significant support from the Core or App Services teams so it's bigger not just technically but also organizationally. (But then we only have to do the work once.)

@ryankeairns
Copy link
Contributor Author

@ryankeairns Are you thinking we tackle just customizing this specific keyboard shortcut or tackle the wider issue of registering and customizing keyboard shortcuts across Kibana broadly?

The former would be quicker and easier. But long-term we'll still need to solve the broad issue which will cause some duplicate work.

The latter we'll probably need significant support from the Core or App Services teams so it's bigger not just technically but also organizationally. (But then we only have to do the work once.)

This one feels like a rather immediate need in that the current situation likely affects a significant swath of people. In this case, I think it's worth doing the quick approach as the bigger topic is not very well-defined and, thus, is not yet on the radar for other teams.

@ryankeairns
Copy link
Contributor Author

Actually, it may be best to tackle this with the upcoming personalization work as this seems like a rather obvious per user setting.

@ryankeairns ryankeairns added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc and removed REASSIGN from Team:Core UI Deprecated label for old Core UI team labels Jan 19, 2021
@pgayvallet pgayvallet added Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) and removed Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc labels Oct 31, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@pgayvallet pgayvallet added the Feature:Navigational Search Global search bar label Oct 31, 2023
@petrklapka petrklapka added the old Used to help sort old issues on GH Projects which don't support the Created search term. label Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:Navigational Search Global search bar old Used to help sort old issues on GH Projects which don't support the Created search term. Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience)
Projects
None yet
Development

No branches or pull requests

6 participants