nvim-ts-rainbow icon indicating copy to clipboard operation
nvim-ts-rainbow copied to clipboard

Add more languages to levels.lua

Open brymer-meneses opened this issue 3 years ago • 21 comments

I have configured this plugin to show custom colors and I want to make them appear in a certain order. However, I get random order of appearance.

Screenshot from 2021-05-07 06-56-17

brymer-meneses avatar May 06 '21 22:05 brymer-meneses

Is this a bug or a known feature?

brymer-meneses avatar May 06 '21 23:05 brymer-meneses

It's not random per se, it just uses all nodes to calculate nesting depth (instead of only brackets) so it appears random

This was adressed by #25 but I reverted it for other reasons, I'll add it again when #29 is complete.

I'm keeping this open as a reminder, thank you.

p00f avatar May 07 '21 01:05 p00f

Any updates on this one?

brymer-meneses avatar May 17 '21 22:05 brymer-meneses

I'll try to do it this week

p00f avatar May 18 '21 02:05 p00f

Thank you! Let me know if I can help!

brymer-meneses avatar May 18 '21 13:05 brymer-meneses

Sorry I got caught up in acads, I'll get to it now

p00f avatar May 26 '21 03:05 p00f

Sorry I got caught up in acads, I'll get to it now

Dude I'm not supporting you or anything, you're working for free. It's okay lol, I'm grateful for your work!

brymer-meneses avatar May 26 '21 05:05 brymer-meneses

You can try out the colornum branch now, I've done lua, c++ and rust.

Let me know if I can help!

Can you test if there's a performance hit against master ? Maybe large C++ files like firefox source

p00f avatar May 26 '21 07:05 p00f

before

image

after

image

p00f avatar May 26 '21 12:05 p00f

You can try out the colornum branch now, I've done lua, c++ and rust.

It's a hit-or-miss.

2021-05-26_23-00

The brackets at the bottom of my screenshot are okay, however the colors at the top are not as expected.

brymer-meneses avatar May 26 '21 15:05 brymer-meneses

There's nothing much I can do for lua though - it's a parser limitation. To highlight print(1+(2+(3))) correctly I'll have to count binary_operation since there is no parenthesized_expression node like java here. This will now do

image

because it counts the concatenation as a level

c++ and rust on the other hand look much better

p00f avatar May 26 '21 15:05 p00f

Similarly there's no node for indexing so this happens

image

(which is maybe fine here because the shapes of the brackets are different)

p00f avatar May 26 '21 15:05 p00f

I guess we'll just have to live with this minor inconvenience then.

Thanks for fixing this!

brymer-meneses avatar May 26 '21 23:05 brymer-meneses

You can try out the colornum branch now, I've done lua, c++ and rust.

Let me know if I can help!

Can you test if there's a performance hit against master ? Maybe large C++ files like firefox source

I have opened a cpp file with 2500 lines taken from tensorflow, there was a slight lag, but not that bad imo.

brymer-meneses avatar May 27 '21 00:05 brymer-meneses

I've merged the colornum branch into master (currently with rust, cpp and fennel queries). Now we only need to add levels for more languages from #25 (and change if needed)

p00f avatar Jun 01 '21 15:06 p00f

I won't be able to do stuff in the coming weeks, feel free to submit PRs

p00f avatar Jun 01 '21 15:06 p00f

Just add the rainbow.level nodes from the PR into the respective table in levels.lua like so: image

p00f avatar Jun 01 '21 15:06 p00f

Stale issue message

github-actions[bot] avatar Aug 01 '21 02:08 github-actions[bot]

Is this still being worked on? I'm trying to mirror my colour settings from Sublime Text and VSCode, so I have put the colours in the same order in my lua config file. However, Python files (at least) are not colouring in the same order as they are in my config file.

karimlevallois avatar Aug 16 '21 18:08 karimlevallois

Is this still being worked on?

Not actively, I (and other contributors) fix this when I notice something

p00f avatar Aug 19 '21 07:08 p00f

Lua is fixed with the new parser

p00f avatar Jan 23 '22 12:01 p00f