Skip to content

Commit

Permalink
Use different colors for Org-Mode levels (#81)
Browse files Browse the repository at this point in the history
It now uses the whole "Frost" palette.
It used `nord8` for all levels before to match the style of Markdown
syntax in Emacs as well as other projects, but it makes sense for this
package to use different colors since users might expect it.
This also matches the default style of the Org-Mode package that makes
use of different colors.

Additionaly this comit removed a duplicate line.
  • Loading branch information
Casper Weiss Bang authored and arcticicestudio committed Dec 3, 2019
1 parent 52756cd commit e095a86
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions nord-theme.el
Original file line number Diff line number Diff line change
Expand Up @@ -627,14 +627,14 @@
`(neo-vc-user-face ((,class (:foreground ,nord4))))

;; > Org
`(org-level-1 ((,class (:foreground ,nord8 :weight bold))))
`(org-level-2 ((,class (:inherit org-level-1))))
`(org-level-3 ((,class (:inherit org-level-1))))
`(org-level-4 ((,class (:inherit org-level-1))))
`(org-level-5 ((,class (:inherit org-level-1))))
`(org-level-6 ((,class (:inherit org-level-1))))
`(org-level-7 ((,class (:inherit org-level-1))))
`(org-level-8 ((,class (:inherit org-level-1))))
`(org-level-1 ((,class (:foreground ,nord7 :weight extra-bold))))
`(org-level-2 ((,class (:foreground ,nord8 :weight bold))))
`(org-level-3 ((,class (:foreground ,nord9 :weight semi-bold))))
`(org-level-4 ((,class (:foreground ,nord10 :weight normal))))
`(org-level-5 ((,class (:inherit org-level-4))))
`(org-level-6 ((,class (:inherit org-level-4))))
`(org-level-7 ((,class (:inherit org-level-4))))
`(org-level-8 ((,class (:inherit org-level-4)))))
`(org-agenda-structure ((,class (:foreground ,nord9))))
`(org-agenda-date ((,class (:foreground ,nord8 :underline nil))))
`(org-agenda-done ((,class (:foreground ,nord14))))
Expand Down

2 comments on commit e095a86

@rien333
Copy link
Contributor

@rien333 rien333 commented on e095a86 Dec 3, 2019

Choose a reason for hiding this comment

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

Pretty sure this has a syntax error on line 637.

@arcticicestudio
Copy link
Contributor

Choose a reason for hiding this comment

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

@rien333 Good catch 💪
Reviewed and merged #82 to stabilize develop branch again.

Please sign in to comment.