Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

add features "choice module to use" and "support float notify window" #382

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

BingCoke
Copy link

choice module

#364

we want to disable some module that we will not use.
So here is a simply solustion.
We can set options.open and open some module to use.

rt.setup({
 tools = {
    -- if you want disable hints use this config
    inlay_hints = {
      auto = false,
    }
  },

  open = {
    crate_graph = false,
    expand_macro = false,
    external_docs = true,
    debuggables = false,
    hover_range = false,
    workspace_refresh = true,
    move_item = false,
    standalone = false,
    dap = false,
    parent_module = false,
    runnables = false,
  },
})

support float notify window

cargo workspace reloaded will trigger vim.notify() which give me a bad editing experience.
So I create a float window to replace it.

Peek 2023-05-14 13-08

@sigmaSd
Copy link

sigmaSd commented May 18, 2023

I feel the word open is not obvious maybe enable is better ?

@sigmaSd
Copy link

sigmaSd commented May 18, 2023

Or maybe naming doesn't matter, I'm interested i where this pr will go, since I have a plugin with the same architecture as rust-tools and I think this is a good feature to add.

@IndianBoy42
Copy link
Contributor

How much time does disabling this actually save? could the loading be made lazier instead to reap similar benefits?

enable is much more common to use in neovim plugins

and you can just use nvim-notify if you want vim.notify to have a floating window, doesn't feel like its the scope of this plugin

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants