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

doom-rouge-brighter-comments does not change comment colour #778

Closed
crestofthebeast opened this issue Mar 17, 2023 · 3 comments
Closed

doom-rouge-brighter-comments does not change comment colour #778

crestofthebeast opened this issue Mar 17, 2023 · 3 comments
Labels
is:bug Something isn't working as intended theme:rouge
Milestone

Comments

@crestofthebeast
Copy link

Theme(s)

doom-rouge

What were you expecting?

(setq doom-rouge-brighter-comments t) to enable a brighter comment colour

What actually happened?

i get default(?) comment behaviour/colours

some screenshots. each was taken after changing the (setq doom-rouge-brighter-comments ) line, then saving/tangling, hitting space h r r, and restarting emacs.
image
image

Steps to reproduce

  1. switch to doom-rouge theme
  2. setq doom-rouge-brighter-comments to t
  3. save config.org/el
  4. restart emacs
  5. open a file with comments in it
  6. weep, for you wanted brighter comments

Operating system

macOS Ventura 13.2.1

Emacs version

GNU Emacs 28.2 (builh 1, aarch-apple-darwin22.3.0, Carbon Version 169 Appkit 2299.4) of 2023-02-25

Installed commit of doom-themes

should be 4c239db

@crestofthebeast crestofthebeast added is:bug Something isn't working as intended needs-triage Issue hasn't been assessed yet labels Mar 17, 2023
@hlissner hlissner added theme:rouge and removed needs-triage Issue hasn't been assessed yet labels Mar 18, 2023
@hlissner hlissner added this to Triage Mar 18, 2023
@hlissner hlissner moved this to Confirmed in Triage Mar 18, 2023
@hlissner
Copy link
Member

Ah. Seems the theme never uses the variables. I'll get around to fixing this when I can, but PRs are welcome, in the meantime.

@crestofthebeast
Copy link
Author

will try to give a PR a crack later today, since i've been meaning to learn doom's theme structure.

@hlissner
Copy link
Member

hlissner commented Sep 16, 2023

Do you have any recommendations for what color to use for the brighter comments? I'm playing with violet, or a violet+red blend, but I'm not sure it fits the asthetic (or is bright enough):

base5 (original)
image

violet
image

violet + red
image

@hlissner hlissner moved this from Confirmed to Resolved in Triage Sep 19, 2023
@hlissner hlissner added this to the v2.4.0 milestone Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:bug Something isn't working as intended theme:rouge
Projects
Archived in project
Development

No branches or pull requests

2 participants