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

Additional Text Attributes #322

Merged
merged 8 commits into from
Jun 11, 2016
Merged

Additional Text Attributes #322

merged 8 commits into from
Jun 11, 2016

Conversation

KronicDeth
Copy link
Owner

Resolves #299

Changelog

Enhancements

  • Default text attributes for "Darcula" and "Default" themes: almost every Text Attribute Key has a unique hue for the Foreground color.
  • Explain how to add additionalTextAttributes to plugin.xml in CONTRIBUTING.md
  • Group Textual Text Attribute Keys Together: Next "Textual" group is created and "Character List", "Escape Sequence", "Sigil", and "String" are moved under the group.
  • Describe relations between different text attributes in COLOR_SCHEMA_DESIGN.xml, so they can be applied to different base schemes, such as applying the current Darcula additonalTextAttributes to Default.

Incompatible Changes

  • "Character List", "Escape Sequence", "Sigil", and "String" are moved under the new "Textual" group.

* Character List
* Escape Sequence
* Sigil
* String
Increase constrast of Variable with Braces and Operators while keeping
relation between it and Parameter and Type Parameter the same.
Describe relations between different text attributes, so they can be
applied to different base schemes, such as applying the current Darcula
additonalTextAttributes to Default.
@KronicDeth KronicDeth merged commit b65be01 into master Jun 11, 2016
@KronicDeth KronicDeth deleted the 299 branch June 11, 2016 04:25
KronicDeth added a commit that referenced this pull request Jun 12, 2016
KronicDeth added a commit that referenced this pull request Jun 14, 2016
@KronicDeth KronicDeth added this to the v4.0.0 milestone Jun 15, 2016
@KronicDeth KronicDeth mentioned this pull request Jul 4, 2016
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.

1 participant