Skip to content

Conversation

keith-hall
Copy link
Member

ST build 3158 added a couple of new color scheme "global" keys. This PR adds syntax highlighting and completions for those.

@wbond
Copy link
Member

wbond commented Mar 30, 2018

P.S. There is another new setting fold_marker that controls the color of the fold marker.

@keith-hall
Copy link
Member Author

Thanks, I missed that :) I may have to find a more reliable documentation (/website) differ, because https://github.com/maliayas/SublimeText_Documentation clearly isn't automated ;)

Copy link
Member

@FichteFoll FichteFoll left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had some time to kill while traveling. Of course there had to be a new release during my vacation ;)

Will catch up on stuff when I'm back home.
Thanks for the PR.

// ^^^^^^^^^^^^^^^^^^^^^^ entity.name.globals.sublime-color-scheme
// ^^^ meta.mapping.value.json string.quoted.double.json
// ^ constant.numeric.json
"selection_border": "#ffffff40",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is missing teas.

Copy link
Member

@FichteFoll FichteFoll Apr 3, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was obviously meant to be "tests".

I'll just add them myself, unless you beat me to it.

3: punctuation.definition.string.end.json
set: [expect-color-string-value, expect-colon]
- match: (")(shadow_width|popup_css|phantom_css)(")
- match: (")(shadow_width|popup_css|phantom_css|selection_corner_radius|selection_border_width)(")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suppose the css keys should get their own match with this change, no?

@FichteFoll FichteFoll merged commit a3c8066 into SublimeText:master Jun 8, 2018
@keith-hall keith-hall deleted the 3158_color_scheme branch June 8, 2018 20:05
@FichteFoll FichteFoll added this to the 3.1.6 milestone Oct 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants