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 538543 - Folder Export: tags patch
Folder Export: tags patch
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: Tags
0.4.x
Other Linux
: Normal enhancement
: ---
Assigned To: F-spot maintainers
F-spot maintainers
Depends on:
Blocks:
 
 
Reported: 2008-06-16 04:12 UTC by Leon Barrett
Modified: 2008-06-17 10:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
The patch: adds tags to folder export in f-spot 0.4.4 (21.57 KB, patch)
2008-06-16 04:13 UTC, Leon Barrett
none Details | Review
A patch to allow folder export with tags and icons (27.87 KB, patch)
2008-06-17 05:42 UTC, Leon Barrett
committed Details | Review

Description Leon Barrett 2008-06-16 04:12:31 UTC
I have created a patch against F-Spot 0.4.4 that does the following:

It adds an option to the Folder Export to export HTML about tags also. When checked, it will make a page listing the tags present in these images, a set of index pages for each tag, and every picture will have a list of its tags placed on its page.

I checked it, and the style conforms to your guidelines, and the links are all valid. I believe this patch should work against the trunk, but the trunk wasn't compiling for me, so I just worked against 0.4.4.

In my opinion, this greatly improves the usefulness of the Folder Export, since I spent a great deal of time tagging my photos and do not want to lose that data.

(I cannot attach the patch in this bug submission form; I'll try to attach it once the bug is submitted.)
Comment 1 Leon Barrett 2008-06-16 04:13:36 UTC
Created attachment 112808 [details] [review]
The patch: adds tags to folder export in f-spot 0.4.4

This is the patch I describe.
Comment 2 Maxxer 2008-06-16 06:03:33 UTC
maybe similar idea was already at bug 336178
Comment 3 Leon Barrett 2008-06-16 06:11:06 UTC
(In reply to comment #2)

Hmm. I guess you're right. Still, it looks like that other patch is against a much older version (0.1.10). In any case, I'll be happy as long as the feature ends up in the next release. It's a shame that I spent an unnecessary day, but I guess that's what I get for assuming no one else had thought of this before.
Comment 4 Maxxer 2008-06-16 11:34:36 UTC
I was wrong, the other bug was slightly different: it was about the option to export tag ICONS below images. Maybe you could merge that code into your, so you will display tag icons instead of names only, which look nicer (and will close two bugs :))! It shouldn't be hard. 

Two notes about this patch:
- Please check all strings you introduce are translatable. At least one exists ("All Photos"), but didn't go further in checking.
- The "Include tags" option in export window should be enabled when selecting "Standalone web gallery" only.

After those modification I think the patch will be committed. Thanks.
Comment 5 Leon Barrett 2008-06-17 05:41:19 UTC
Okay. I have modified the patch so that:
- The tag icons are also exported, with their own checkboxes
- The strings are translatable
- The checkboxes are disabled correctly

I will attach the latest patch.
Comment 6 Leon Barrett 2008-06-17 05:42:00 UTC
Created attachment 112889 [details] [review]
A patch to allow folder export with tags and icons
Comment 7 Maxxer 2008-06-17 10:57:34 UTC
thanks. committed in r4069. will be uploaded soon to the online repository.