Re: Default mm-uu-extract face hard to read on ttys

Re: Default mm-uu-extract face hard to read on ttys

[Reiner Steib]

On Tue, Feb 26 2008, Sascha Wilde wrote:

> The default mm-uu-extract face is hard to read on ttys, as it defaults
> to light yellow on dark green, which gets translated to white on
> (bright) green on 8 color tty.

Could you suggest change to the face definition?  (IIRC Emacs 21
doesn't support `min-colors').

(defface mm-uu-extract '(;; Colors from `gnus-cite-3' plus background:
			 (((class color)
			   (background dark))
			  (:foreground "light yellow"
			   :background "dark green"))
			 (((class color)
			   (background light))
			  (:foreground "dark green"
			   :background "light yellow"))
			 (t
			  ()))
  "Face for extracted buffers."
  ;; See `mm-uu-verbatim-marks-extract'.
  :version "23.0" ;; No Gnus
  :group 'gnus-article-mime)

> It is used for highlighting text in messages which is marked a verbatim:
>
> --8<---------------cut here---------------start------------->8---
> this is an example
> --8<---------------cut here---------------end--------------->8---

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/

Re: Default mm-uu-extract face hard to read on ttys

[Sascha Wilde]

Reiner Steib <reinersteib+gmane@imap.cc> wrote:

> On Tue, Feb 26 2008, Sascha Wilde wrote:
>
>> The default mm-uu-extract face is hard to read on ttys, as it defaults
>> to light yellow on dark green, which gets translated to white on
>> (bright) green on 8 color tty.
>
> Could you suggest change to the face definition?  (IIRC Emacs 21
> doesn't support `min-colors').

While I don't see why we should bother about the features of Emacs 21
for changes in 22/23 I think that testing for tty should be sufficient.

How about something along the lines of:

(defface mm-uu-extract '(;; Colors from `gnus-cite-3' plus background:
			 (((type tty)
			   (class color)
			   (background dark))
			  (:foreground "white"
			   :background "dark blue"))
			 (((type tty)
			   (class color)
			   (background light))
			  (:foreground "dark blue"
			   :background "white"))
			 (((class color)
			   (background dark))
			  (:foreground "light yellow"
			   :background "dark green"))
			 (((class color)
			   (background light))
			  (:foreground "dark green"
			   :background "light yellow"))			 
			 (t
			  ()))
  "Face for extracted buffers."
  ;; See `mm-uu-verbatim-marks-extract'.
  :version "23.0" ;; No Gnus
  :group 'gnus-article-mime)

cheers
sascha
-- 
Sascha Wilde : VI is to EMACS as masturbation is to making love:
             : effective and always available but probably not your
             : first choice...

Re: Default mm-uu-extract face hard to read on ttys

[Bastien]

> Sascha Wilde : VI is to EMACS as masturbation is to making love:
>              : effective and always available but probably not your
>              : first choice...

Nice shot!

I would add: "VI is like cheap sex: inserting is fun, but you always
need to escape at some point."

-- 
Bastien

Re: Default mm-uu-extract face hard to read on ttys

[Sascha Wilde]

Bastien <bzg@altern.org> wrote:

>> Sascha Wilde : VI is to EMACS as masturbation is to making love:
>>              : effective and always available but probably not your
>>              : first choice...

> Nice shot!

Not by me I have to admit -- I shamelessly riped it frome someone elses
sig (unfortunatly I have no idea who was the original author).

> I would add: "VI is like cheap sex: inserting is fun, but you always
> need to escape at some point."

:-)

May I sig that, too?
-- 
Sascha Wilde
... mein Opa [...]  würde an dieser Stelle zu Dir sagen: Junge, such Dir 
ne Frau, bau Dir ein Haus, mach ein Kind und laß' die Finger von dem Zeug,
das Du gerade machst. -- Michael Winklhofer in d.a.e.auktionshaeuser

Re: Default mm-uu-extract face hard to read on ttys

[Bastien]

Sascha Wilde <wilde@sha-bang.de> writes:

> Bastien <bzg@altern.org> wrote:
>
>>> Sascha Wilde : VI is to EMACS as masturbation is to making love:
>>>              : effective and always available but probably not your
>>>              : first choice...
>
>> Nice shot!
>
> Not by me I have to admit -- I shamelessly riped it frome someone elses
> sig (unfortunatly I have no idea who was the original author).
>
>> I would add: "VI is like cheap sex: inserting is fun, but you always
>> need to escape at some point."
>
> :-)
>
> May I sig that, too?

Of course you can.  One has to enter history, whatever the door :)

-- 
Bastien

Re: Default mm-uu-extract face hard to read on ttys

[Reiner Steib]

On Wed, Feb 27 2008, Sascha Wilde wrote:

> Reiner Steib <reinersteib+gmane@imap.cc> wrote:
>> Could you suggest change to the face definition?  (IIRC Emacs 21
>> doesn't support `min-colors').
>
> While I don't see why we should bother about the features of Emacs 21
> for changes in 22/23 

