all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Mauro Aranda <maurooaranda@gmail.com>
Cc: stefan@marxist.se, 3192@debbugs.gnu.org
Subject: bug#3192: 23.0.92; disabling tooltip-mode inhibits showing Customize error msgs
Date: Sat, 07 Sep 2019 12:55:07 +0300	[thread overview]
Message-ID: <83ef0s5t38.fsf@gnu.org> (raw)
In-Reply-To: <CABczVwdH837UiuvMmePt1s5rPnB--dPs9LCj-0GuNd+59pw43w@mail.gmail.com> (message from Mauro Aranda on Mon, 2 Sep 2019 08:16:21 -0300)

> From: Mauro Aranda <maurooaranda@gmail.com>
> Date: Mon, 2 Sep 2019 08:16:21 -0300
> Cc: Stefan Kangas <stefan@marxist.se>, 3192@debbugs.gnu.org
> 
> >> +     ;; Only stop displaying the message when the current message is our own.
> >> +     ;; This has the advantage of not clearing the echo area when
> >> +     ;; running after an error message was displayed.  (Bug#3192)
> >> +     ((equal-including-properties tooltip-help-message (current-message))
> >>        (message nil)))))
> >
> > I think this will effectively disable tooltip messages in the echo
> > area in too many situations.
> 
> Sorry, I don't think I follow.  The clause I propose would replace a
> clause whose condition is `t'.  So the new code would run, at most, in the
> same cases that the current code, right?  And the two clauses (the
> current, and my proposed replace) call `message' with a nil argument, so
> if there is some disabling, it is already happening, right?

My point is that this fix is not as good as possible.  I'm asking
whether we can do better than this.  If not, can you explain why not?

> But I can't figure out why this code (or the current code) would be
> disabling tooltip messages.  AFAIU, the case when there is some message
> to show is already handled in the first of the clauses, with the
> condition: (stringp help).

If this is already handled, then why do we need this change at all?
What use case does it handle that the current code doesn't?

> > How about if we show the tooltip in these cases for only some short
> > enough period of time, like 2 sec, and then restore the original
> > contents of the echo area?
> 
> There should be no tooltip message to show, because this condition would
> be reached if HELP is nil.  (HELP might be either a string or nil, just
> like in `tooltip-show-help' function).  So I think when the two
> previous conditions were not meet, it indicates that it is time to clear
> the echo area.  But we shouldn't do that unconditionally, only when the
> current message is that of a tooltip.

Then I guess I didn't understand your analysis of the original use
case and what happens there.  Could you perhaps describe that use case
in more detail?

Thanks.





  reply	other threads:[~2019-09-07  9:55 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-02 15:28 bug#3192: 23.0.92; disabling tooltip-mode inhibits showing Customize error msgs Drew Adams
2012-01-29 10:14 ` Chong Yidong
2012-01-29 11:59   ` Juanma Barranquero
2019-08-30 14:44 ` Mauro Aranda
2019-08-31  2:34   ` Stefan Kangas
2019-09-01 10:43     ` Mauro Aranda
2019-09-01 14:02       ` Eli Zaretskii
2019-09-02 11:16         ` Mauro Aranda
2019-09-07  9:55           ` Eli Zaretskii [this message]
2019-09-07 21:14             ` Mauro Aranda
2019-09-14  8:27               ` Eli Zaretskii
2019-09-16 14:43                 ` Mauro Aranda
2019-09-16 20:18                   ` Lars Ingebrigtsen
2019-09-16 22:20                     ` Mauro Aranda

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83ef0s5t38.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=3192@debbugs.gnu.org \
    --cc=maurooaranda@gmail.com \
    --cc=stefan@marxist.se \
    /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 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.