Bug 149269 - Thumbnails are square (HTML Export tool) [patch]
Summary: Thumbnails are square (HTML Export tool) [patch]
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-Generic-HTMLGallery (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-27 18:42 UTC by Eric Vaandering
Modified: 2017-07-09 17:25 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 2.6.0
Sentry Crash Report:


Attachments
proposed patch to allow non-square thumbnails in HTML Export and enable the feature in floating_cards (8.26 KB, patch)
2011-12-18 14:21 UTC, Jiri Bohac
Details
refreshed patch against current git (8.32 KB, patch)
2011-12-18 21:27 UTC, Jiri Bohac
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Vaandering 2007-08-27 18:42:43 UTC
Version:            (using KDE KDE 3.5.4)
Installed from:    Ubuntu Packages
OS:                Linux

Exporting a gallery (at least a "simple" one) to HTML generates an index page that contains square thumbnails. 

The photos themselves are not square and I as a photographer go through a lot of effort to compose a shot the way I want it and it's not nice to have the edges cropped off (not to mention that if you photograph using the rule of thirds the subject is not in the center).

The thumbnail should preserve the aspect ratio of the original image and using CSS the <li> element that the image sits in can still be made square so that a gallery of portrait and landscape pictures can all be shown in a nice fashion.
Comment 1 Michal Thoma 2009-05-14 18:22:15 UTC
Nearly two years passed and still we have square thumbnails.

Croped thumbnails are actually showstopper for any serious use. And that's pity, because this plugin is very fine in other respects.

I just wanted to create image gallery and send it to one magazine for selection. But with cropped thumbs it's useless. I have to look for other web gallery creation tool...
Comment 2 JohnH 2009-05-14 18:39:36 UTC
I recall hearing that this would be corrected for KDE 4 - Haven't seen that it has been but then I only use KDE 4 long enough to confirm that I'm still not ready for it. That the kipi tools aren't automatically available in gwenview is one of the things that chases me away.
Comment 3 Jiri Bohac 2011-12-18 14:21:05 UTC
Created attachment 66864 [details]
proposed patch to allow non-square thumbnails in HTML Export and enable the feature in floating_cards

(copied from the patch description)

HTML Export: allow non-square thumbnails

Allow the themes to specify in their desktop file that they can cope with
non-square thumbnails. The user can then choose between square and non-square
thumbnails.

Add the option to floating_cards.desktop
Comment 4 caulier.gilles 2011-12-18 15:58:57 UTC
Jiri,

Thanks for your patch. I will review it

Gilles Caulier
Comment 5 Jiri Bohac 2011-12-18 21:27:44 UTC
Created attachment 66886 [details]
refreshed patch against current git
Comment 6 caulier.gilles 2011-12-31 16:32:44 UTC
Jiri,

You patch work fine into "floating_cards" theme. Thumbs are sqared or not depending of option selected...

But this theme sound like broken. Only one image (the first one in fact) is exported. All others do not appear in HTML page.

Gilles Caulier
Comment 7 Jiri Bohac 2012-01-01 20:34:26 UTC
(In reply to comment #6)

Thanks for trying this out, Gilles!

> But this theme sound like broken. Only one image (the first one in fact) is
> exported. All others do not appear in HTML page.

Strange - I am using the theme and it works fine for me.
Did the theme work for you before applying this patch?
Can you send me or attach the generated gallery including the gallery.xml file (you can remove the jpeg files...)? I'll have a look at it.

Thanks!
Comment 8 caulier.gilles 2012-01-03 14:33:42 UTC
Jiri,

Yes theme work fine without your patch. I use Chrome browser here.

Gilles Caulier
Comment 9 caulier.gilles 2012-01-03 14:40:42 UTC
I retested you patch on another computer and now it work fine (:=)))

It's probably a side effect with patch from bug #289283

Gilles
Comment 10 caulier.gilles 2012-01-03 14:41:56 UTC
Git commit d75ba9f099fdbd0a29c5e4baa6b803de057272bb by Gilles Caulier.
Committed on 03/01/2012 at 15:40.
Pushed by cgilles into branch 'master'.

Apply patch #66886 from Jiri Bohac

M  +17   -1    htmlexport/THEME_HOWTO
M  +4    -0    htmlexport/htmlexportconfig.kcfg
M  +8    -5    htmlexport/imagegenerationfunctor.cpp
M  +23   -6    htmlexport/imagesettingspage.ui
M  +4    -0    htmlexport/theme.cpp
M  +1    -0    htmlexport/theme.h
M  +3    -0    htmlexport/themes/floating_cards/floating_cards.desktop
M  +5    -0    htmlexport/wizard.cpp

http://commits.kde.org/kipi-plugins/d75ba9f099fdbd0a29c5e4baa6b803de057272bb