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

:digraphs doesn't work well #39

Closed
p00f opened this issue Oct 7, 2022 · 9 comments
Closed

:digraphs doesn't work well #39

p00f opened this issue Oct 7, 2022 · 9 comments

Comments

@p00f
Copy link

p00f commented Oct 7, 2022

Describe the bug
:digraphs just shows a wide notification at the top of the screen

Which version of Neovim are you using?
neovim/neovim@cfdb4cb

To Reproduce
Steps to reproduce the behavior:
:digraphs

Expected Behavior
Digraphs in a new buffer (as described in readme: "no more more-prompt")

Screenshots
image

Noice Log
Log file is empty

@folke
Copy link
Owner

folke commented Oct 9, 2022

This is a current limitation of nvim-notify. See rcarriga/nvim-notify#137

@folke folke closed this as completed Oct 9, 2022
@p00f
Copy link
Author

p00f commented Oct 9, 2022

Shouldn't this use a split instead of nvim-notify? It's too big to fit in a float notification, even if it occupies the whole screen

@Shougo
Copy link
Contributor

Shougo commented Oct 9, 2022

Hm... If the messages are too long, it should be outputted in buffer.
But it takes to time to implement it.

@Shougo
Copy link
Contributor

Shougo commented Oct 9, 2022

Digraphs in a new buffer (as described in readme: "no more more-prompt")

I think it does not mean "it outputs the buffer automatically".
noice.nvim is not magic.

@folke
Copy link
Owner

folke commented Oct 9, 2022

You can add a route for this using the min_width filter (update noice, just added this new filter):

  local noice = require("noice")
  noice.setup({
    routes = {
      {
        view = "split",
        filter = { min_width = 500 },
      },
    },
  })

This will send messages wider than 500 characters to the split window

@folke
Copy link
Owner

folke commented Oct 9, 2022

Will think about changing some of the default routes to handle this by default, but the config above fixes it

@p00f
Copy link
Author

p00f commented Oct 9, 2022

Thanks!

@p00f
Copy link
Author

p00f commented Dec 19, 2022

Doing this skips the space after the number though:
noice with split filter:

image

vanilla nvim:
image

(this means the alignment of the "table" is wrong too)

@folke
Copy link
Owner

folke commented Dec 19, 2022

@p00f that's a Neovim issue. There's other commands right now that also display wrong like :map. I opened an issue about the map one already

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

No branches or pull requests

3 participants