* Patch: Replacing the style of the More/Hide Rest button in Customize
@ 2006-02-18 23:17 Lennart Borgman
2006-02-18 23:59 ` Chong Yidong
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Lennart Borgman @ 2006-02-18 23:17 UTC (permalink / raw)
[-- Attachment #1: Type: text/plain, Size: 260 bytes --]
The attached small patch makes the More/Hide Rest button in customize
look more like a link, which in my opinion is more common (web pages). I
have sent something like it before, but we never got any further because
of problems with my papers at that time.
[-- Attachment #2: wid-edit-more-hide.patch --]
[-- Type: text/plain, Size: 1491 bytes --]
Index: lisp/wid-edit.el
===================================================================
RCS file: /cvsroot/emacs/emacs/lisp/wid-edit.el,v
retrieving revision 1.166
diff -c -r1.166 wid-edit.el
*** lisp/wid-edit.el 16 Feb 2006 15:58:32 -0000 1.166
--- lisp/wid-edit.el 18 Feb 2006 23:08:34 -0000
***************
*** 124,129 ****
--- 124,136 ----
:type 'face
:group 'widget-faces)
+ (defface widget-link-face
+ `((((class color) (background light)) (:foreground "blue" :underline t))
+ (((class color) (background dark)) (:foreground "cyan" :underline t)))
+ "Face used to highlight attributes that are links."
+ :group 'custom-faces)
+
+
;; TTY gets special definitions here and in the next defface, because
;; the gray colors defined for other displays cause black text on a black
;; background, at least on light-background TTYs.
***************
*** 2917,2924 ****
(widget-create-child-and-convert
widget 'visibility
:help-echo "Show or hide rest of the documentation."
! :on "Hide Rest"
! :off "More"
:always-active t
:action 'widget-parent-action
shown))
--- 2924,2934 ----
(widget-create-child-and-convert
widget 'visibility
:help-echo "Show or hide rest of the documentation."
! :on "<<"
! :off "More..."
! :button-face 'widget-link-face
! :button-prefix ""
! :button-suffix ""
:always-active t
:action 'widget-parent-action
shown))
[-- Attachment #3: Type: text/plain, Size: 142 bytes --]
_______________________________________________
Emacs-devel mailing list
Emacs-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-devel
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Patch: Replacing the style of the More/Hide Rest button in Customize
2006-02-18 23:17 Patch: Replacing the style of the More/Hide Rest button in Customize Lennart Borgman
@ 2006-02-18 23:59 ` Chong Yidong
2006-02-19 0:09 ` Lennart Borgman
2006-02-19 17:17 ` Juri Linkov
2006-02-19 22:47 ` Richard M. Stallman
2 siblings, 1 reply; 9+ messages in thread
From: Chong Yidong @ 2006-02-18 23:59 UTC (permalink / raw)
Cc: Emacs Devel
Lennart Borgman <lennart.borgman.073@student.lu.se> writes:
> + (defface widget-link-face
> + `((((class color) (background light)) (:foreground "blue" :underline t))
> + (((class color) (background dark)) (:foreground "cyan" :underline t)))
> + "Face used to highlight attributes that are links."
> + :group 'custom-faces)
Probably better to inherit the `link' face.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Patch: Replacing the style of the More/Hide Rest button in Customize
2006-02-18 23:59 ` Chong Yidong
@ 2006-02-19 0:09 ` Lennart Borgman
0 siblings, 0 replies; 9+ messages in thread
From: Lennart Borgman @ 2006-02-19 0:09 UTC (permalink / raw)
Cc: Emacs Devel
Chong Yidong wrote:
> Lennart Borgman <lennart.borgman.073@student.lu.se> writes:
>
>
>> + (defface widget-link-face
>> + `((((class color) (background light)) (:foreground "blue" :underline t))
>> + (((class color) (background dark)) (:foreground "cyan" :underline t)))
>> + "Face used to highlight attributes that are links."
>> + :group 'custom-faces)
>>
>
> Probably better to inherit the `link' face.
>
Thanks, yes you are right. It seems like Juri added that face on
2006-01-23. I wrote this patch long ago.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Patch: Replacing the style of the More/Hide Rest button in Customize
2006-02-18 23:17 Patch: Replacing the style of the More/Hide Rest button in Customize Lennart Borgman
2006-02-18 23:59 ` Chong Yidong
@ 2006-02-19 17:17 ` Juri Linkov
2006-02-19 21:06 ` Lennart Borgman
2006-02-19 22:47 ` Richard M. Stallman
2 siblings, 1 reply; 9+ messages in thread
From: Juri Linkov @ 2006-02-19 17:17 UTC (permalink / raw)
Cc: emacs-devel
> The attached small patch makes the More/Hide Rest button in customize look
> more like a link, which in my opinion is more common (web pages). I have
> sent something like it before, but we never got any further because of
> problems with my papers at that time.
Are you sure the "More/Hide Rest" button is a link? To me it looks
like a typical button where pushing performs some action. A link for me
is something that leads to another place.
--
Juri Linkov
http://www.jurta.org/emacs/
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Patch: Replacing the style of the More/Hide Rest button in Customize
2006-02-19 17:17 ` Juri Linkov
@ 2006-02-19 21:06 ` Lennart Borgman
2006-02-20 0:52 ` Juri Linkov
0 siblings, 1 reply; 9+ messages in thread
From: Lennart Borgman @ 2006-02-19 21:06 UTC (permalink / raw)
Cc: emacs-devel
Juri Linkov wrote:
>> The attached small patch makes the More/Hide Rest button in customize look
>> more like a link, which in my opinion is more common (web pages). I have
>> sent something like it before, but we never got any further because of
>> problems with my papers at that time.
>>
>
> Are you sure the "More/Hide Rest" button is a link? To me it looks
> like a typical button where pushing performs some action. A link for me
> is something that leads to another place.
>
It is a matter of taste. Or maybe perception. I think the Customize
buffers are easier to view with "More/Hide Rest" looking like links. And
I do not think it is hard to understand the meaning, I have seen this
usage in many web pages.
I got some positive comments about this change before but there might be
others who disagree of course.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Patch: Replacing the style of the More/Hide Rest button in Customize
2006-02-18 23:17 Patch: Replacing the style of the More/Hide Rest button in Customize Lennart Borgman
2006-02-18 23:59 ` Chong Yidong
2006-02-19 17:17 ` Juri Linkov
@ 2006-02-19 22:47 ` Richard M. Stallman
2006-02-19 22:49 ` Lennart Borgman
2 siblings, 1 reply; 9+ messages in thread
From: Richard M. Stallman @ 2006-02-19 22:47 UTC (permalink / raw)
Cc: emacs-devel
The attached small patch makes the More/Hide Rest button in customize
look more like a link, which in my opinion is more common (web pages).
They are not links, so why should they look like links?
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Patch: Replacing the style of the More/Hide Rest button in Customize
2006-02-19 22:47 ` Richard M. Stallman
@ 2006-02-19 22:49 ` Lennart Borgman
2006-02-20 18:42 ` Richard M. Stallman
0 siblings, 1 reply; 9+ messages in thread
From: Lennart Borgman @ 2006-02-19 22:49 UTC (permalink / raw)
Cc: emacs-devel
Richard M. Stallman wrote:
> The attached small patch makes the More/Hide Rest button in customize
> look more like a link, which in my opinion is more common (web pages).
>
> They are not links, so why should they look like links?
>
Because I believe this makes customize buffers more readable. And it is
a rather common usage on web pages.
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Patch: Replacing the style of the More/Hide Rest button in Customize
2006-02-19 21:06 ` Lennart Borgman
@ 2006-02-20 0:52 ` Juri Linkov
0 siblings, 0 replies; 9+ messages in thread
From: Juri Linkov @ 2006-02-20 0:52 UTC (permalink / raw)
Cc: emacs-devel
>> Are you sure the "More/Hide Rest" button is a link? To me it looks
>> like a typical button where pushing performs some action. A link for me
>> is something that leads to another place.
>>
> It is a matter of taste. Or maybe perception. I think the Customize
> buffers are easier to view with "More/Hide Rest" looking like links. And
> I do not think it is hard to understand the meaning, I have seen this
> usage in many web pages.
I've never seen a link that hides/shows parts of a page. Instead of that,
usually the following icons indicate a place to click to show/hide:
[+] - show
[-] - hide
[>] - show
[v] - hide
You can see how good this looks in customization buffers after copying
emacs/etc/images/ezimage/doc-plus.xpm to emacs/etc/images/right.xpm, and
emacs/etc/images/ezimage/doc-minus.xpm to emacs/etc/images/down.xpm
in your Emacs source tree and restarting Emacs.
If you don't like these icons then you can try copying
emacs/etc/images/ezimage/tag-gt.xpm to emacs/etc/images/right.xpm, and
emacs/etc/images/ezimage/tag-v.xpm to emacs/etc/images/down.xpm.
--
Juri Linkov
http://www.jurta.org/emacs/
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: Patch: Replacing the style of the More/Hide Rest button in Customize
2006-02-19 22:49 ` Lennart Borgman
@ 2006-02-20 18:42 ` Richard M. Stallman
0 siblings, 0 replies; 9+ messages in thread
From: Richard M. Stallman @ 2006-02-20 18:42 UTC (permalink / raw)
Cc: emacs-devel
Because I believe this makes customize buffers more readable. And it is
a rather common usage on web pages.
I think we should maintain the distinction between links
and buttons that really do something.
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2006-02-20 18:42 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-02-18 23:17 Patch: Replacing the style of the More/Hide Rest button in Customize Lennart Borgman
2006-02-18 23:59 ` Chong Yidong
2006-02-19 0:09 ` Lennart Borgman
2006-02-19 17:17 ` Juri Linkov
2006-02-19 21:06 ` Lennart Borgman
2006-02-20 0:52 ` Juri Linkov
2006-02-19 22:47 ` Richard M. Stallman
2006-02-19 22:49 ` Lennart Borgman
2006-02-20 18:42 ` Richard M. Stallman
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.