unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Empty tooltips under X
@ 2015-10-28 10:01 martin rudalics
  2015-10-28 10:21 ` Tassilo Horn
  0 siblings, 1 reply; 14+ messages in thread
From: martin rudalics @ 2015-10-28 10:01 UTC (permalink / raw)
  To: emacs-devel

Dear people

With current master/trunk tooltips under X are broken here.  I get a box
with correct borders/background and correct dimensions but without text.
Can anyone see that?

Tooltips under Windows are not affected.

Thanks, martin



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

* Re: Empty tooltips under X
  2015-10-28 10:01 Empty tooltips under X martin rudalics
@ 2015-10-28 10:21 ` Tassilo Horn
  2015-10-28 10:45   ` martin rudalics
  0 siblings, 1 reply; 14+ messages in thread
From: Tassilo Horn @ 2015-10-28 10:21 UTC (permalink / raw)
  To: martin rudalics; +Cc: emacs-devel

martin rudalics <rudalics@gmx.at> writes:

Hi Martin,

> With current master/trunk tooltips under X are broken here.  I get a
> box with correct borders/background and correct dimensions but without
> text.  Can anyone see that?

I'm on

,----[ C-h v emacs-repository-version RET ]
| emacs-repository-version is a variable defined in ‘version.el’.
| Its value is "9db11fa2d635935318364370f31821023c2501b9"
| 
| Documentation:
| String giving the repository revision from which this Emacs was built.
| Value is nil if Emacs was not built from a repository checkout,
| or if we could not determine the revision.
`----

and (tooltip-show "This is a test tooltip") shows the tip correctly.
Emacs is built with GTK3 support.

Bye,
Tassilo



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

* Re: Empty tooltips under X
  2015-10-28 10:21 ` Tassilo Horn
@ 2015-10-28 10:45   ` martin rudalics
  2015-10-28 13:23     ` Tassilo Horn
  0 siblings, 1 reply; 14+ messages in thread
From: martin rudalics @ 2015-10-28 10:45 UTC (permalink / raw)
  To: emacs-devel

 > Hi Martin,
 >
 >> With current master/trunk tooltips under X are broken here.  I get a
 >> box with correct borders/background and correct dimensions but without
 >> text.  Can anyone see that?
 >
 > I'm on
 >
 > ,----[ C-h v emacs-repository-version RET ]
 > | emacs-repository-version is a variable defined in ‘version.el’.
 > | Its value is "9db11fa2d635935318364370f31821023c2501b9"
 > |
 > | Documentation:
 > | String giving the repository revision from which this Emacs was built.
 > | Value is nil if Emacs was not built from a repository checkout,
 > | or if we could not determine the revision.
 > `----
 >
 > and (tooltip-show "This is a test tooltip") shows the tip correctly.
 > Emacs is built with GTK3 support.

Indeed, GTK+ tooltip support still works.  What happens when you set
‘x-gtk-use-system-tooltips’ to nil?

martin




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

* Re: Empty tooltips under X
  2015-10-28 10:45   ` martin rudalics
@ 2015-10-28 13:23     ` Tassilo Horn
  2015-10-28 14:09       ` Kaushal Modi
  2015-10-28 19:20       ` martin rudalics
  0 siblings, 2 replies; 14+ messages in thread
From: Tassilo Horn @ 2015-10-28 13:23 UTC (permalink / raw)
  To: martin rudalics; +Cc: emacs-devel

martin rudalics <rudalics@gmx.at> writes:

>>> With current master/trunk tooltips under X are broken here.  I get a
>>> box with correct borders/background and correct dimensions but without
>>> text.  Can anyone see that?
>>
>> I'm on
>>
>> ,----[ C-h v emacs-repository-version RET ]
>> | emacs-repository-version is a variable defined in ‘version.el’.
>> | Its value is "9db11fa2d635935318364370f31821023c2501b9"
>> |
>> | Documentation:
>> | String giving the repository revision from which this Emacs was built.
>> | Value is nil if Emacs was not built from a repository checkout,
>> | or if we could not determine the revision.
>> `----
>>
>> and (tooltip-show "This is a test tooltip") shows the tip correctly.
>> Emacs is built with GTK3 support.
>
> Indeed, GTK+ tooltip support still works.  What happens when you set
> ‘x-gtk-use-system-tooltips’ to nil?

The text is still shown.  Because the tooltips looks identical to the
GTK version, I also stepped thru `x-show-tip' in order to validate we're
really using Emacs' own tooltip code.

Bye,
Tassilo



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

* Re: Empty tooltips under X
  2015-10-28 13:23     ` Tassilo Horn
@ 2015-10-28 14:09       ` Kaushal Modi
  2015-10-28 19:20         ` martin rudalics
  2015-10-28 19:20       ` martin rudalics
  1 sibling, 1 reply; 14+ messages in thread
From: Kaushal Modi @ 2015-10-28 14:09 UTC (permalink / raw)
  To: martin rudalics, emacs-devel

> With current master/trunk tooltips under X are broken here.  I get a box
> with correct borders/background and correct dimensions but without text.
> Can anyone see that?

The tooltip is working fine for me.

>  (tooltip-show "This is a test tooltip") shows the tip correctly.

Same here.

> What happens when you set ‘x-gtk-use-system-tooltips’ to nil?

OK. So now when I eval the below,

(let (x-gtk-use-system-tooltips)
  (tooltip-show "This is a test tooltip"))

I get an empty tooltip, but with different background color and border
style than when I eval just (tooltip-show "This is a test tooltip")


Built on x86_64-unknown-linux-gnu

Configure options: --prefix=/home/kmodi/usr_local/apps/6/emacs/master
'CPPFLAGS=-fgnu89-inline -I/home/kmodi/usr_local/6/include
-I/usr/include/freetype2 -I/usr/include' 'CFLAGS=-ggdb3 -O0'
'CXXFLAGS=-ggdb3 -O0' 'LDFLAGS=-L/home/kmodi/usr_local/6/lib
-L/home/kmodi/usr_local/6/lib64 -ggdb3'

Features: XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GCONF
GSETTINGS NOTIFY ACL LIBSELINUX GNUTLS LIBXML2 FREETYPE LIBOTF XFT
ZLIB TOOLKIT_SCROLL_BARS GTK2 X11



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

* Re: Empty tooltips under X
  2015-10-28 13:23     ` Tassilo Horn
  2015-10-28 14:09       ` Kaushal Modi
@ 2015-10-28 19:20       ` martin rudalics
  2015-10-28 20:08         ` Eli Zaretskii
  2015-11-06 18:13         ` martin rudalics
  1 sibling, 2 replies; 14+ messages in thread
From: martin rudalics @ 2015-10-28 19:20 UTC (permalink / raw)
  To: emacs-devel

 >> Indeed, GTK+ tooltip support still works.  What happens when you set
 >> ‘x-gtk-use-system-tooltips’ to nil?
 >
 > The text is still shown.  Because the tooltips looks identical to the
 > GTK version, I also stepped thru `x-show-tip' in order to validate we're
 > really using Emacs' own tooltip code.

Looks like a Heisenbug.  Sometimes the tooltip is shown for a very short
moment only, sometimes it's shown quite normally, sometimes not at all.

martin




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

* Re: Empty tooltips under X
  2015-10-28 14:09       ` Kaushal Modi
@ 2015-10-28 19:20         ` martin rudalics
  2015-10-28 19:47           ` Kaushal Modi
  0 siblings, 1 reply; 14+ messages in thread
From: martin rudalics @ 2015-10-28 19:20 UTC (permalink / raw)
  To: Kaushal Modi, emacs-devel

 > OK. So now when I eval the below,
 >
 > (let (x-gtk-use-system-tooltips)
 >    (tooltip-show "This is a test tooltip"))
 >
 > I get an empty tooltip, but with different background color and border

Because this is the Emacs tooltip where you can customize the
background.  The other one is the tooltip provided by Gtk+ where the
background is theirs.

 > style than when I eval just (tooltip-show "This is a test tooltip")

Can you reproduce that reliably?  With emacs -Q?

martin



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

* Re: Empty tooltips under X
  2015-10-28 19:20         ` martin rudalics
@ 2015-10-28 19:47           ` Kaushal Modi
  2015-10-28 19:51             ` Kaushal Modi
  0 siblings, 1 reply; 14+ messages in thread
From: Kaushal Modi @ 2015-10-28 19:47 UTC (permalink / raw)
  To: martin rudalics; +Cc: emacs-devel

My earlier observation was NOT in emacs -Q.

With emacs -Q, I do get the tooltip consistently with
x-gtk-use-system-tooltips set to nil or t.

But with my emacs config, the tooltip shows up consistently only with
the default value of x-gtk-use-system-tooltips (t).

But about,

> Can you reproduce that reliably?

I got the tooltip to show up just once (out of like 10 times) with my
emacs config with x-gtk-use-system-tooltips set to nil.



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

* Re: Empty tooltips under X
  2015-10-28 19:47           ` Kaushal Modi
@ 2015-10-28 19:51             ` Kaushal Modi
  0 siblings, 0 replies; 14+ messages in thread
From: Kaushal Modi @ 2015-10-28 19:51 UTC (permalink / raw)
  To: martin rudalics; +Cc: emacs-devel

Oh I emailed too soon. I saw some odd behavior when evaluating the
below in emacs using my config,

(let (x-gtk-use-system-tooltips)
  (tooltip-show "This is a test tooltip"))

The tooltip text does show up but after a delay of about 4-5 seconds.



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

* Re: Empty tooltips under X
  2015-10-28 19:20       ` martin rudalics
@ 2015-10-28 20:08         ` Eli Zaretskii
  2015-11-06 18:13         ` martin rudalics
  1 sibling, 0 replies; 14+ messages in thread
From: Eli Zaretskii @ 2015-10-28 20:08 UTC (permalink / raw)
  To: martin rudalics; +Cc: emacs-devel

> Date: Wed, 28 Oct 2015 20:20:23 +0100
> From: martin rudalics <rudalics@gmx.at>
> 
>  >> Indeed, GTK+ tooltip support still works.  What happens when you set
>  >> ‘x-gtk-use-system-tooltips’ to nil?
>  >
>  > The text is still shown.  Because the tooltips looks identical to the
>  > GTK version, I also stepped thru `x-show-tip' in order to validate we're
>  > really using Emacs' own tooltip code.
> 
> Looks like a Heisenbug.  Sometimes the tooltip is shown for a very short
> moment only, sometimes it's shown quite normally, sometimes not at all.

