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

Configuration for italic #89

Merged
merged 1 commit into from
Dec 30, 2017

Conversation

arcticicestudio
Copy link
Contributor

@arcticicestudio arcticicestudio commented Dec 29, 2017

Closes #88

With enabled option for italic comments

Markdown syntax styling

The configuration to enable italic comments (1) is like the name suggests
limited to comments only, but there are other use cases like the
italic style in Markdown (2).

This commit adds a new theme configuration to explicitly enable italics.
The configurations is enabled by default when running in "gui" mode and
is disabled by default in "term" mode.

The reason for this standard behavior is the fact that there are still
shells and terminals which are not able to handle italics. Enabling
italics for terminals by default would cause unpredictable bugs and
graphical glitches.

The new configuration is named "nord_italic".

References:

  (1) https://github.com/arcticicestudio/nord-vim#italic-comments
  (2) #84

GH-88
Copy link
Contributor

@dylnmc dylnmc left a comment

Choose a reason for hiding this comment

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

I suppose that if someone wants to explicitly not have italics in markdown, for instance, then he/she can specify that in an autocmd ColorScheme nord? Better, though. noice

arcticicestudio added a commit that referenced this pull request Dec 30, 2017
The groups "markdownBoldDelimiter" and "markdownItalicDelimiter" for the
delimiter of italic and bold formatted text are now highlighted as
keywords with "nord9". This also ensures that both styles are easily
recognizable even when the "nord_italic" (1) theme configuration is not
enabled.

References:

  (1) #89

GH-90
@arcticicestudio arcticicestudio merged commit dbfc55f into develop Dec 30, 2017
@arcticicestudio arcticicestudio deleted the feature/gh-88-configuration-for-italics branch December 30, 2017 08:52
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.

2 participants