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

2 files changed, 12 insertions, 1 deletions

diff --git a/lisp/ChangeLog b/lisp/ChangeLog
index d1f4dce9159..4297e1a4d11 100644
--- 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 fc026154e21..89c3bf73f3c 100644
--- 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"