If you invoke trace-redisplay (after turning off blink-cursor-mode),
then make the tooltip appear several times, do you see different
traces depending on whether you see or don't see the text?




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

* Re: Empty tooltips under X
  2015-10-28 19:20       ` martin rudalics
  2015-10-28 20:08         ` Eli Zaretskii
@ 2015-11-06 18:13         ` martin rudalics
  2015-11-06 19:26           ` Eli Zaretskii
  1 sibling, 1 reply; 14+ messages in thread
From: martin rudalics @ 2015-11-06 18:13 UTC (permalink / raw)
  To: emacs-devel, eliz@gnu.org

[-- Attachment #1: Type: text/plain, Size: 368 bytes --]

  > Looks like a Heisenbug.  Sometimes the tooltip is shown for a very short
  > moment only, sometimes it's shown quite normally, sometimes not at all.

I've been able to trace this back to the following commit:

[9d6ec23f7d4f8fbbfdcea353c4b58e47f76a7342] Update frame title when redisplay scrolls selected window

Any objections against the attached patch?

martin


[-- Attachment #2: xdisp.diff --]
[-- Type: text/plain, Size: 501 bytes --]

--- a/src/xdisp.c
+++ b/src/xdisp.c
@@ -11541,9 +11541,10 @@ x_consider_frame_title (Lisp_Object frame)
 {
   struct frame *f = XFRAME (frame);

-  if (FRAME_WINDOW_P (f)
-      || FRAME_MINIBUF_ONLY_P (f)
-      || f->explicit_name)
+  if ((FRAME_WINDOW_P (f)
+       || FRAME_MINIBUF_ONLY_P (f)
+       || f->explicit_name)
+      && NILP (Fframe_parameter (frame, Qtooltip)))
     {
       /* Do we have more than one visible frame on this X display?  */
       Lisp_Object tail, other_frame, fmt;

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

* Re: Empty tooltips under X
  2015-11-06 18:13         ` martin rudalics
@ 2015-11-06 19:26           ` Eli Zaretskii
  2015-11-07 13:26             ` martin rudalics
  0 siblings, 1 reply; 14+ messages in thread
From: Eli Zaretskii @ 2015-11-06 19:26 UTC (permalink / raw)
  To: martin rudalics; +Cc: emacs-devel

> Date: Fri, 06 Nov 2015 19:13:27 +0100
> From: martin rudalics <rudalics@gmx.at>
> 
>   > Looks like a Heisenbug.  Sometimes the tooltip is shown for a very short
>   > moment only, sometimes it's shown quite normally, sometimes not at all.
> 
> I've been able to trace this back to the following commit:
> 
> [9d6ec23f7d4f8fbbfdcea353c4b58e47f76a7342] Update frame title when redisplay scrolls selected window
> 
> Any objections against the attached patch?

None whatsoever.  Thanks.



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

* Re: Empty tooltips under X
  2015-11-06 19:26           ` Eli Zaretskii
@ 2015-11-07 13:26             ` martin rudalics
  2015-11-10 19:13               ` Kaushal Modi
  0 siblings, 1 reply; 14+ messages in thread
From: martin rudalics @ 2015-11-07 13:26 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: emacs-devel

>> Any objections against the attached patch?
>
> None whatsoever.  Thanks.

Applied.

martin





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

* Re: Empty tooltips under X
  2015-11-07 13:26             ` martin rudalics
@ 2015-11-10 19:13               ` Kaushal Modi
  0 siblings, 0 replies; 14+ messages in thread
From: Kaushal Modi @ 2015-11-10 19:13 UTC (permalink / raw)
  To: martin rudalics; +Cc: Eli Zaretskii, Emacs developers

> Applied.

Just FYI, this patch fixes the issue for me too. Thanks!



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

end of thread, other threads:[~2015-11-10 19:13 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-10-28 10:01 Empty tooltips under X martin rudalics
2015-10-28 10:21 ` Tassilo Horn
2015-10-28 10:45   ` martin rudalics
2015-10-28 13:23     ` Tassilo Horn
2015-10-28 14:09       ` Kaushal Modi
2015-10-28 19:20         ` martin rudalics
2015-10-28 19:47           ` Kaushal Modi
2015-10-28 19:51             ` Kaushal Modi
2015-10-28 19:20       ` martin rudalics
2015-10-28 20:08         ` Eli Zaretskii
2015-11-06 18:13         ` martin rudalics
2015-11-06 19:26           ` Eli Zaretskii
2015-11-07 13:26             ` martin rudalics
2015-11-10 19:13               ` Kaushal Modi

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).