Nik Revenco

Results 190 comments of Nik Revenco

Current script: ```lua settings.add("color.suffix", "", "Color for when suffix is matched") settings.add("suffix.aliases", ".git=git clone;", "Suffix aliases") local suffixes = {} for suffix_prepend_pair in string.gmatch(settings.get("suffix.aliases"), "([^;]+)") do local suffix = {}...

> This fixes some subtle yet important issues, such as: > > * Didn't respect quotes. > > * Converted all runs of 2 or more spaces into a 1...

> I'm certainly not opposed to this being merged, but when I implemented #8147, I didn't feel the text object was necessary, since you can emulate `match_inner_sub_word` by chaining `move_prev_sub_word_start`...

> Nice feature! I noticed it doesn't look the subword object was added to the match context menu, though. Okay, I just added it

> I would consider that a bug in the wiki. The wiki should not suggest things that aren't merged yet. Fixed by removing from the wiki

> trying this feature out in my fork and loving it! > > except, I'd rather it be a horizontal split. I could also see an argument for `::Replace`, too...

I just fixed it with merge conflicts since https://github.com/helix-editor/helix/pull/10122 added support for hover with multiple language servers. This is how it looks like with multiple language servers: ![image](https://github.com/user-attachments/assets/c1e6d7ef-d795-4ff1-a8ce-d953d4ad4848)

> @nik-rev What do you think about making this work in-place instead of opening a new split? Similar to `goto_definition` when it has to jump to a different file. This...

> This feature seems cool! When is this more useful than simply `gd` which is where the docs come from and has the docs right there? At least for rust...

> Hello! Seeing the many many pull requests, I want to try helping out with some code reviews. I'm new to helix, so weight my review accordingly. > > Could...