unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Drew Adams <drew.adams@oracle.com>
Cc: 10127@debbugs.gnu.org
Subject: bug#10127: 24.0.91; wrong window width calc for `C-u C-x =' when *Help* in separate frame
Date: Fri, 25 Nov 2011 10:06:52 +0200	[thread overview]
Message-ID: <83wrao8w2r.fsf@gnu.org> (raw)
In-Reply-To: <0797981120E94694A8751992E871E874@us.oracle.com>

> From: "Drew Adams" <drew.adams@oracle.com>
> Cc: <10127@debbugs.gnu.org>
> Date: Thu, 24 Nov 2011 11:08:59 -0800
> 
> > > There is no reason to base the display output width on the window
> > > width of the current buffer - no relation.
> > 
> > How else would you suggest to make the text aligned nicely?  That's
> > the intent, I believe.
> 
> Well, the bug describes how "nicely" the text is laid out now.
> 
> Keep it simple.  Do not try to second-guess where *Help* will be displayed or
> how wide its window might be.  Keep the text in *Help* to the normal max width,
> as much as possible.
> 
> There is nothing wonderful about this:
> 
>              foo: jkjkj
>   something-else: lllmmnlkjlj
>      and-another: hhhhhmlkklmkklj
>  and-yet-another: 232iulkjlikjkm
> 
> This is just as readable:
> 
>  foo:             jkjkj
>  something-else:  lllmmnlkjlj
>  and-another:     hhhhhmlkklmkklj
>  and-yet-another: 232iulkjlikjkm
> 
> So is this:
> 
>  foo            : jkjkj
>  something-else : lllmmnlkjlj
>  and-another    : hhhhhmlkklmkklj
>  and-yet-another: 232iulkjlikjkm
> 
> And so is this:
> 
>  foo: jkjkj
>  something-else: lllmmnlkjlj
>  and-another: hhhhhmlkklmkklj
>  and-yet-another: 232iulkjlikjkm

Your suggestions won't work with variable-size characters and
variable-pitch fonts.  The original code uses display features to
align the text even in those cases, because this command is _about_
displaying characters with various fonts, so it cannot just DTRT in
95% of cases, it needs to work in 100%.

> There are many, many ways to display such info, and which do not require
> calculating the window width.  We do the same kind of thing in our online
> manuals, when we describe functions etc., and even when we list menu items.

None of the manuals needs to cope with arbitrary characters and
arbitrary fonts.  The on-line manuals are actually quite restrictive
in the repertory of character sets and typefaces they support.

> Be less "clever".  Be more helpful to more users, who can have different
> preferences for displaying *Help*.

Be less "clever".  Be more helpful to Emacs development by actually
understanding the underlying the problems and the current solutions
before you judge them.  Do not assume that whoever wrote the code did
that out of sheer "cleverness".

To summarize: I agree that this command should be fixed for the use
case when the window width is very different from the default one.  I
just don't think the direction you propose for the solution is the
right one.





  parent reply	other threads:[~2011-11-25  8:06 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-24 18:29 bug#10127: 24.0.91; wrong window width calc for `C-u C-x =' when *Help* in separate frame Drew Adams
2011-11-24 18:49 ` Eli Zaretskii
2011-11-24 19:08   ` Drew Adams
2011-11-24 20:40     ` Juanma Barranquero
2011-11-24 20:52       ` Drew Adams
2011-11-24 21:06         ` Juanma Barranquero
2011-11-25  8:06     ` Eli Zaretskii [this message]
2011-11-25 15:25       ` Drew Adams
2011-11-25 18:23         ` Eli Zaretskii
2011-11-25 19:41           ` Drew Adams
2011-11-26 14:44             ` Eli Zaretskii
2011-11-26 14:45             ` Eli Zaretskii
2011-11-26 17:56               ` Drew Adams
2012-08-09  8:13 ` Chong Yidong

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=83wrao8w2r.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=10127@debbugs.gnu.org \
    --cc=drew.adams@oracle.com \
    /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).