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

There should be a setting that "resets" cmdpal when the activation key is triggered #271

Closed
Tracked by #450
joadoumie opened this issue Dec 31, 2024 · 2 comments · Fixed by #326
Closed
Tracked by #450
Assignees
Labels
Area-HostUX UX elements of the host CmdPal application Area-Settings Anything to do with core settings

Comments

@joadoumie
Copy link
Collaborator

Sometimes, you go to a new page in cmdpal and then can see some info and actually just do something that makes cmdpal lose focus rather than actually invoking a command.

In this case, the next time I go back to cmdpal, I don't want to be brought back to the previous screen, I want to be back in the root view with a blank search state.

Example, I invoke the window walker command:

image

I get distracted, or simply decide to dismiss the palette. The next time I invoke the palette, I should be able to return to the root state. Right now when I invoke the palette again, the same page is shown.

I believe this should likely be the default behavior and the alternative behavior could be the setting. This way whenever I invoke cmdpal I'm confident I can type and immediately get the results that I want.

@joadoumie joadoumie added Area-HostUX UX elements of the host CmdPal application Area-Settings Anything to do with core settings labels Dec 31, 2024
@zadjii
Copy link
Collaborator

zadjii commented Dec 31, 2024

I can make this a setting.

Probably trivial to add a setting on top of #264 for just "does summon reset to the root view or not".

Mildly harder for "Two different keybinds, one for each behavior". Honestly that just sounds confusing to users

@joadoumie
Copy link
Collaborator Author

Don't think two key binds is necessary - agreed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-HostUX UX elements of the host CmdPal application Area-Settings Anything to do with core settings
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants