[Digikam-devel] [Bug 148572] show filename with each thumb/image in light table

Gilles Caulier caulier.gilles at gmail.com
Wed Jul 22 08:10:26 BST 2009


https://bugs.kde.org/show_bug.cgi?id=148572





--- Comment #13 from Gilles Caulier <caulier gilles gmail com>  2009-07-22 09:10:08 ---
Pieter,

Thanks a lots for your patch.

Yes, it's not the right file to attach you patch, but #173061 is not the right
one too.

Your patch touch only lighttablepreview. It's not optimum. It will be better to
share your patch between lighttablepreview AND imagepreviewview. Both are based
on previewwidget class. The advantage is to have your patch available on
lighttable and preview mode from album gui. The best way is to create a common
class based on previewwidget to include your code. Both classes
lighttablepreview and imagepreviewview must be based on this new common class.

There is a bug in your patch when you zoom and pan inside preview widget, and
when you use paniconwidget from the right bottom corner of preview view.
Overwrited text in canvas is not cleaned properly. I will attach a screen-shot.

About setup:

- i think opacity settings is not necessary.
- Why not to separate iso speed, f-spot, and exposure options.

Excepted that setup code is fine of course. The same can be done for preview
mode.

Gilles Caulier

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the Digikam-devel mailing list