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 167944 - Detached Toolbars unable to move/reattach
Detached Toolbars unable to move/reattach
Status: RESOLVED FIXED
Product: bonobo
Classification: Deprecated
Component: libbonoboui
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Federico Mena Quintero
bonobo qa
: 272022 272376 274287 311511 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2005-02-20 13:52 UTC by Joe McCann
Modified: 2006-10-22 14:35 UTC
See Also:
GNOME target: ---
GNOME version: 2.7/2.8


Attachments
patch for bonobo-dock-item.c resolve floating mode problem (19.92 KB, patch)
2005-07-31 19:20 UTC, Pavel
none Details | Review
Much simplier patch (1.61 KB, patch)
2006-03-23 19:38 UTC, Nickolay V. Shmyrev
committed Details | Review

Description Joe McCann 2005-02-20 13:52:38 UTC
Version details: 2.8.1
Distribution/Version: Gentoo

With libbonoboui, I can't move a detached toolbar after the initial move from
the application.  

Steps to reproduce:

drag toolbar from application to any spot on the desktop
try and move toolbar again
Comment 1 Joe McCann 2005-02-20 13:56:26 UTC
reproduced here as well: http://bugs.gentoo.org/show_bug.cgi?id=82384
Comment 2 Gustavo Carneiro 2005-02-20 15:02:46 UTC
There's a workaround I discovered: press space when the toolbar handle has focus
and it will re-attach itself to the main window.
Comment 3 Ivan Candelas 2005-06-11 22:12:02 UTC
Same bug in gnome 2.10.1 
fedora core 3
rpms from nrpms.net
Comment 4 Michael Meeks 2005-06-13 08:27:15 UTC
Federico - I thought this got fixed ? :-)
Comment 5 Trent "Lathiat" Lloyd 2005-07-29 08:18:34 UTC
This is still a problem with 2.12
Comment 6 Frederic Crozat 2005-07-29 08:19:31 UTC
Confirmed with libbonobo 2.10.0
Comment 7 Sebastien Bacher 2005-07-29 08:57:56 UTC
*** Bug 311511 has been marked as a duplicate of this bug. ***
Comment 8 Sebastien Bacher 2005-07-29 09:04:47 UTC
*** Bug 274287 has been marked as a duplicate of this bug. ***
Comment 9 Pavel 2005-07-31 19:20:45 UTC
Created attachment 50029 [details] [review]
patch for bonobo-dock-item.c resolve floating mode problem

All those bugs disappear but then we change with 'gconf-editor' the
key'/desktop/gnome/interface/toolbar_detachable' from '1' to '0' (or by gnome
gnome-ui-properties) and toolbar is floating, it remain floating without
grip...
Comment 10 Not Zed 2005-08-10 06:32:59 UTC
*** Bug 272376 has been marked as a duplicate of this bug. ***
Comment 11 Daniel Holbach 2005-09-20 20:36:49 UTC
did the patch get any attention already?
Comment 12 Pavel 2005-12-23 14:13:53 UTC
I want to know - work my patch? or only on my machine ?
Comment 13 Trent "Lathiat" Lloyd 2006-02-26 22:29:46 UTC
2.14 is approaching and this still isn't fixed :(
Comment 14 Nickolay V. Shmyrev 2006-03-22 16:07:59 UTC
Pavel, can you describe the reason of problem and why you patch makes thing work? It's unclear for me how can you remove detachable window.
Comment 15 Pavel 2006-03-22 18:29:32 UTC
Ohh... English for me is more diffiсult that 'C' or 'C++' ;-) maybe Russian ;-)))

Usind  'gdb' and 'xwininfo' I see what floating window don't receive  some (motion) events, and 'greep' is widget what hasn't own  X window it use parent window ...
I use a gdk 'floating' window instead. And reparent child panel to it when it's showed. And vice versa ... 
However main widow lose focus if I set it in the grip ;-( 
Comment 16 Nickolay V. Shmyrev 2006-03-23 19:35:29 UTC
Yeah, now I see, there reason is that BonoboDockItemGrip can't handle button press itself and the parent window that contains BonoboDockItemGrip can't handle button press also. We need somehow enable events either by packing into GtkEventBox or by changing property of floating_window.

The patch Pavel attached makes handling of this situation more like in GtkHandleBox (i.e) with additional GdkWindow instead of GtkWindow. As I've understood, it was the way this thing was previously implemented. 

Federico: why have you decided to change the design to new? That requires size allocation handling and so on? Previous was was easier to my mind.
Comment 17 Nickolay V. Shmyrev 2006-03-23 19:38:14 UTC
Created attachment 61867 [details] [review]
Much simplier patch

This patch enables button press handling, it's much easier and do the work. But it would be nice to make situation clearer. Either we should return to old GdkWindow handling or fix current.
Comment 18 Federico Mena Quintero 2006-03-28 00:41:37 UTC
If the patch works for you, commit it.  Please test it very well with Evolution first (in the main shell window and in the mail message composer).

The original design was broken: the undocked window was a child widget of its parent dock, but its allocation obviously was outside the parent's own allocation --- this doesn't make sense, and GTK+ does not support it.  That's why I rewrote it to do its own handling as a separate widget.
Comment 19 Kjartan Maraas 2006-06-17 09:43:01 UTC
Nikolay, did you have a chance to test it according to Federico's comments?
Comment 20 Nickolay V. Shmyrev 2006-06-17 10:10:31 UTC
I've committed the patch, it works fine. Although I still think it would be better to return to original scheme. It's used in gdl also for example.
Comment 21 Kjartan Maraas 2006-10-22 14:35:11 UTC
*** Bug 272022 has been marked as a duplicate of this bug. ***