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

feat: Added "Silent" theme #865

Closed
wants to merge 3 commits into from
Closed

feat: Added "Silent" theme #865

wants to merge 3 commits into from

Conversation

torbratsberg
Copy link
Contributor

Simple theme for quick operations where you barely need to look at the Telescope window.

Silent theme is a small window without preview.
Made for super quick operations.
@disrupted
Copy link
Contributor

I like it, can imagine this really well for LSP code actions. Bonus points if we figure out how to place the popup relative to the current cursor location.

@torbratsberg
Copy link
Contributor Author

@disrupted I agree! I couldn't find a way to set it relative to the cursor with the current configuration options, but that was the initial plan.

@Conni2461
Copy link
Member

Isn't this just require('telescope.themes').get_dropdown{ previewer = false } with a different result_height? Also might be interesting for you but there is a huge refactor of how layouts work here: #823

@tjdevries
Copy link
Member

Yeah, this seems very close to the dropdown theme?

@torbratsberg
Copy link
Contributor Author

It is very similar to the dropdown theme, but intended to be used for smaller and quicker actions. I just saw this PR #878 which has almost the same theme as Silent, but relative to the cursor which is probably a better layout for this kind of theme. So this PR can be closed.

@sudormrfbin
Copy link

I like it, can imagine this really well for LSP code actions. Bonus points if we figure out how to place the popup relative to the current cursor location.

There's an open PR for cursor relative positioning - #878

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.

6 participants