Would there be interest in changing the hover/signature help info to wrap function definitions in parentheses? as it is right now vs as i'm suggesting
I think the latter looks more lispy 🙂
me too, which is why i'm asking
I like it
I don't know, kinda of unnecessary 2 more chars (visual noisy)
in emacs for example looks pretty ok
I'd say for vim is more important to support the markdown render
Also bear in mind that doc does this (arglists wrapped in (..`)` but not the function):
i am not interested in mirroring doc, i'm interested in what makes the most sense for clojure-lsp
my feeling is that functions specifically are used in parentheses, so wrapping the doc in parentheses is a way to indicate both the type and the usage
but if eric doesn't like it, then never mind
let em check how it looks
How it looks in emacs at least:
I think I'm ok with that
seems that being more lispy helps understand what is that sym
i can open a PR if you'd like
sure, just make sure to consider the case where we have the cursor in something that's not a call but we show the call:
I like the parens, as it reads naturally for me, as I look at parens all day 🙂
Also it helps vim and other editors syntax highlight to consider as function
i have semantics highlighting turned off and use treesitter for syntax highlighting, in case that matters
Is that :semantic-tokens? false in the config.edn file for clojure-lsp?
yes
but it seems trisitter considers that paren, you can see yours green with the change
what do you think?
Looks good!
cool, i'll open a PR!