Skip to content
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

Update color scheme word_separators, change LICENSE year #357

Merged
merged 4 commits into from
Feb 26, 2022

Conversation

MattDMo
Copy link
Member

@MattDMo MattDMo commented Oct 30, 2021

I removed # from the color scheme word_separators so hex colors can be completely selected on double-click. I also updated the year in the LICENSE to 2021. This will need to be changed again in 2 months. Maybe somebody can write an action for that...

so hex colors can be selected completely
remove # from word_separators, update year in LICENSE
@@ -5,5 +5,5 @@
{ "selector": "meta.mapping.key punctuation.definition.string.begin" },
],
"extensions": ["sublime-color-scheme"],
"word_separators": "./\\()\"':,.;<>~!@#$%^&*|+=[]{}`~?" // default without '-'
"word_separators": "./\\()\"':,.;<>~!@$%^&*|+=[]{}`~?" // default without '-'

Choose a reason for hiding this comment

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

Update comment, too?

Copy link
Member Author

Choose a reason for hiding this comment

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

Just did

Copy link

@michaelblyons michaelblyons left a comment

Choose a reason for hiding this comment

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

LGTM.

I do wish (now that I think about it) that you could have scope-sensitive word separators. Seems like a big ask of ST core, though.

@MattDMo
Copy link
Member Author

MattDMo commented Nov 5, 2021

Yeah, I don't think we'll be seeing that anytime soon...

@FichteFoll FichteFoll added this to the 3.3.2 milestone Feb 26, 2022
@FichteFoll
Copy link
Member

Thanks, let's hope this isn't too unexpected for users (since CSS doesn't do this either, at least at the moment).

@FichteFoll FichteFoll merged commit ae1fb8a into SublimeText:master Feb 26, 2022
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.

4 participants