Skip to content

feat: function to re-open the most recent search#371

Draft
justDeeevin wants to merge 3 commits intodmtrKovalenko:mainfrom
justDeeevin:resume
Draft

feat: function to re-open the most recent search#371
justDeeevin wants to merge 3 commits intodmtrKovalenko:mainfrom
justDeeevin:resume

Conversation

@justDeeevin
Copy link
Copy Markdown

@justDeeevin justDeeevin commented Apr 13, 2026

Replicates the "resume" functionality of telescope and Snacks.nvim.picker (see #159)

Fairly simple to do; just retain the previous state.

This entirely overrode libraries sent from lsp clients. It makes more sense for individuals to configure their location of the neovim library (not everyone has it installed with homebrew!), and others can be handled with tools like LazyDev.
@justDeeevin
Copy link
Copy Markdown
Author

justDeeevin commented Apr 13, 2026

Draft because I can't seem to retain the preview... I got rid of line 595, but that isn't cutting it. Unsure what else could be clearing the preview.

I could always opt to update the preview instead but that feels wasteful

@dmtrKovalenko
Copy link
Copy Markdown
Owner

plz revert the formatting changes

Replicates the "resume" functionality of telescope and Snacks.nvim.picker (see dmtrKovalenko#159)
@justDeeevin
Copy link
Copy Markdown
Author

justDeeevin commented Apr 14, 2026

Gotcha, wasn't sure whether that would be agreeable so i figured i'd leave it in. Pretty easy to just get rid of the commit b/c jujutsu vcs :3

What do you think of the changes? Any advice or insights about the preview problem?

@dmtrKovalenko
Copy link
Copy Markdown
Owner

Does it work? Do you still want to proceee with changes

@justDeeevin
Copy link
Copy Markdown
Author

Resuming the previous search works for all but the preview, which I'm seeking advice on. It could technically go in as-is but I really want to pin this down.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants