GNOME Bugzilla – Bug 340831
add toolbar icon for image collection
Last modified: 2007-06-29 23:26:24 UTC
the new GUI of EOG in 2.14.x is just GREAT. it's exactly what it should be. thank you very much! you can't compare it to what it was in 2.12.x. anyway. a small wish: the "image collection" feature is great. i think it should be made more discoverable by adding a new toolbar icon (toggle button) "image collection". when you would click it, it would hide or show the image collection. Other information:
Given that this bug mentioned in [1], I assume we want to fix it. Marking as new, gnome-love and increasing version. If someone takes it, please generate a patch for SVN trunk (former CVS HEAD) and eog-ng branch. [1] http://live.gnome.org/EyeOfGnome/EogNg,
Created attachment 79243 [details] [review] patch to eog-ng branch Claudio, if it is correct, I'll make a patch to HEAD too. Note that: I used "gtk-select-color" as icon. Was the better I saw from GTK_STOCK_* for image collection. No?
IMHO, we shouldn't use an icon whose purpose is other than the one we need, even when it looks familiar. Investigating the Icon Naming Specification[1] or asking their developers about the possibility of creating such an icon could be options. Besides that, I think the patch is correct. [1] http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html
Created attachment 79521 [details] f-spot uses this icon Claudio, there is no icon for image collection at specifications. F-Spot uses this icon for "image browse". What do you think?
I think we need Lucas' opinion on this. What do you think, Lucas?
IMHO, the icon is good enough. Some pending issues though: 1. What's the license of this icon? Compatible with EOG? 2. Is this icon available in all dimensions (16x16, 22x22, 24x24, and 32x32)? 2. Shouldn't it better to submit this icon to gnome-icon-theme? We can add the icon to EOG and when/if it gets into g-i-t we just use it there. We need to decide about our default toolbar for EOG. Discussion about it should take place in bug #310455. Also, even if this item is not agreed for default toolbar, it should definitely be available on a possible editable toolbar (bug #393586).
Option 3 (second 2) is better. How to submit to gnome-icon-theme. We have no icon for slideshow, f-spot have one. A editable toolbar is a very good idea.
(In reply to comment #7) > Option 3 (second 2) is better. How to submit to gnome-icon-theme. We have no > icon for slideshow, f-spot have one. Normally, you open a bug in the freedesktop bugzilla, requesting the icon to be added to the icon specification. Once it's added, the artists take care of drawing it. I think this icon is not Tango'ified enough -- they could prefer to draw it from scratch; but I may be wrong and they'd just take it, if the license fits and all the needed sizes are available.
The f-spot icon seem compatible with the style guidelines to me (and I draw these things all day ;) )
The problem I saw is that f-spot icon for image collection is only available in 24x24.
We can easily draw the other needed sizes of the f-spot icons since they are needed in both eog and f-spot. I'll put it on my todo. The icon license of the f-spot slideshow icon is gpl, just as f-spot itself. You'll need to check with jimmac and dobey if they think it's ok to add it to g-i-t, we can make it themeable anyway if not.
Created attachment 90852 [details] hope this works
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. 2007-06-30 Lucas Rocha <lucasr@geladinho> * data/eog-toolbar.xml: added image collection show/hide action button to the available list of toolbar items. Fixes bug #340831. * src/eog-window.c: use new image collection icon. * data/icons/*/actions/eog-image-collection.[png|svg]: new image collection icon. Thanks to Andreas Nilsson and Jakub Steiner.