(header-line): Change tty-variant to use underlining.

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index d1f4dce915907f7e8316922fc634446779d2dc55..4297e1a4d1112803286be150a70c9f243977b169 100644 (file)
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,5 +1,7 @@
 2000-10-12  Miles Bader  <miles@lsi.nec.co.jp>
 
+       * faces.el (header-line): Change tty-variant to use underlining.
+
        * isearch.el (isearch-set-lazy-highlight-faces-at): New function.
        (isearch-highlight): Restore lazy-isearch face properties at old
        position, and suppress them at new position.

diff --git a/lisp/faces.el b/lisp/faces.el
index fc026154e2114f74b80aff48b9209adfc95b208a..89c3bf73f3cdb93ea6d0a6007ecf253c17b63eb0 100644 (file)
--- a/lisp/faces.el
+++ b/lisp/faces.el
@@ -1535,7 +1535,16 @@ created."
 
 (defface header-line
   '((((type tty))
-     (:inverse-video t))
+     ;; This used to be `:inverse-video t', but that doesn't look very
+     ;; good when combined with inverse-video mode-lines and multiple
+     ;; windows.  Underlining looks better, and is more consistent with
+     ;; the window-system face variants, which deemphasize the
+     ;; header-line in relation to the mode-line face.  If a terminal
+     ;; can't underline, then the header-line will end up without any
+     ;; highlighting; this may be too confusing in general, although it
+     ;; happens to look good with the only current use of header-lines,
+     ;; the info browser. XXX
+     (:underline t))
     (((class color) (background light))
      (:box (:line-width 1 :style released-button)
           :background "grey90" :foreground "grey20"