all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Minimal patch: Use link face as default for button face
@ 2006-06-25  8:09 Lennart Borgman
  2006-06-25 18:38 ` Eli Zaretskii
  0 siblings, 1 reply; 10+ messages in thread
From: Lennart Borgman @ 2006-06-25  8:09 UTC (permalink / raw)


Now that there is a face called 'link should not this be used in button.el?


diff -u -r1.21 button.el
--- button.el   6 Feb 2006 14:33:32 -0000       1.21
+++ button.el   25 Jun 2006 08:07:32 -0000
@@ -52,9 +52,8 @@
 ;; Globals

 ;; Use color for the MS-DOS port because it doesn't support underline.
-(defface button '((((type pc) (class color))
-                  (:foreground "lightblue"))
-                 (t :underline t))
+(defface button
+  '((t :inherit link))
   "Default face used for buttons."
   :group 'basic-faces)

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-25  8:09 Minimal patch: Use link face as default for button face Lennart Borgman
@ 2006-06-25 18:38 ` Eli Zaretskii
  2006-06-25 19:44   ` Lennart Borgman
  2006-06-26 11:33   ` Richard Stallman
  0 siblings, 2 replies; 10+ messages in thread
From: Eli Zaretskii @ 2006-06-25 18:38 UTC (permalink / raw)
  Cc: emacs-devel

