Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Convert to sublime color scheme #44

Merged
merged 40 commits into from
Aug 14, 2019
Merged

Convert to sublime color scheme #44

merged 40 commits into from
Aug 14, 2019

Conversation

laurent-indermuehle
Copy link
Contributor

I would like to propose a new color scheme with :

  • Use variables instead of RGB code, many colors that looked alike have been merged
  • Global variables are set, including the ones introduced in last Sublime builds
  • Hashed Syntax Highlighting for JSON keys
  • Support natively any markups (Markdown and Restructured Text tested)
  • New color palette with enhanced contrast
  • Purple color for misspelled words
  • ... many other small changes

The original color scheme is kept in glacier-legacy.tmTheme

laurent-indermuehle and others added 30 commits July 19, 2019 21:31
…s logic: lighter variants start at 1 and each increment is darker.
… in green and italic instead of orange-red and bold.
…punction was colored) and background of code block with darker tone.
…ause I was prefixing with green. Closes Honiix/glacier-theme#1
…s pink. Added Purple. Removed black-1 -> use blue-dark-2 instead. Rename light to bright.
@joeyfigaro
Copy link
Contributor

Thank you for doing this, @Honiix! I'll take a look at this over lunch.

Copy link
Contributor

@joeyfigaro joeyfigaro left a comment

Choose a reason for hiding this comment

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

I saw one instance of a potential typo, but I'm not familiar with the tools/tech used for the work—thank you so much for your contribution/work! This is a huge update for a project that's been neglected, to say the least. Barring the potential typo, this looks good to me.

"key2": "value2",
}
],
"glosrsay": {
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this guy a typo?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, it's a typo, could be "Glossary". But it's just a sample file used to test the colors. It has no impact on the theme.

Copy link
Contributor

Choose a reason for hiding this comment

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

Cool, thanks for clarifying 💯

@joeyfigaro joeyfigaro merged commit 39740e9 into shovelandsandbox:master Aug 14, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants