After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 350107 - [SexyTooltip] sexy_tooltip_position_to_widget positions widget off by size of window decorations
[SexyTooltip] sexy_tooltip_position_to_widget positions widget off by size of...
Status: RESOLVED WONTFIX
Product: libsexy
Classification: Deprecated
Component: general
0.1.5
Other Linux
: Normal normal
: ---
Assigned To: Christian Hammond
Libsexy Maintainers
gnome[unmaintained]
Depends on:
Blocks: 345592
 
 
Reported: 2006-08-06 00:46 UTC by Ed Catmur
Modified: 2014-08-30 13:19 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
sexytooltip.c (1.34 KB, text/plain)
2006-08-06 00:47 UTC, Ed Catmur
  Details
sexy-tooltip-origin.patch (431 bytes, patch)
2006-08-06 00:57 UTC, Ed Catmur
none Details | Review
before.png (1.34 KB, image/png)
2006-09-02 21:23 UTC, Ed Catmur
  Details
after.png (1.83 KB, image/png)
2006-09-02 21:24 UTC, Ed Catmur
  Details

Description Ed Catmur 2006-08-06 00:46:37 UTC
sexy_tooltip_position_to_widget () positions the tooltip up and to the left by the size of the window decorations. This is because it uses gdk_window_get_root_origin; should be gdk_window_get_origin.

This is with gtk+-2.10.1.
Comment 1 Ed Catmur 2006-08-06 00:47:28 UTC
Created attachment 70284 [details]
sexytooltip.c

Testcase. Compile with gcc sexytooltip.c $(pkg-config --cflags --libs libsexy).
Comment 2 Ed Catmur 2006-08-06 00:57:30 UTC
Created attachment 70285 [details] [review]
sexy-tooltip-origin.patch

Patch.
Comment 3 Christian Hammond 2006-09-02 20:57:52 UTC
Can you show me some screenshots of the before and after on your system with that test case? Here, it looks even more incorrect after. Also, did you mean to compute the location in your enter() function and then call two separate position_to_* functions?
Comment 4 Ed Catmur 2006-09-02 21:23:14 UTC
Created attachment 72097 [details]
before.png
Comment 5 Ed Catmur 2006-09-02 21:24:16 UTC
Created attachment 72098 [details]
after.png
Comment 6 Ed Catmur 2006-09-02 21:28:19 UTC
(In reply to comment #3)
> Can you show me some screenshots of the before and after on your system with
> that test case? Here, it looks even more incorrect after. Also, did you mean to
> compute the location in your enter() function and then call two separate
> position_to_* functions?

Um. I guess that was a testcase and also a check of the proposed patch (the calculated position is that in sexy_tooltip_position_to_widget, with gdk_window_get_origin for gdk_window_get_root_origin; swap round the code to get the old and proposed new behaviours).
Comment 7 Ed Catmur 2007-01-08 09:45:08 UTC
Far more serious: when used on a notification area icon, gdk_window_get_root_origin() always returns (0,0), positioning the tooltip at the origin.

See http://bugzilla.gnome.org/show_bug.cgi?id=345592#c35

Please?
Comment 8 Alex Lancaster 2007-01-09 00:59:58 UTC
Yes, it would be good to fix this upstream rather than have to hack around it in rhythmbox, better link to problem:

bug #345592 comment #35
Comment 9 André Klapper 2014-08-30 13:19:48 UTC
libsexy has not seen any releases since 2007:
http://releases.chipx86.com/libsexy/libsexy/
and the homepage is down and I have no idea where one could find a codebase.

So this project is not under active development anymore and it looks unlikely that there will be any further active development.

Closing this report as WONTFIX as part of Bugzilla Housekeeping to reflect
reality. Please feel free to reopen this bug report in the future if anyone
takes the responsibility for active development again.