> Date: Sun, 25 Jun 2006 10:09:22 +0200
> From: Lennart Borgman <lennart.borgman.073@student.lu.se>
> 
> Now that there is a face called 'link should not this be used in button.el?
> 
> 
> diff -u -r1.21 button.el
> --- button.el   6 Feb 2006 14:33:32 -0000       1.21
> +++ button.el   25 Jun 2006 08:07:32 -0000
> @@ -52,9 +52,8 @@
>  ;; Globals
> 
>  ;; Use color for the MS-DOS port because it doesn't support underline.
> -(defface button '((((type pc) (class color))
> -                  (:foreground "lightblue"))
> -                 (t :underline t))
> +(defface button
> +  '((t :inherit link))
>    "Default face used for buttons."
>    :group 'basic-faces)

This modifies the face default for (type pc).  I don't think it's
time to change defaults now.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-25 18:38 ` Eli Zaretskii
@ 2006-06-25 19:44   ` Lennart Borgman
  2006-06-26  3:27     ` Eli Zaretskii
  2006-06-26 11:33   ` Richard Stallman
  1 sibling, 1 reply; 10+ messages in thread
From: Lennart Borgman @ 2006-06-25 19:44 UTC (permalink / raw)
  Cc: emacs-devel

Eli Zaretskii wrote:
>> Date: Sun, 25 Jun 2006 10:09:22 +0200
>> From: Lennart Borgman <lennart.borgman.073@student.lu.se>
>>
>> Now that there is a face called 'link should not this be used in button.el?
>>
>>
>> diff -u -r1.21 button.el
>> --- button.el   6 Feb 2006 14:33:32 -0000       1.21
>> +++ button.el   25 Jun 2006 08:07:32 -0000
>> @@ -52,9 +52,8 @@
>>  ;; Globals
>>
>>  ;; Use color for the MS-DOS port because it doesn't support underline.
>> -(defface button '((((type pc) (class color))
>> -                  (:foreground "lightblue"))
>> -                 (t :underline t))
>> +(defface button
>> +  '((t :inherit link))
>>    "Default face used for buttons."
>>    :group 'basic-faces)
>>     
>
> This modifies the face default for (type pc).  I don't think it's
> time to change defaults now.
>   
Would it not be better then to move this default to the face "link"? 
What is the benefit of not doing this now?

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-25 19:44   ` Lennart Borgman
@ 2006-06-26  3:27     ` Eli Zaretskii
  0 siblings, 0 replies; 10+ messages in thread
From: Eli Zaretskii @ 2006-06-26  3:27 UTC (permalink / raw)
  Cc: emacs-devel

> Date: Sun, 25 Jun 2006 21:44:42 +0200
> From: Lennart Borgman <lennart.borgman.073@student.lu.se>
> CC:  emacs-devel@gnu.org
> 
> Would it not be better then to move this default to the face "link"? 

Maybe, I don't know.  Personally, I don't think we should do any such
changes at this time.

> What is the benefit of not doing this now?

Changing such a fundamental face might cause all kinds of unintended
consequences.  We really don't need this now, especially since the
`link' face had its current definition for ages, and no one
complained, AFAIK.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-25 18:38 ` Eli Zaretskii
  2006-06-25 19:44   ` Lennart Borgman
@ 2006-06-26 11:33   ` Richard Stallman
  2006-06-26 13:46     ` Lennart Borgman
  1 sibling, 1 reply; 10+ messages in thread
From: Richard Stallman @ 2006-06-26 11:33 UTC (permalink / raw)
  Cc: lennart.borgman.073, emacs-devel

    > -(defface button '((((type pc) (class color))
    > -                  (:foreground "lightblue"))
    > -                 (t :underline t))
    > +(defface button
    > +  '((t :inherit link))
    >    "Default face used for buttons."
    >    :group 'basic-faces)

    This modifies the face default for (type pc).  I don't think it's
    time to change defaults now.

It makes changes on many kinds of platforms, since `link'
does a lot more than just underline.

Buttons are different from links, so I think this change is probably a
mistake on general principles.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-26 11:33   ` Richard Stallman
@ 2006-06-26 13:46     ` Lennart Borgman
  2006-06-27 16:14       ` Richard Stallman
  0 siblings, 1 reply; 10+ messages in thread
From: Lennart Borgman @ 2006-06-26 13:46 UTC (permalink / raw)
  Cc: Eli Zaretskii, emacs-devel

Richard Stallman wrote:
>     > -(defface button '((((type pc) (class color))
>     > -                  (:foreground "lightblue"))
>     > -                 (t :underline t))
>     > +(defface button
>     > +  '((t :inherit link))
>     >    "Default face used for buttons."
>     >    :group 'basic-faces)
>
>     This modifies the face default for (type pc).  I don't think it's
>     time to change defaults now.
>
> It makes changes on many kinds of platforms, since `link'
> does a lot more than just underline.
>
> Buttons are different from links, so I think this change is probably a
> mistake on general principles.
>   

I am not sure if I am correct, but it seems to me that the basic 
"button" is indeed a link. Compare help buffers for example. The "button 
links" there have the face "button".

"Buttons" are technically different from widgets but to the reader a 
widget link and a button link should look the same in my opinion.

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-26 13:46     ` Lennart Borgman
@ 2006-06-27 16:14       ` Richard Stallman
  2006-06-28  6:52         ` Miles Bader
  0 siblings, 1 reply; 10+ messages in thread
From: Richard Stallman @ 2006-06-27 16:14 UTC (permalink / raw)
  Cc: eliz, emacs-devel

    I am not sure if I am correct, but it seems to me that the basic 
    "button" is indeed a link. Compare help buffers for example. The "button 
    links" there have the face "button".

Without rechecking when buttons are used and when links are used,
I don't know how much this is true.  Would someone like to check?

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-27 16:14       ` Richard Stallman
@ 2006-06-28  6:52         ` Miles Bader
  2006-06-29 12:59           ` Richard Stallman
  0 siblings, 1 reply; 10+ messages in thread
From: Miles Bader @ 2006-06-28  6:52 UTC (permalink / raw)
  Cc: Lennart Borgman, eliz, emacs-devel

Richard Stallman <rms@gnu.org> writes:
>     I am not sure if I am correct, but it seems to me that the basic 
>     "button" is indeed a link. Compare help buffers for example. The "button 
>     links" there have the face "button".
>
> Without rechecking when buttons are used and when links are used,
> I don't know how much this is true.  Would someone like to check?

Well... a button is not "basically a link", as there is no standard
default action associated with buttons.

Probably the _majority_ of buttons used in the Emacs distrution act
basically like links, but there are also certainly those that don't
(e.g., in "international/mule-defs.el" it uses a button for changing the
sort order of a displayed table).

As buttons can be defined hierarchically -- a button's behavior is
(usually) defined by its "button-class", and a button-class can inherit
from a superclass -- maybe a cleaner way would be to add a standard
button-class named "link", and make link-like buttons inherit from it.

Then the "link" button class would use the "link" face by default.

-Miles

-- 
My spirit felt washed.  With blood.  [Eli Shin, on "The Passion of the Christ"]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-28  6:52         ` Miles Bader
@ 2006-06-29 12:59           ` Richard Stallman
  2006-06-30  1:54             ` Miles Bader
  0 siblings, 1 reply; 10+ messages in thread
From: Richard Stallman @ 2006-06-29 12:59 UTC (permalink / raw)
  Cc: lennart.borgman.073, eliz, emacs-devel

    As buttons can be defined hierarchically -- a button's behavior is
    (usually) defined by its "button-class", and a button-class can inherit
    from a superclass -- maybe a cleaner way would be to add a standard
    button-class named "link", and make link-like buttons inherit from it.

    Then the "link" button class would use the "link" face by default.

Could you put that in etc/TODO?

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: Minimal patch: Use link face as default for button face
  2006-06-29 12:59           ` Richard Stallman
@ 2006-06-30  1:54             ` Miles Bader
  0 siblings, 0 replies; 10+ messages in thread
From: Miles Bader @ 2006-06-30  1:54 UTC (permalink / raw)
  Cc: lennart.borgman.073, eliz, emacs-devel

Richard Stallman <rms@gnu.org> writes:
> > Add a standard button-class named "link", and make link-like buttons
> > inherit from it.  Then the "link" button class would use the "link"
> > face by default.
>
> Could you put that in etc/TODO?

Ok I did

-miles

-- 
"Though they may have different meanings, the cries of 'Yeeeee-haw!' and
 'Allahu akbar!' are, in spirit, not actually all that different."

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2006-06-30  1:54 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-06-25  8:09 Minimal patch: Use link face as default for button face Lennart Borgman
2006-06-25 18:38 ` Eli Zaretskii
2006-06-25 19:44   ` Lennart Borgman
2006-06-26  3:27     ` Eli Zaretskii
2006-06-26 11:33   ` Richard Stallman
2006-06-26 13:46     ` Lennart Borgman
2006-06-27 16:14       ` Richard Stallman
2006-06-28  6:52         ` Miles Bader
2006-06-29 12:59           ` Richard Stallman
2006-06-30  1:54             ` Miles Bader

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.