Because No Gnus (aka Gnus 5.13) is supposed to run on Emacs >= 21.
See (info "(gnus)Emacsen").

> I think that testing for tty should be sufficient.
>
> How about something along the lines of:
>
> (defface mm-uu-extract '(;; Colors from `gnus-cite-3' plus background:
> 			 (((type tty)
> 			   (class color)
> 			   (background dark))
> 			  (:foreground "white"
> 			   :background "dark blue"))
> 			 (((type tty)
> 			   (class color)
> 			   (background light))
> 			  (:foreground "dark blue"
> 			   :background "white"))
[...]

I've installed a similar change (in Gnus CVS; will be synced to Emacs
later).  Is this color okay?

(defface mm-uu-extract '(;; Inspired by `gnus-cite-3'
			 (((type tty)
			   (class color)
			   (background dark))
			  (:background "dark blue"))
			 (((class color)
			   (background dark))
			  (:foreground "light yellow"
			   :background "dark green"))
			 (((type tty)
			   (class color)
			   (background light))
			  (:foreground "dark blue"))
			 (((class color)
			   (background light))
			  (:foreground "dark green"
			   :background "light yellow"))
			 (t
			  ()))
  "Face for extracted buffers."
  ;; See `mm-uu-verbatim-marks-extract'.
  :version "23.0" ;; No Gnus
  :group 'gnus-article-mime)

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/

Re: Default mm-uu-extract face hard to read on ttys

[Sascha Wilde]

Reiner Steib <reinersteib+gmane@imap.cc> wrote:

> I've installed a similar change (in Gnus CVS; will be synced to Emacs
> later).  Is this color okay?

Looks good.  Thanx.

cheers
sascha
-- 
Sascha Wilde : "Der Nicht-Denkende glaubt, dass niemand denkt,
             : der Denkende weiss es!"
             : (Gabriel Laub)

Gnus problem with nil font in face definition (was: Default mm-uu-extract face hard to read on ttys)

[Ted Zlatanov]

With the latest Emacs from CVS and the latest Gnus from CVS, many Gnus
faces look strange.  They were fine with checkouts from about a month
ago.

It seems that the cause is faces that have the font set to nil, instead
of unspecified.  I don't know if this is a Gnus-only problem, and it may
be caused by an Emacs problem, so I am cc-ing both emacs-devel and ding.

For example, describe-face on message-header-to says:

Face: message-header-to (sample) (customize this face)
Documentation: Face used for displaying From headers.
Defined in `message.el'.

        Family: unspecified
         Width: unspecified
        Height: unspecified
        Weight: bold
         Slant: unspecified
    Foreground: DarkOliveGreen1
    Background: unspecified
     Underline: unspecified
      Overline: unspecified
Strike-through: unspecified
           Box: unspecified
       Inverse: unspecified
       Stipple: unspecified
          Font: nil
       Fontset: unspecified
       Inherit: unspecified

but message-header-name is:

