all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* indented meta-info in describe-variable
@ 2007-10-09 10:10 Juanma Barranquero
  2007-10-09 13:33 ` Stefan Monnier
  0 siblings, 1 reply; 6+ messages in thread
From: Juanma Barranquero @ 2007-10-09 10:10 UTC (permalink / raw)
  To: Emacs development discussions, Stefan Monnier

Is this change

2007-10-09  Stefan Monnier  <monnier@iro.umontreal.ca>

        * help-fns.el (describe-variable): Slightly change the layout of
        meta-info to separate it better from the docstring.

responsable of this?

  follow-mode-off-hook is a variable defined in `follow.el'.
  Its value is nil

    This variable is obsolete since 22.2;
  use `follow-mode-hook' instead.

  Documentation:
  [...]


That's not an improvement IMHO.

             Juanma

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

* Re: indented meta-info in describe-variable
  2007-10-09 10:10 indented meta-info in describe-variable Juanma Barranquero
@ 2007-10-09 13:33 ` Stefan Monnier
  2007-10-09 13:55   ` Juanma Barranquero
  2007-10-10  8:49   ` Juanma Barranquero
  0 siblings, 2 replies; 6+ messages in thread
From: Stefan Monnier @ 2007-10-09 13:33 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Emacs development discussions

> 2007-10-09  Stefan Monnier  <monnier@iro.umontreal.ca>

>         * help-fns.el (describe-variable): Slightly change the layout of
>         meta-info to separate it better from the docstring.

> responsable of this?

>   follow-mode-off-hook is a variable defined in `follow.el'.
>   Its value is nil

>     This variable is obsolete since 22.2;
>   use `follow-mode-hook' instead.

>   Documentation:
>   [...]

> That's not an improvement IMHO.

Yes, it's responsible for the added "  " in front of "This" indeed.  And it
should have been responsible for adding "  " in front of "use ..." as well,
but obviously I missed this part.  Thanks for pointing it out.

The reason I made this change is that when a variable has been made
buffer-local and is obsolete and has an safe-local property, the
docstring is pushed too far back for my taste and becomes difficult to
distinguish from that meta-info.


        Stefan

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

* Re: indented meta-info in describe-variable
  2007-10-09 13:33 ` Stefan Monnier
@ 2007-10-09 13:55   ` Juanma Barranquero
  2007-10-10  8:49   ` Juanma Barranquero
  1 sibling, 0 replies; 6+ messages in thread
From: Juanma Barranquero @ 2007-10-09 13:55 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs development discussions

On 10/9/07, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> The reason I made this change is that when a variable has been made
> buffer-local and is obsolete and has an safe-local property, the
> docstring is pushed too far back for my taste and becomes difficult to
> distinguish from that meta-info.

I understand the reason, and I agree with the sentiment. However, the
indenting looks ugly to me when the meta-info is a oneliner.

I suspect most people would hate this even more, but I'd use faces.
That's a good use of font-locking in my view :)

             Juanma

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

* Re: indented meta-info in describe-variable
  2007-10-09 13:33 ` Stefan Monnier
  2007-10-09 13:55   ` Juanma Barranquero
@ 2007-10-10  8:49   ` Juanma Barranquero
  2007-10-10 19:30     ` Stefan Monnier
  1 sibling, 1 reply; 6+ messages in thread
From: Juanma Barranquero @ 2007-10-10  8:49 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs development discussions

Is this difference in whitespace intended?

(defvar test nil "test")
(make-variable-buffer-local 'test)

and then:

------------------------------------------------------------
test's value is nil

  Automatically becomes buffer-local when set in any fashion.

Documentation:
test
------------------------------------------------------------
test's value is t
Local in buffer *scratch*; global value is nil
  Automatically becomes buffer-local when set in any fashion.

Documentation:
test
------------------------------------------------------------

             Juanma

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

* Re: indented meta-info in describe-variable
  2007-10-10  8:49   ` Juanma Barranquero
@ 2007-10-10 19:30     ` Stefan Monnier
  2007-10-10 21:37       ` Juanma Barranquero
  0 siblings, 1 reply; 6+ messages in thread
From: Stefan Monnier @ 2007-10-10 19:30 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: Emacs development discussions

> Is this difference in whitespace intended?
> (defvar test nil "test")
> (make-variable-buffer-local 'test)

> and then:

> ------------------------------------------------------------
> test's value is nil

>   Automatically becomes buffer-local when set in any fashion.

> Documentation:
> test
> ------------------------------------------------------------
> test's value is t
> Local in buffer *scratch*; global value is nil
>   Automatically becomes buffer-local when set in any fashion.

> Documentation:
> test
> ------------------------------------------------------------

Should be fixed now,


        Stefan


PS: As for using faces instead of indentation, feel free to try it out, but
it's always tricky business, and may not help on a tty.

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

* Re: indented meta-info in describe-variable
  2007-10-10 19:30     ` Stefan Monnier
@ 2007-10-10 21:37       ` Juanma Barranquero
  0 siblings, 0 replies; 6+ messages in thread
From: Juanma Barranquero @ 2007-10-10 21:37 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Emacs development discussions

On 10/10/07, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> Should be fixed now,

Thanks.

> PS: As for using faces instead of indentation, feel free to try it out, but
> it's always tricky business

The more I think about it, the more I like the idea. The output of
describe-variable is a quite structured text, with clearly defined
sections. font-locking seems like a perfect match for it.

> and may not help on a tty.

That wouldn't be worse that the default face used now.

             Juanma

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

end of thread, other threads:[~2007-10-10 21:37 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-10-09 10:10 indented meta-info in describe-variable Juanma Barranquero
2007-10-09 13:33 ` Stefan Monnier
2007-10-09 13:55   ` Juanma Barranquero
2007-10-10  8:49   ` Juanma Barranquero
2007-10-10 19:30     ` Stefan Monnier
2007-10-10 21:37       ` Juanma Barranquero

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.