-
-
Notifications
You must be signed in to change notification settings - Fork 113
Add custom elementary syntax styles #1076
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
Conversation
@micahilbery is this supposed to fix the long-standing issue of the highlight cursor colour in the solarised theme being the same? |
@hanaral If you're talking about the bracket highlighting and selection highlighting being the same then yes this fixes that. |
It looks like that the selection style is applied in preference to the bracket-match style so brackets that are both matching and selected only get the selection style. Presumably this behaviour is controlled by |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are pretty cool! The dark style is especially really pretty! I'm going to try to use these more daily and see where I notice any issues :)
One thing I noticed is that we're missing some fields that are provided by the classic
style. Namely the git diff colors for the gutter: https://github.com/GNOME/gtksourceview/blob/master/data/styles/classic.xml#L110
@micahilbery Will you be addressing the issue with gutter colors? |
Yes, I hope to work on it today. Sorry for the delay. My day job and seasonal plans have pretty well wiped out my free time. |
I've updated the style so that we match classic and tested the git gutters. Afaict everything seems to be working as it should. Sorry this took me so long to get to. |
Thanks - I'll try these out for a while to see if any issues arise. |
I've merged this PR into my 'merge-dev-fixes- branch for dog-fooding. Do far I've noticed that it is harder to see the selected text, especially for solarized-light: However, it is easier to distinguish selected brackets from highlighted brackets. Is there a reason why selected text in the dark version is inverted in color whereas in the light version it isn't? |
I can confirm that the gutter colors are now working. |
@micahilbery I'm inclined to merge this as @danirabbit is 👍 Just one question: "Follow System Style" uses the standard "solarized-light" and "solarized-dark" styles - is that intended? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am going to go ahead and merge this as tweaks to themes can be done after further feedback.
This adds custom styles for syntax highlighting based on the elementary color palette.
fixes #29