[Digikam-devel] Review Request 109298: Adding photo informations to the album view item

[Marco Berghoff]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109298/#review29155
-----------------------------------------------------------


My original idea was to increase the number of photos in Album view, therefore one need to reduce the information below and the most important information should also available in the thumbnail bar.

Well to expand this overlay to the Preview mode should be easy and the information in the previewed photos is also a nice feature independent of this overlay.

You're right, the overlay bloat the GUI, but it is acceptable for few information.
* Most of the photos are landscape so that there are some space on the top. 
* The shortcut and toolbar icon is there for switch on / off the overlay. I need this also.
* The tool tips shows information only for one photo, so you cannot compare them.
* The new table view is good for comparison of information but it has one drawback: you only have a few photos on the side.

Marco

- Marco Berghoff


On March 13, 2013, 12:10 a.m., Marco Berghoff wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109298/
> -----------------------------------------------------------
> 
> (Updated March 13, 2013, 12:10 a.m.)
> 
> 
> Review request for Digikam and Gilles Caulier.
> 
> 
> Description
> -------
> 
> This patch adds a line with informations about aperture, exposure time, ISO sensitivity and focal length between filename and title. It looks like "F7.1 1/900s ISO 200 8mm".
> 
> A checkbox for switching this line on/off is added to the albumview setup dialog.
> 
> 
> Diffs
> -----
> 
>   utilities/setup/setupalbumview.cpp f2d5e7c 
>   utilities/importui/views/importsettings.cpp 0175079 
>   utilities/importui/views/importsettings.h 1c3274e 
>   libs/widgets/itemview/itemviewimagedelegatepriv.h 8735e6a 
>   libs/widgets/itemview/itemviewimagedelegate.cpp 1a1632c 
>   libs/widgets/itemview/itemviewimagedelegate.h d1c3fd0 
>   digikam/utils/albumsettings.h 501213a 
>   digikam/utils/albumsettings.cpp b838479 
>   digikam/main/digikamui.rc f410ad7 
>   digikam/main/digikamapp_p.h 2d9f0c2 
>   digikam/main/digikamapp.cpp 290840f 
>   digikam/items/imagethumbnaildelegate.cpp d34b21e 
>   digikam/main/digikamapp.h 0286bce 
>   digikam/items/imagedelegatepriv.h 0ffa233 
>   digikam/items/imagedelegate.cpp 6dfad4f 
>   digikam/items/digikamimagedelegate.cpp 3e65a85 
> 
> Diff: http://git.reviewboard.kde.org/r/109298/diff/
> 
> 
> Testing
> -------
> 
> Compiles fine.
> Albumview items have the infoline if the checkmark is set in the setup dialog and hide it if not.
> 
> 
> Thanks,
> 
> Marco Berghoff
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20130313/79b93235/attachment.html>