From 5087ed26c5624a95847a83d3e4cbfb2eb2b532d7 Mon Sep 17 00:00:00 2001
From: Craig Jennings <c@cjennings.net>
Date: Sun, 14 Jun 2026 14:22:07 -0500
Subject: refactor(faces): let the theme own face appearance

Strip per-module face configuration so the active theme is the single source of face styling. Removed the dired text-greying set-face-attribute calls (kept the buffer-local face-remap), the org appearance set-face-attribute calls, the highlight-indent-guides colors, the rainbow-delimiters depth colors (which were overriding the theme's blue with red), and the cj/music-* defface forms plus the emms-playlist custom-set-faces. Left nerd-icons' set-face-foreground alone, since it's runtime tint logic rather than static config. Each touched file is backed up beside it as <name>.el.faces.bak (untracked). The cj/music-* faces now come from the theme instead of the module, so until the theme provides them music rendering falls back to the default face.
---
 modules/music-config.el | 48 ------------------------------------------------
 1 file changed, 48 deletions(-)

(limited to 'modules/music-config.el')

diff --git a/modules/music-config.el b/modules/music-config.el
index 799db133..be836429 100644
--- a/modules/music-config.el
+++ b/modules/music-config.el
@@ -722,54 +722,6 @@ For URL tracks: decoded URL."
 
   (setq emms-track-description-function #'cj/music--track-description)
 
-  ;; Playlist faces
-  (defface cj/music-header-face
-    '((((class color) (background dark))
-       (:foreground "#969385"))
-      (((class color) (background light))
-       (:foreground "gray50")))
-    "Face for playlist header labels.")
-
-  (defface cj/music-header-value-face
-    '((((class color) (background dark))
-       (:foreground "#d0cbc0"))
-      (((class color) (background light))
-       (:foreground "gray30")))
-    "Face for playlist header values.")
-
-  (defface cj/music-mode-on-face
-    '((((class color) (background dark))
-       (:foreground "#d7af5f"))
-      (((class color) (background light))
-       (:foreground "DarkGoldenrod")))
-    "Face for active mode indicators in the playlist header.")
-
-  (defface cj/music-mode-off-face
-    '((((class color) (background dark))
-       (:foreground "#58574e"))
-      (((class color) (background light))
-       (:foreground "gray70")))
-    "Face for inactive mode indicators in the playlist header.")
-
-  (defface cj/music-keyhint-face
-    '((((class color) (background dark))
-       (:foreground "#8a9496"))
-      (((class color) (background light))
-       (:foreground "gray50")))
-    "Face for keybinding hints in the playlist header.")
-
-  (custom-set-faces
-   '(emms-playlist-track-face
-     ((((class color) (background dark))
-       (:foreground "#8a9496"))
-      (((class color) (background light))
-       (:foreground "gray50"))))
-   '(emms-playlist-selected-face
-     ((((class color) (background dark))
-       (:foreground "#d7af5f" :weight bold))
-      (((class color) (background light))
-       (:foreground "DarkGoldenrod" :weight bold)))))
-
   ;; Multi-line header overlay
   (defvar-local cj/music--header-overlay nil
     "Overlay displaying the playlist header.")
-- 
cgit v1.2.3