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

Clear jumplist #4757

Open
dariooddenino opened this issue Nov 15, 2022 · 5 comments · May be fixed by #6251 or #9215
Open

Clear jumplist #4757

dariooddenino opened this issue Nov 15, 2022 · 5 comments · May be fixed by #6251 or #9215
Labels
A-helix-term Area: Helix term improvements C-enhancement Category: Improvements E-easy Call for participation: Experience needed to fix: Easy / not much E-good-first-issue Call for participation: Issues suitable for new contributors

Comments

@dariooddenino
Copy link
Contributor

I think it could be useful to have both a clear all jumplist entries command and a key binding to delete entries from <space> j.

@dariooddenino dariooddenino added the C-enhancement Category: Improvements label Nov 15, 2022
@kirawi kirawi added E-easy Call for participation: Experience needed to fix: Easy / not much A-helix-term Area: Helix term improvements E-good-first-issue Call for participation: Issues suitable for new contributors labels Nov 15, 2022
@swork1
Copy link

swork1 commented Nov 16, 2022

I think it could be useful to have both a clear all jumplist entries command and a key binding to delete entries from <space> j.

Are we wanting this to be a nested menu? This would change the default keybind from <space> j to <space> j j for opening the jumplist.
<space> j brings up menu
j opens jumplist
c clears jumplist

PickerAction is currently being implemented in this pr

We should be able to use that to implement keybindings in the picker menu:
ctrl-x for delete

@QiBaobin
Copy link
Contributor

Once we support delete action on jumplist picker, we can select all them first to achieve cleaning jumplist.

Multiple selection for pickers is being implemented in #4837 .

@Bruce-Hopkins Bruce-Hopkins linked a pull request Mar 10, 2023 that will close this issue
@Bruce-Hopkins
Copy link
Contributor

I made a typed command to clear the jumplist. Does this look like a good solution?

@godalming123
Copy link

@Bruce-Hopkins I think that maybe implementing removing just one item first would be more useful as a user does not want to clear their whole jumplist just to remove a few items although I suppose that it is better then nothing

@ondrej-ivanko
Copy link

This for me is sorely needed. I'm really confused on what basis the lines are added to jumplist picker, when I'm not really using ctrl+s keybinding. I suppose there are an editor and a user jumplists, which are mixed together.

Without a proper control what is being add to jumplist picker, I would definitely welcome a shortcut to clear the jumplist, to at least partially alleviate the issue. I would just clear selected redundant lines. Clear the whole jumplist would be also welcome, but limited in use for me.

It would be nice if jumplist picker could be bounded to it's buffer via config for example. It would mean less filtering by name in picker window. I wouldn't change the default behaviour though, as lot of people might still love to jump over different buffers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-helix-term Area: Helix term improvements C-enhancement Category: Improvements E-easy Call for participation: Experience needed to fix: Easy / not much E-good-first-issue Call for participation: Issues suitable for new contributors
Projects
None yet
7 participants