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

[dashboard]Omnibar issues and enhancements #16198

Closed
geido opened this issue Aug 11, 2021 · 7 comments
Closed

[dashboard]Omnibar issues and enhancements #16198

geido opened this issue Aug 11, 2021 · 7 comments
Assignees
Labels
enhancement:committed Enhancement planned / committed by a contributor inactive Inactive for >= 30 days P3 Priority item - Normal

Comments

@geido
Copy link
Member

geido commented Aug 11, 2021

Issues with the current implementation

There are several issues currently affecting the experimental Omnibar functionality:

  • The Omnibar only opens up on a Dashboard page but not on the Dashboards' list
  • The Omnibar does not reset the input when closed and opened back. Currently, it keeps the previous search
  • Selecting an option in the dropdown is not possible with the arrow keys due to the autocomplete
  • Closing the Omnibar is only possible through reloading. The Omnibar should be closable with the ESC key (this PR introduces this ability fix(Dashboard): Omnibar dropdown visibility and keyboard commands #16168). I suggest it should close also when clicking outside of the modal.

(thanks to @michael-s-molina for spotting some of these)

Solutions

  • Make the Omnibar appear on the Dashboards' list with cmd + k or ctrl + k
  • Clear up the search when the Omnibar is closed
  • Disable autocomplete
  • Close the Omnibar when clicking outside

Looking forward

I believe that the Omnibar does not provide much value right now as its only ability is to search for Dashboards. Also, it is only available on the Dashboard page (soon on the Dashboards' list) and nowhere else.

It would make more sense if the Omnibar was available everywhere and would allow searching for the main items in Superset, including the Charts.

@geido
Copy link
Member Author

geido commented Aug 11, 2021

I'll start working on the solutions that I have detailed up there as soon as priorities will allow

@srinify
Copy link
Contributor

srinify commented Aug 11, 2021

cc @hughhhh as well who I believe did some of the Omnibar work

@geido
Copy link
Member Author

geido commented Aug 11, 2021

@junlincc FYI

@junlincc junlincc added the P3 Priority item - Normal label Aug 11, 2021
@junlincc
Copy link
Member

junlincc commented Aug 11, 2021

@geido @michael-s-molina thank you for spotting it and taking it on. All good points feel free to work on any low hanging fruit to uplevel superset, while I am planning for Q4. :) Please make sure your PR description and video captures all enhancement.

@junlincc junlincc changed the title Omnibar issues and enhancements [dashboard]Omnibar issues and enhancements Aug 11, 2021
@junlincc junlincc added viz:dashboard:omnibar enhancement:committed Enhancement planned / committed by a contributor labels Aug 11, 2021
@junlincc
Copy link
Member

And let's remove the feature flag once all the enhancement goes in :) @rusackas sgty?

@geido geido self-assigned this Aug 13, 2021
@geido geido mentioned this issue Aug 16, 2021
9 tasks
@geido geido removed the Flexiana label Jan 27, 2022
@stale
Copy link

stale bot commented Apr 17, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. For admin, please label this issue .pinned to prevent stale bot from closing the issue.

@stale stale bot added the inactive Inactive for >= 30 days label Apr 17, 2022
@rusackas
Copy link
Member

rusackas commented Feb 2, 2023

I believe the omnibar has been removed as of 2.0, and this is no longer relevant. If I'm taking crazy pills, let me know.

@rusackas rusackas closed this as completed Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement:committed Enhancement planned / committed by a contributor inactive Inactive for >= 30 days P3 Priority item - Normal
Projects
None yet
Development

No branches or pull requests

4 participants