Face: message-header-name (sample) (customize this face)
Documentation: Face used for displaying header names.
Defined in `message.el'.

        Family: unspecified
         Width: unspecified
        Height: unspecified
        Weight: unspecified
         Slant: unspecified
    Foreground: green
    Background: unspecified
     Underline: unspecified
      Overline: unspecified
Strike-through: unspecified
           Box: unspecified
       Inverse: unspecified
       Stipple: unspecified
          Font: unspecified
       Fontset: unspecified
       Inherit: unspecified

For me, message-header-name appears normal, using the right font from
my X defaults:

#define FONT_ISO_XL -misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
Emacs*font: FONT_ISO_XL

but message-header-to is set in a large proportional font, which looks
ugly.  describe-font crashed my Emacs session, so I'm not sure about the
specifics of the font that was used.

Ted

Re: Gnus problem with nil font in face definition

[Stefan Monnier]

> ugly.  describe-font crashed my Emacs session, so I'm not sure about the
> specifics of the font that was used.

If you can reproduce this, please report it via M-x report-emacs-bug.


        Stefan "who hasn't looked at the rest of your report, yet"

Re: Gnus problem with nil font in face definition

[Ted Zlatanov]

On Thu, 28 Feb 2008 12:51:57 -0500 Stefan Monnier <monnier@iro.umontreal.ca> wrote: 

>> ugly.  describe-font crashed my Emacs session, so I'm not sure about the
>> specifics of the font that was used.

SM> If you can reproduce this, please report it via M-x report-emacs-bug.

I saw it twice this morning and then I couldn't reproduce it again
inside GDB or outside it.  Very strange.

Ted

Re: Gnus problem with nil font in face definition

[Reiner Steib]

On Thu, Feb 28 2008, Ted Zlatanov wrote:

> It seems that the cause is faces that have the font set to nil, instead
> of unspecified.  I don't know if this is a Gnus-only problem, and it may
> be caused by an Emacs problem, so I am cc-ing both emacs-devel and ding.

I'm quite sure that nothing in Gnus/Message has changed in this area,
so it is probably related to the unicode merge (new font back end?)

For those not using Gnus, a test case might be:

- emacs -Q -fn '-misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1'

- Type:
  To: foo@bar.invalid

- M-x message-mode RET

Clean up:
- C-x k yes RET
- rmdir ~/Mail/drafts/    (if it has been created)

Bye, Reiner.
-- 
       ,,,
      (o o)
---ooO-(_)-Ooo---  |  PGP key available  |  http://rsteib.home.pages.de/

Re: Gnus problem with nil font in face definition

[Ted Zlatanov]

On Thu, 28 Feb 2008 21:23:18 +0100 Reiner Steib <reinersteib+gmane@imap.cc> wrote: 

RS> On Thu, Feb 28 2008, Ted Zlatanov wrote:
>> It seems that the cause is faces that have the font set to nil, instead
>> of unspecified.  I don't know if this is a Gnus-only problem, and it may
>> be caused by an Emacs problem, so I am cc-ing both emacs-devel and ding.

RS> I'm quite sure that nothing in Gnus/Message has changed in this area,
RS> so it is probably related to the unicode merge (new font back end?)

RS> For those not using Gnus, a test case might be:

RS> - emacs -Q -fn '-misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1'

RS> - Type:
RS>   To: foo@bar.invalid

RS> - M-x message-mode RET

I tested this and it also shows the wrong font, so it's definitely not a
Gnus issue.

When I tested this earlier I looked at a cperl-mode buffer and it didn't
look like it was using the proportional font.  I checked again now and I
had simply not scrolled down enough to see a face that was affected by
this bug.

Ted

Re: Gnus problem with nil font in face definition

[Kenichi Handa]

In article <v9wsoohlu1.fsf@marauder.physik.uni-ulm.de>, Reiner Steib <reinersteib+gmane@imap.cc> writes:

> On Thu, Feb 28 2008, Ted Zlatanov wrote:
> > It seems that the cause is faces that have the font set to nil, instead
> > of unspecified.  I don't know if this is a Gnus-only problem, and it may
> > be caused by an Emacs problem, so I am cc-ing both emacs-devel and ding.

> I'm quite sure that nothing in Gnus/Message has changed in this area,
> so it is probably related to the unicode merge (new font back end?)

> For those not using Gnus, a test case might be:

> - emacs -Q -fn '-misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1'

I found what is wrong with the current font-backend code.

% xlsfonts -fn -misc-fixed-*-*-*--20-200-75-75-c-100-iso10646-1

lists these fonts.

-misc-fixed-bold-r-normal--20-200-75-75-c-100-iso10646-1
-misc-fixed-bold-r-normal--20-200-75-75-c-100-iso10646-1
-misc-fixed-bold-r-semicondensed--20-200-75-75-c-100-iso10646-1
-misc-fixed-medium-o-normal--20-200-75-75-c-100-iso10646-1
-misc-fixed-medium-o-semicondensed--20-200-75-75-c-100-iso10646-1
-misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
-misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
-misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
-misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
-misc-fixed-medium-r-semicondensed--20-200-75-75-c-100-iso10646-1

but the result of

% xlsfonts -fn -misc-fixed-*-*-*--*-*-*-*-c-*-iso10646-1

doesn't contain this font:
-misc-fixed-bold-r-normal--20-200-75-75-c-100-iso10646-1

I'm now working on handling such a case correctly.
By the way, if you specify this font (i.e. a font whose bold
versoin also exists without auto-scaling):

-misc-fixed-medium-r-normal--18-120-100-100-c-90-iso10646-1

Emacs can find the correct bold version (at least in my
environment).

---
Kenichi Handa
handa@ni.aist.go.jp

Re: Gnus problem with nil font in face definition

[Ted Zlatanov]

On Fri, 29 Feb 2008 16:28:34 +0900 Kenichi Handa <handa@m17n.org> wrote: 

KH> In article <v9wsoohlu1.fsf@marauder.physik.uni-ulm.de>, Reiner Steib <reinersteib+gmane@imap.cc> writes:
>> On Thu, Feb 28 2008, Ted Zlatanov wrote:
>> > It seems that the cause is faces that have the font set to nil, instead
>> > of unspecified.  I don't know if this is a Gnus-only problem, and it may
>> > be caused by an Emacs problem, so I am cc-ing both emacs-devel and ding.

>> I'm quite sure that nothing in Gnus/Message has changed in this area,
>> so it is probably related to the unicode merge (new font back end?)

>> For those not using Gnus, a test case might be:

>> - emacs -Q -fn '-misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1'

KH> I found what is wrong with the current font-backend code.

KH> % xlsfonts -fn -misc-fixed-*-*-*--20-200-75-75-c-100-iso10646-1

KH> lists these fonts.

KH> -misc-fixed-bold-r-normal--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-bold-r-normal--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-bold-r-semicondensed--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-medium-o-normal--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-medium-o-semicondensed--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-medium-r-normal--20-200-75-75-c-100-iso10646-1
KH> -misc-fixed-medium-r-semicondensed--20-200-75-75-c-100-iso10646-1

KH> but the result of

KH> % xlsfonts -fn -misc-fixed-*-*-*--*-*-*-*-c-*-iso10646-1

KH> doesn't contain this font:
KH> -misc-fixed-bold-r-normal--20-200-75-75-c-100-iso10646-1

I don't know the font selection semantics, but this seems
counterintuitive.

KH> I'm now working on handling such a case correctly.
KH> By the way, if you specify this font (i.e. a font whose bold
KH> versoin also exists without auto-scaling):

KH> -misc-fixed-medium-r-normal--18-120-100-100-c-90-iso10646-1

KH> Emacs can find the correct bold version (at least in my
KH> environment).

Was my guess correct that a nil font in the face specification is the
problem?  Or is it specifically a problem with the font I use, as I
think you're saying?  Or both?

Ted