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

Comments are hard to read #23

Closed
jcinnamond opened this issue Apr 1, 2017 · 8 comments
Closed

Comments are hard to read #23

jcinnamond opened this issue Apr 1, 2017 · 8 comments

Comments

@jcinnamond
Copy link

Nord is lovely, but I'm finding it hard to read comments with them set to nord3:

nord3

What do you think about changing it to nord9? I can create a PR with the change if you're happy with it, but if everyone prefers the de-emphasized current scheme then I'll just change my local copy:

nord9

@arcticicestudio
Copy link
Contributor

arcticicestudio commented Apr 3, 2017

Thanks for your report and nice to see you like it 👍

It looks like there are general color problems since the background it definitely not part of the Nord color palette. The color of the code also looks strange except the comments which are colored correct. but are hard to read related to the background color.

I'm not a Windows user so you may need to help me out there.

  • Is Emacs running in terminal- or GUI mode? If it runs in terminal mode make sure to set up your terminal emulator as described in the README installation section to make the Nord colors available for the terminal.
  • Can you post a list of your installed Emacs packages so I can try to reproduce it? (I've created some VirtualBox containers with Windows 7 and Windows 10 for several other Nord port projects 😄 )

@mandarvaze
Copy link

mandarvaze commented Apr 5, 2017

I just installed this on macOS
Lovely theme.

But I too have same problem - comments are hard to read (python, if it matters)
Problem can be seen on the syntax packages page (https://github.com/arcticicestudio/nord-emacs#syntax-packages)
Refer to C# screen shot.

@jcinnamond
Copy link
Author

Thanks for replying to this @arcticicestudio. You're right about the background - I had an entry in my .emacs file from customize that was overriding the theme. I've removed that so I now have the proper nord background. I still think that comments are hard to read. It's particularly bad if you highlight a region. Here is the same file with the correct background color and a region highlighted:

nord

@arcticicestudio
Copy link
Contributor

arcticicestudio commented Apr 6, 2017

@mandarvaze The low contrast for comments has been selected specifically to not disturb code flow and keep the focus of the developer on the code. Using other colors for it like e.g. nord7 would completely mix the separation between the code and comments. nord7 is used for classes/types/interfaces/documentation tags and you would be unable to differ if it is part of the running code or just a comment.

@jcinnamond Unfortunately this is related to the fact that it is not possible to use alpha channel values for opacity so Nord Emacs can only make use of the Nord colors.
You may change it to nord6, but this would make it hard to read code which is colorized with the default foreground nord4.
The current selection color is the only color which does not collidate with any code color, its only side-effect is that it makes comments a little bit harder to read, but does not make them totally unvisible 😮

It is totally fine that users do not agree to all highlighting color assignments, everyone has different preferences. That is the reason why themes exist 😉

If you'd like to change individual parts of the theme you don't have to change it in the theme file itself, it will be overriden each time you update your packages.
I'm not a Emac user, but I know from Vim that you can define custom color definitions as soon as a specific theme gets loaded. Maybe you know a better way, but it should work if you override definitions you like to change by adding it to (custom-set-faces in your ~/.emacs.d¸init.el

@jcinnamond
Copy link
Author

Thanks again @arcticicestudio for replying to this and for the great work with Nord. I'll stick to overriding it in my local config.

@mandarvaze
Copy link

@jcinnamond Can you share your local config details ? It would be helpful to those that need it.

@jcinnamond
Copy link
Author

@mandarvaze my full config is here: https://github.com/jcinnamond/emacs

The pertinent bits are:

  (load-theme 'nord t)
  (set-face-attribute 'font-lock-comment-face nil
                      :foreground "#81A1C1") ; nord9
  (set-face-attribute 'vertical-border nil
                      :foreground "#EBCB8B") ; nord13

@arcticicestudio
Copy link
Contributor

I've posted this comment in #32 which is related to this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants