GNU bug report logs - #12840
24.2.50; ediff: mode-line of control buffer - improvement request

Previous Next

Package: emacs;

Reported by: Michael Heerdegen <michael_heerdegen <at> web.de>

Date: Thu, 8 Nov 2012 17:30:02 UTC

Severity: wishlist

Tags: moreinfo

Found in version 24.2.50

Fixed in version 29.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 12840 in the body.
You can then email your comments to 12840 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#12840; Package emacs. (Thu, 08 Nov 2012 17:30:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Michael Heerdegen <michael_heerdegen <at> web.de>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Thu, 08 Nov 2012 17:30:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Michael Heerdegen <michael_heerdegen <at> web.de>
To: bug-gnu-emacs <at> gnu.org
Subject: 24.2.50; ediff: mode-line of control buffer - improvement request
Date: Thu, 08 Nov 2012 18:29:08 +0100
Hello,

I use ediff with this setting:

(setq ediff-window-setup-function 'ediff-setup-windows-plain)

Then, the mode-line of the control buffer looks like that:

-- *Ediff Control Panel*   diff 1 of 1        Quick Help

My questions/requests:

1.  It would be much more convenient if the "diff ... of ..." part would
be highlighted somehow, so that it is more visible.  It is an extremely
relevant information, and can now be recognized very badly.

The function to change would probably be
`ediff-make-wide-control-buffer-id'.  I tested with this, for example:

(defun ediff-make-wide-control-buffer-id ()
  (list (concat "%b   "
                (propertize (cond ((< ediff-current-difference 0)
                                   (format "At start of %d diffs"
                                           ediff-number-of-differences))
                                  ((>= ediff-current-difference ediff-number-of-differences)
                                   (format "At end of %d diffs"
                                           ediff-number-of-differences))
                                  (t
                                   (format "diff %d of %d"
                                           (1+ ediff-current-difference)
                                           ediff-number-of-differences)))
                            'face 'mode-line-buffer-id))))

2. What's the "Quick Help" part for?  I don't see any use for it.  It
seems to be always present, but it doesn't indicate anything, and I
cannot click on it.  Also, if I didn't hit ? in the control buffer, the
only content of this buffer already is "Type ? for help".

This is more a question.  I just don't see the purpose of showing this
string.  Maybe I've overlooked something.


Thanks,

Michael.



In GNU Emacs 24.2.50.1 (x86_64-pc-linux-gnu, GTK+ Version 3.4.2)
 of 2012-11-01 on dex, modified by Debian
 (emacs-snapshot package, version 2:20121101-1)
Windowing system distributor `The X.Org Foundation', version 11.0.10707000
System Description:	Debian GNU/Linux testing (wheezy)

Configured using:
 `configure '--build' 'x86_64-linux-gnu' '--host' 'x86_64-linux-gnu'
 '--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib'
 '--localstatedir=/var' '--infodir=/usr/share/info'
 '--mandir=/usr/share/man' '--with-pop=yes'
 '--enable-locallisppath=/etc/emacs-snapshot:/etc/emacs:/usr/local/share/emacs/24.2.50/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/24.2.50/site-lisp:/usr/share/emacs/site-lisp'
 '--without-compress-info' '--with-crt-dir=/usr/lib/x86_64-linux-gnu/'
 '--with-x=yes' '--with-x-toolkit=gtk3' '--with-imagemagick=yes'
 'build_alias=x86_64-linux-gnu' 'host_alias=x86_64-linux-gnu'
 'CFLAGS=-DDEBIAN -DSITELOAD_PURESIZE_EXTRA=5000 -g -O2' 'LDFLAGS=-g
 -Wl,--as-needed -znocombreloc' 'CPPFLAGS=-D_FORTIFY_SOURCE=2''

Important settings:
  value of $LC_ALL: de_DE.utf8
  value of $LC_TIME: C
  value of $LANG: de_DE.utf8
  locale-coding-system: utf-8-unix
  default enable-multibyte-characters: t





Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#12840; Package emacs. (Tue, 10 May 2022 12:10:01 GMT) Full text and rfc822 format available.

Message #8 received at 12840 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: 12840 <at> debbugs.gnu.org
Subject: Re: bug#12840: 24.2.50; ediff: mode-line of control buffer -
 improvement request
Date: Tue, 10 May 2022 14:08:57 +0200
[Message part 1 (text/plain, inline)]
Michael Heerdegen <michael_heerdegen <at> web.de> writes:

> 1.  It would be much more convenient if the "diff ... of ..." part would
> be highlighted somehow, so that it is more visible.  It is an extremely
> relevant information, and can now be recognized very badly.
>
> The function to change would probably be
> `ediff-make-wide-control-buffer-id'.  I tested with this, for example:

[...]

>                             'face 'mode-line-buffer-id))))

I wouldn't mind making that bit bold, but it is somewhat unusual for
modes to do that.  On the other and, there isn't really a lot of other
info in that mode line, so it wouldn't be distracting, either:

[Message part 2 (image/png, inline)]
[Message part 3 (text/plain, inline)]
> 2. What's the "Quick Help" part for?  I don't see any use for it.  It
> seems to be always present, but it doesn't indicate anything, and I
> cannot click on it.  Also, if I didn't hit ? in the control buffer, the
> only content of this buffer already is "Type ? for help".

The "Quick Help" bit has a popup that says what mouse-2 and mouse-3
does, which seems somewhat helpful.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

Added tag(s) moreinfo. Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Tue, 10 May 2022 12:10:01 GMT) Full text and rfc822 format available.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#12840; Package emacs. (Wed, 11 May 2022 00:04:01 GMT) Full text and rfc822 format available.

Message #13 received at 12840 <at> debbugs.gnu.org (full text, mbox):

From: Michael Heerdegen <michael_heerdegen <at> web.de>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 12840 <at> debbugs.gnu.org
Subject: Re: bug#12840: 24.2.50; ediff: mode-line of control buffer -
 improvement request
Date: Wed, 11 May 2022 02:02:56 +0200
Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> I wouldn't mind making that bit bold, but it is somewhat unusual for
> modes to do that.  On the other and, there isn't really a lot of other
> info in that mode line, so it wouldn't be distracting, either:
> [screenshot]

I can only speak for me.  When looking at the mode line I want to see
the diff numbers in nearly all cases, and I don't want distractions when
using Ediff, so I made them outstanding (red).

> > 2. What's the "Quick Help" part for?  I don't see any use for it.  It
> > seems to be always present, but it doesn't indicate anything, and I
> > cannot click on it.  Also, if I didn't hit ? in the control buffer, the
> > only content of this buffer already is "Type ? for help".
>
> The "Quick Help" bit has a popup that says what mouse-2 and mouse-3
> does, which seems somewhat helpful.

But that's the same mouse tooltip as the rest of the mode-line of the
control buffer has.

Those mouse bindings are not "Quick Help" nor Ediff specific, they are
the very basic bindings from mouse.el:

(global-set-key [mode-line mouse-1] #'mouse-select-window)
(global-set-key [mode-line mouse-2] #'mouse-delete-other-windows)
(global-set-key [mode-line mouse-3] #'mouse-delete-window)

No problem about the tooltip - but that doesn't explain why "Quick Help"
is displayed.

The "quick help" is what you get when you hit `?`.  Look: Without the

  (setq ediff-window-setup-function 'ediff-setup-windows-plain)

setting, the control frame doesn't display a mode-line, but after
hitting `?` it does, and the "Quick Help" info gets visible.  My
interpretation is that 'ediff-setup-windows-plain has been added later,
after quick help, and the "Quick Help" string was never intended to be
visible all the time.  Plausible?

Michael.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#12840; Package emacs. (Wed, 11 May 2022 11:52:01 GMT) Full text and rfc822 format available.

Message #16 received at 12840 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: 12840 <at> debbugs.gnu.org
Subject: Re: bug#12840: 24.2.50; ediff: mode-line of control buffer -
 improvement request
Date: Wed, 11 May 2022 13:51:42 +0200
Michael Heerdegen <michael_heerdegen <at> web.de> writes:

> The "quick help" is what you get when you hit `?`.  Look: Without the
>
>   (setq ediff-window-setup-function 'ediff-setup-windows-plain)
>
> setting, the control frame doesn't display a mode-line, but after
> hitting `?` it does, and the "Quick Help" info gets visible.  My
> interpretation is that 'ediff-setup-windows-plain has been added later,
> after quick help, and the "Quick Help" string was never intended to be
> visible all the time.  Plausible?

Yes, that sounds likely to me.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#12840; Package emacs. (Mon, 13 Jun 2022 13:51:01 GMT) Full text and rfc822 format available.

Message #19 received at 12840 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: 12840 <at> debbugs.gnu.org
Subject: Re: bug#12840: 24.2.50; ediff: mode-line of control buffer -
 improvement request
Date: Mon, 13 Jun 2022 15:50:31 +0200
Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Michael Heerdegen <michael_heerdegen <at> web.de> writes:
>
>> The "quick help" is what you get when you hit `?`.  Look: Without the
>>
>>   (setq ediff-window-setup-function 'ediff-setup-windows-plain)
>>
>> setting, the control frame doesn't display a mode-line, but after
>> hitting `?` it does, and the "Quick Help" info gets visible.  My
>> interpretation is that 'ediff-setup-windows-plain has been added later,
>> after quick help, and the "Quick Help" string was never intended to be
>> visible all the time.  Plausible?
>
> Yes, that sounds likely to me.

I've now done these changes in Emacs 29 -- removed the "Quick Help"
thing with the plain setting, and made the informative parts bold (using
your suggested function).

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




bug marked as fixed in version 29.1, send any further explanations to 12840 <at> debbugs.gnu.org and Michael Heerdegen <michael_heerdegen <at> web.de> Request was from Lars Ingebrigtsen <larsi <at> gnus.org> to control <at> debbugs.gnu.org. (Mon, 13 Jun 2022 13:51:01 GMT) Full text and rfc822 format available.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#12840; Package emacs. (Tue, 14 Jun 2022 14:46:02 GMT) Full text and rfc822 format available.

Message #24 received at 12840 <at> debbugs.gnu.org (full text, mbox):

From: Michael Heerdegen <michael_heerdegen <at> web.de>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 12840 <at> debbugs.gnu.org
Subject: Re: bug#12840: 24.2.50; ediff: mode-line of control buffer -
 improvement request
Date: Tue, 14 Jun 2022 16:45:02 +0200
[Message part 1 (text/plain, inline)]
Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> I've now done these changes in Emacs 29 -- removed the "Quick Help"
> thing with the plain setting, and made the informative parts bold
> (using your suggested function).

Thanks.

I would rather like to avoid to decide based on the value of
`ediff-window-setup-function' since users may wish to implement their
own setup functions.

Maybe a better approach of deciding would be to make it depend on
whether the Quick Help is currently displayed?  Seems there is a
variable we could use:

[0001-WIP-Tune-when-to-show-Quick-Help-in-Ediff.patch (text/x-diff, inline)]
From fb2ff90ea8011eb7ebe1961869a74374a266c1d6 Mon Sep 17 00:00:00 2001
From: Michael Heerdegen <michael_heerdegen <at> web.de>
Date: Tue, 14 Jun 2022 15:09:31 +0200
Subject: [PATCH] WIP: Tune when to show "Quick Help" in Ediff

---
 lisp/vc/ediff-wind.el | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/lisp/vc/ediff-wind.el b/lisp/vc/ediff-wind.el
index 4549b910b1..6db3667545 100644
--- a/lisp/vc/ediff-wind.el
+++ b/lisp/vc/ediff-wind.el
@@ -1136,9 +1136,7 @@ ediff-refresh-mode-lines
 	  (if (ediff-narrow-control-frame-p)
 	      (list "   " mode-line-buffer-identification)
 	    (list "-- " mode-line-buffer-identification
-                  (and (not (eq ediff-window-setup-function
-                                'ediff-setup-windows-plain))
-                       "        Quick Help"))))
+                  (list 'ediff-use-long-help-message "        Quick Help"))))
     ;; control buffer id
     (setq mode-line-buffer-identification
 	  (if (ediff-narrow-control-frame-p)
--
2.30.2

[Message part 3 (text/plain, inline)]

Michael.

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#12840; Package emacs. (Wed, 15 Jun 2022 12:11:02 GMT) Full text and rfc822 format available.

Message #27 received at 12840 <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Michael Heerdegen <michael_heerdegen <at> web.de>
Cc: 12840 <at> debbugs.gnu.org
Subject: Re: bug#12840: 24.2.50; ediff: mode-line of control buffer -
 improvement request
Date: Wed, 15 Jun 2022 14:09:56 +0200
Michael Heerdegen <michael_heerdegen <at> web.de> writes:

> Maybe a better approach of deciding would be to make it depend on
> whether the Quick Help is currently displayed?  Seems there is a
> variable we could use:

Sure, that's better; please go ahead and push.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#12840; Package emacs. (Sun, 19 Jun 2022 12:31:02 GMT) Full text and rfc822 format available.

Message #30 received at 12840 <at> debbugs.gnu.org (full text, mbox):

From: Michael Heerdegen <michael_heerdegen <at> web.de>
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 12840 <at> debbugs.gnu.org
Subject: Re: bug#12840: 24.2.50; ediff: mode-line of control buffer -
 improvement request
Date: Sun, 19 Jun 2022 14:30:29 +0200
Lars Ingebrigtsen <larsi <at> gnus.org> writes:

> Sure, that's better; please go ahead and push.

Done.

I also investigated what the original intention could have been.
Originally, "Howdy!" was shown instead of "Quick Help".

Later this was changed to "Quick Help" (to say something meaningful, I
guess).  So I would say we are compatible with the original intention.

Michael.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Mon, 18 Jul 2022 11:24:08 GMT) Full text and rfc822 format available.

This bug report was last modified 1 year and 254 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.