bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#4234: allow xterm to detect the background color


From: Dan Nicolaescu
Subject: bug#4234: allow xterm to detect the background color
Date: Sat, 22 Aug 2009 10:15:12 -0700 (PDT)

emacs/emacsclient -nw  running in an xterm set the frame background mode
to 'light.   
Not all users use the default white background, so they get the wrong
face colors. 
Newer xterms can tell what the background color is
(see http://permalink.gmane.org/gmane.emacs.devel/111270)

This patch implements reads the background color from xterm (iff it
supports the feature) and sets the background mode accordingly.

--- xterm.el.~1.62.~    2009-01-08 14:24:21.000000000 -0800
+++ xterm.el    2009-08-22 10:12:38.000000000 -0700
@@ -462,8 +462,6 @@
       (set-keymap-parent input-decode-map map)))
 
     (xterm-register-default-colors)
-    ;; This recomputes all the default faces given the colors we've just set 
up.
-    (tty-set-up-initial-frame-faces)
     
     ;; Try to turn on the modifyOtherKeys feature on modern xterms.
     ;; When it is turned on many more key bindings work: things like
@@ -472,7 +470,8 @@
     ;; modifyOtherKeys. At this time only xterm does.
     (let ((coding-system-for-read 'binary)
          (chr nil)
-         (str nil))
+         (str nil)
+         version)
       ;; Pending input can be mistakenly returned by the calls to
       ;; read-event below.  Discard it.
       (discard-input)
@@ -494,8 +493,23 @@
            ;; NUMBER2 is the xterm version number, look for something
            ;; greater than 216, the version when modifyOtherKeys was
            ;; introduced.
-           (when (>= (string-to-number
-                      (substring str (match-beginning 1) (match-end 1))) 216)
+           (setq version (string-to-number
+                          (substring str (match-beginning 1) (match-end 1))))
+           ;; xterm version 242 supports reporting the background
+           ;; color, maybe earlier versions do too...
+           (when (>= version 242)
+             (send-string-to-terminal "\e]11;?\e\\")
+             (when (equal (read-event nil nil 2) ?\e)
+               (when (equal (read-event nil nil 2) ?\])
+                 (setq str "")
+                 (while (not (equal (setq chr (read-event nil nil 2)) ?\\))
+                   (setq str (concat str (string chr))))
+                 (when (string-match 
"11;rgb:\\([a-f0-9]+\\)/\\([a-f0-9]+\\)/\\([a-f0-9]+\\)" str)
+                   (xterm-set-background-mode
+                    (string-to-number (match-string 1 str) 16)
+                    (string-to-number (match-string 2 str) 16)
+                    (string-to-number (match-string 3 str) 16))))))
+           (when (>= version 216)
              ;; Make sure that the modifyOtherKeys state is restored when
              ;; suspending, resuming and exiting.
              (add-hook 'suspend-hook 'xterm-turn-off-modify-other-keys)
@@ -508,6 +522,9 @@
                    xterm-modify-other-keys-terminal-list)
              (xterm-turn-on-modify-other-keys))))))
 
+    ;; This recomputes all the default faces given the colors we've just set 
up.
+    (tty-set-up-initial-frame-faces)
+
     (run-hooks 'terminal-init-xterm-hook))
 
 ;; Set up colors, for those versions of xterm that support it.
@@ -649,5 +666,11 @@ versions of xterm."
          (delq terminal xterm-modify-other-keys-terminal-list))
     (send-string-to-terminal "\e[>4m" terminal)))
 
+(defun xterm-set-background-mode (redc greenc bluec)
+  ;; Use the heuristic in `frame-set-background-mode' to decide if a
+  ;; frame is dark.
+  (when (< (+ redc greenc bluec) (* .6 (+ 65535 65535 65535)))
+    (set-terminal-parameter nil 'background-mode 'dark)))
+
 ;; arch-tag: 12e7ebdd-1e6c-4b25-b0f9-35ace25e855a
 ;;; xterm.el ends here






reply via email to

[Prev in Thread] Current Thread [Next in Thread]