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

Add Customizeable faces #6

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

Conversation

PommesSchranke
Copy link

  • Customizable faces
    apache-mode now defines it's own faces. That way, faces may be
    customized just for the purposes of this mode without affecting
    your entire emacs session.

  • Highlight unknown

    s and variables, too (default: font-lock-warning-face)
    This was done for two reasons:

    • Firstly, errors become more visible in apache config files.
    • Secondly, you may define the '...-unknown-...' faces to
      match the other faces, such that you may use this mode to
      edit files with apache.conf syntax and have syntax highlighting
      there (https://metacpan.org/pod/Config::General for example).

* Customizable faces
  apache-mode now defines it's own faces.  That way, faces may be
  customized just for the purposes of this mode without affecting
  your entire emacs session.

* Highlight unknown <SECTION>s and variables, too
  This was done for two reasons:
  - Firstly, errors become more visible in apache config files.
  - Secondly, you may define the '...-unknown-...' faces to
    match the other faces, such that you may use this mode to
    edit files with apache.conf syntax and have syntax highlighting
    there (https://metacpan.org/pod/Config::General for example).
@zonuexe
Copy link
Member

zonuexe commented Jun 23, 2022

@PommesSchranke Thank you for your contribution.
I will check it later and merge it. Adjustments may be needed for the matcher.

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.

2 participants