unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: Diff mode faces
Date: Fri, 17 Jun 2005 16:34:28 +0200	[thread overview]
Message-ID: <uk6kt128b.fsf@gnu.org> (raw)
In-Reply-To: <87oea5urwm.fsf@jurta.org> (message from Juri Linkov on Fri, 17 Jun 2005 14:47:05 +0300)

> From: Juri Linkov <juri@jurta.org>
> Date: Fri, 17 Jun 2005 14:47:05 +0300
> 
> The default diff face used to highlight changed lines on tty
> (magenta/yellow bold italic) makes an indigestible fruit salad.

In your opinion, perhaps.  These faces are there since 5 years ago,
and I don't recall anyone complaining.

> OTOH, the same face has no highlighting under X.  This makes
> sense, because changed lines are the primary text in diff files
> that doesn't need special highlighting.

They need highlighting to stand out against the unchanged lines.
Since most tty's don't have too many colors, they cannot make use of
the technique similar to what we do on X, which is to dim the context
lines by using light-grey colors for them.

> I propose instead of highlighting whole lines on tty to use at least
> the same approach as introduced recently for comments on tty to highlight
> only comment delimiters in new face font-lock-comment-delimiter-face.
> Similarly, only diff indicators (the first character of the line) could
> be highlighted in diff buffers on tty.

I don't think this is a good idea to make such a change.  Certainly
not now, before a release.  Haven't we already seen enough trouble
from this comment-delimiter face innovation?

Anyway, the results look IMHO ugly.  Partial highlighting of lines
generally looks ugly, and this case is no exception.

(I could never understand the urge to modify the default colors.  If
you dislike the defaults, customize the faces and be done with them.)

  parent reply	other threads:[~2005-06-17 14:34 UTC|newest]

Thread overview: 59+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-17 11:47 Diff mode faces Juri Linkov
2005-06-17 13:11 ` Jason Rumney
2005-06-17 14:28 ` Stefan Monnier
2005-06-18 13:54   ` Juri Linkov
2005-06-17 14:34 ` Eli Zaretskii [this message]
2005-06-18 13:57   ` Juri Linkov
2005-06-18 15:27     ` Randal L. Schwartz
2005-06-18 16:46       ` Eli Zaretskii
2005-06-19 13:09         ` Juri Linkov
2005-06-19 19:58           ` Eli Zaretskii
2005-06-20  4:48             ` Juri Linkov
2005-06-20 20:18               ` Eli Zaretskii
2005-06-21 16:28                 ` Background mode (was: Diff mode faces) Juri Linkov
2005-06-27  0:03                   ` Background mode Juri Linkov
2005-06-27 16:46                     ` Richard M. Stallman
2005-06-27 23:52                       ` Juri Linkov
2005-06-28 18:47                         ` Richard M. Stallman
2005-06-29  3:55                           ` Stefan Monnier
2005-06-29 22:21                             ` Miles Bader
2005-06-30 17:45                               ` Stefan Monnier
2005-07-01  4:03                                 ` Richard M. Stallman
2005-07-01 15:01                                   ` Stefan Monnier
2005-06-27 23:55                     ` Juri Linkov
2005-06-19 13:05       ` Diff mode faces Juri Linkov
2005-06-19 17:10         ` Luc Teirlinck
2005-06-19 17:34           ` Randal L. Schwartz
2005-06-27 23:55           ` Juri Linkov
2005-06-28  4:57             ` Miles Bader
2005-07-01 23:59               ` Juri Linkov
2005-07-02  3:37                 ` Miles Bader
2005-07-05 19:11               ` Richard M. Stallman
2005-07-06 20:53                 ` Juri Linkov
2005-07-07  4:05                   ` Miles Bader
2005-07-07  6:03                     ` Juri Linkov
2005-07-07  4:42                   ` Eli Zaretskii
2005-07-09 20:56                     ` Juri Linkov
2005-07-10  3:34                       ` Eli Zaretskii
2005-07-11  0:06                         ` Juri Linkov
2005-07-11 13:43                           ` Stefan Monnier
2005-07-11 19:37                             ` Eli Zaretskii
2005-07-11 19:46                               ` Stefan Monnier
2005-07-12  3:33                                 ` Eli Zaretskii
2005-07-12  6:51                             ` Juri Linkov
2005-07-16 11:17                               ` Eli Zaretskii
2005-06-28 13:10             ` Randal L. Schwartz
2005-06-30 21:30             ` Richard M. Stallman
2005-06-30 21:30             ` Richard M. Stallman
2005-07-01 10:13               ` Eli Zaretskii
2005-07-01 23:59                 ` Juri Linkov
2005-07-03 18:56                   ` xterm colors (was: Diff mode faces) Gaëtan LEURENT
2005-07-04  5:59                     ` Eli Zaretskii
2005-07-04  6:17                     ` Richard M. Stallman
2005-07-01 23:59               ` Diff mode faces Juri Linkov
2005-06-20  0:25         ` Miles Bader
2005-06-18  2:21 ` Richard Stallman
2005-06-18 13:54   ` Juri Linkov
2005-06-19  3:51     ` Richard Stallman
2005-06-19 14:05       ` Juri Linkov
2005-06-20  3:50         ` Richard Stallman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=uk6kt128b.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).