-
Notifications
You must be signed in to change notification settings - Fork 210
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
[FLI-919] Context based command bar #2904
Comments
Hi, can I work on this issue? |
Yes please @og118 ! Assigning to you. Thank you! |
Hey @markphelps, I went through the code, and have a few questions before I start working on this issue:
Creating variants, rules and/or rollouts are currently being done in a navigation drawer, so ideally I assume that this command to create a new variant/rule/rollout should open up the navigation pane? Please confirm this behavior. (Attaching an image for your reference) I need some guidance on how should I go about this? The variables that control the drawer open/close is in the local state of the components. How should I update them from |
Ah yeah I didnt think about the fact that some of the components are drawers instead of new pages. I think maybe in the short term we could just exclude anything that opens a drawer or modal? Maybe we just support Unless you dont think it would be too much work to move the open / close props to redux? |
Sure, I'll go ahead and implement the routing based commands, they are pretty straightforward. Also I will let you know how much effort would be needed for moving the open/close drawer variables by today. |
Problem
Now that we have a command bar in v1.39: https://twitter.com/flipt_io/status/1772254008809251019, it would be neat if it took into account the current section in the application that you are on.
More below 👇🏻
Ideal Solution
If im on the
/flags
page, and I open the command bar it should allow me to quickly jump toNew Flag
, same for segmentsIf I'm on a specific flag page like
/flags/example
, and I hit the command bar, it should allow me to quickly create a new variant or rule (variant flag type), or rollout (boolean flag type)Similiar for segment -> constraint
Additional actions like
Copy To Namespace
should be able to be performed quickly by opening the command bar on a single flag or segment page in the UITheres likely more we can do, but these seem like good first steps
Search
Additional Context
This is the library we use for command bar: https://github.com/pacocoursey/cmdk
Here is where its implemented in Flipt UI: https://github.com/flipt-io/flipt/blob/main/ui/src/components/command/CommandDialog.tsx
FLI-919
The text was updated successfully, but these errors were encountered: