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

fix(org): compat with org-modern horizontal rules #819

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TheJJ
Copy link

@TheJJ TheJJ commented Feb 7, 2024

org-modern fontifies horizontal rules using org-modern-horizontal-rule.
doom-themes does so too but using org-meta-line before org-modern does it.

to resolve this conflict, add a switch to disable doom-themes fontifications.

since I introduced a new face, it should not cause any change by default.

Ref: minad/org-modern#59
Ref: minad/org-modern#145


  • I searched the issue tracker and this hasn't been PRed before.
  • My changes are not on the do-not-PR list for this project.
  • My commits conform to the git conventions.
  • My changes are visual; I've included before and after screenshots.
  • I am blindly checking these off.
  • Any relevant issues or PRs have been linked to.
  • This a draft PR; I need more time to finish it.

@TheJJ
Copy link
Author

TheJJ commented Sep 15, 2024

hey @hlissner! could you please have a look if this is alright to fix this doom themes bug?

miniad said in minad/org-modern#59 (comment) that adjustments in org-modern could be made if some knobs are missing there.

…ations

this fixes compatibility with org modern:
org-modern fontifies horizontal rules using org-modern-horizontal-rule.
doom-themes does so too but using org-meta-line, and has precedence.

to resolve this conflict, add a switch to disable doom-themes fontifications.
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

Successfully merging this pull request may close these issues.

1 participant