-
Notifications
You must be signed in to change notification settings - Fork 12
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
Dealing with different paren types separately #6
Comments
On Tue, Jan 10, 2012 at 1:24 PM, Sam Aaron <
Oops! Yep, this is something I haven't redone yet. I actually have most of To be honest I'm surprised - at first I thought using separate delimiter I do plan on adding that behaviour in again (it's on the todo) especially Glad you pulled in the mode. I think I'll add independent delimiter colors Thanks,
|
I'm using the mode in C++ and there I have a similar problem. I don't want to color {} as they are quite easy to spot with the indenting scheme, but even if I turn coloring off they still affect the nesting depth of my other delimiters. I found this super-confusing. Do you plan to fix this? |
Development has moved to https://github.com/Fanael/rainbow-delimiters. If the issue still persists, please report it there. |
So, I merged your pull request into the live-coding config, and then went about getting it working with my personal emacs config and I totally love the speed, but one thing that I do have an issue with is that nested parens of different sorts are now mixed.
Before, the colouring of my parens would be determined by the depth of that particular type of paren whereas now the colour is determined regardless of paren type.
( ( <- here I expect the second ( to be my 2nd paren colour for ( parens
( { <- here I expect the first { to be the the 1st paren colour for { parens
Previously I had one colour sequence for ( parens and a separate colour sequence for { [ parens (this is because i was using rainbow-parens for ( and rainbow-delimiters for [ and {
I really liked this behaviour:
Is this something that might be possible in a future update? Otherwise I might be tempted to revert back to the old, (slow) version which I'm very comfortable with.
The text was updated successfully, but these errors were encountered: