[Digikam-devel] [digikam] [Bug 361047] Wishlist: make grouped images more prominently visible [patch]

[via KDE Bugzilla]

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

--- Comment #11 from caulier.gilles at gmail.com ---
Thanks Omar for you patch.

1/ code cosmetic : 

* Always add a space after a coma, especially  with methods arguments.
* Try to always align code with multi-lines
* Wrap code with bracket after if/else statements, especially with multi-lines.

2/ new border : why the rotated frames are not centered with thumb center ?
It's wanted ? They show more visible on the bottom side.

3/ optimization : these calls to generateFuzzyRect() are time consuming :

+QPixmap ThumbBarDock::generateFuzzyRectForGroup(const QSize& size, const
QColor& color, int radius)
+{
+    // Create two normal borders
+    QPixmap border1 = generateFuzzyRect(size,color,radius);
+    QPixmap border2 = generateFuzzyRect(size,color,radius);

For some thumbnails it's not a problem. But imagine in an album with 100
grouped items ?
Why not yo simply make deep copy of first pixmap ?

4/ Go to Setup/Album-View, and turn on huge thumbnail support for HD screen
(thumbs >256 pixels size). The grouped frame do not work.

Gilles Caulier

-- 
You are receiving this mail because:
You are the assignee for the bug.