branch: externals/modus-themes
commit f984e5587e13216af2e9ce4799291ddc4e2e0267
Author: Protesilaos Stavrou <[email protected]>
Commit: Protesilaos Stavrou <[email protected]>

    STOP transient.el from trying to do semantic colouring
    
    Read the comment. We cannot give users this experience by default. I
    shall review this once things change.
---
 modus-themes.el | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/modus-themes.el b/modus-themes.el
index a9e88bcc99..d6d7aeaa24 100644
--- a/modus-themes.el
+++ b/modus-themes.el
@@ -3704,7 +3704,18 @@ FG and BG are the main colors."
     `(transient-heading ((,c :inherit bold :foreground ,fg-main)))
     `(transient-inactive-argument ((,c :inherit shadow)))
     `(transient-inactive-value ((,c :inherit shadow)))
+    ;; NOTE 2023-12-09 10:30:09 +0200: The new user option
+    ;; `transient-semantic-coloring' is enabled by default.  This is
+    ;; not good for us, because we are making it harder for users who
+    ;; need accessible colors to use the transient interfaces.  I
+    ;; could set that user option to nil, but I think it is less
+    ;; intrusive to enforce uniformity among the relevant faces.
+    ;; Those who want semantic coloring can modify these faces.
     `(transient-key ((,c :inherit modus-themes-key-binding)))
+    `(transient-key-exit ((,c :inherit modus-themes-key-binding)))
+    `(transient-key-noop ((,c :inherit (shadow modus-themes-key-binding))))
+    `(transient-key-return ((,c :inherit modus-themes-key-binding)))
+    `(transient-key-stay ((,c :inherit modus-themes-key-binding)))
     `(transient-mismatched-key ((,c :underline t)))
     `(transient-nonstandard-key ((,c :underline t)))
     `(transient-pink ((,c :inherit bold :foreground ,magenta)))

Reply via email to