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 435614 - Wen DnD rotate left/right icons to "Toolbar Editor" the icon not appear
Wen DnD rotate left/right icons to "Toolbar Editor" the icon not appear
Status: RESOLVED FIXED
Product: eog
Classification: Core
Component: general
git master
Other Linux
: Normal minor
: ---
Assigned To: EOG Maintainers
EOG Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-05-03 20:02 UTC by Bruno Boaventura
Modified: 2007-05-17 08:40 UTC
See Also:
GNOME target: ---
GNOME version: 2.17/2.18



Description Bruno Boaventura 2007-05-03 20:02:11 UTC
Steps:

1. Open "Toolbar Editor" (Edit->Toolbar);
2. Drag and Drop Left or Right (rotate icons) to the Editor;

Expected result:
Icon should appear.

Result:
Only the Label appear.
Comment 1 Diego Escalante Urrelo (not reading bugmail) 2007-05-04 19:18:34 UTC
(eog:18226): Gtk-CRITICAL **: gtk_drag_source_set_icon_pixbuf: assertion `GDK_IS_PIXBUF (pixbuf)' failed

(eog:18226): GLib-GObject-CRITICAL **: g_object_unref: assertion `G_IS_OBJECT (object)' failed

confirming.
Comment 2 Felix Riemann 2007-05-05 10:54:49 UTC
Yes, the toolbar-editor code tries to retrieve the icons using their stock-id. 
The problem: EOG's rotate and flip icons aren't stock icons, but themed (named) icons. They require a different approach of retrieving them.
Comment 3 Felix Riemann 2007-05-05 11:20:31 UTC
I just committed a fix to trunk (rev 3578).

2007-05-05  Felix Riemann  <>

	* cut-n-paste/toolbar-editor/egg-toolbar-editor.c:
	(event_box_realize_cb), (editor_create_item_from_name):
	Fix usage of themed icons and have them displayed under the pointer
	while dragging them to the toolbar. Fixes bug 435614.

This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.

Please note that you might need to do a "make clean" first before rebuilding as the build system doesn't seem to catch changes in the toolbar editor and thus doesn't update EOG's executable.
Comment 4 Lucas Rocha 2007-05-05 12:41:32 UTC
Felix, please apply this patch on libegg too so that other EggToolbarEditor users can get this improvement as well. Make sure nothing brakes, of course. :-)
Comment 5 Claudio Saavedra 2007-05-07 18:42:35 UTC
It would be useful to publicite the fix also for people using their local copy of the editable toolbar. Maybe an entry in a blog in p.g.o could be useful, or a heads up in gnome-hackers@gnome.org.
Comment 6 Felix Riemann 2007-05-07 19:17:54 UTC
Should I commit it directly or should I open a bug report?
Comment 7 Felix Riemann 2007-05-07 19:26:30 UTC
Ah, I read the README. Will submit a patch.
Comment 8 Felix Riemann 2007-05-08 15:27:57 UTC
Patch has landed in libegg and epiphany: bug 436684 .
Comment 9 Lucas Rocha 2007-05-08 17:29:49 UTC
Evince uses toolbar editor too.
Comment 10 Bruno Boaventura 2007-05-16 20:08:45 UTC
The same with "Slideshow". Icon doesn't appear.
Comment 11 Felix Riemann 2007-05-17 08:40:01 UTC
True, but that's a different problem. The problem with the slideshow item is that there is no icon set for it yet. See bug 374582 comment 8.

Setting back to fixed.