Review Request 108574: Turned on sorting by pixel size and added sorting by aspect ratio

classic Classic list List threaded Threaded
22 messages Options
12
Reply | Threaded
Open this post in threaded view
|

Re: Review Request 108574: Turned on sorting by pixel size and added sorting by aspect ratio

Gilles Caulier-5
This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108574/

This review has been submitted with commit cf81fb884fd84f225f182a562251559ceb5006ec by Pankaj Kumar to branch master.

- Commit


On January 28th, 2013, 3:46 p.m. UTC, Stephen Toothman wrote:

Review request for Digikam.
By Stephen Toothman.

Updated Jan. 28, 2013, 3:46 p.m.

Description

I turned on sorting by pixel size (width * height) and added sorting by aspect ratio (width and height divided by greatest common denominator).  I use digikam to manage pictures and wallpapers I download and create, So these are two essential tools to help me stay organized.  

Testing

I have tested the functionality and both work as expected.  I have used these patches since v2.5 and I am finally ready to share them as I think they are stable.
Bugs: none

Diffs

  • digikam/items/digikamimagedelegate.cpp (3e65a85)
  • digikam/items/imagedelegate.cpp (6dfad4f)
  • digikam/items/imagedelegatepriv.h (0ffa233)
  • digikam/main/digikamapp.cpp (f9a1206)
  • digikam/utils/albumsettings.h (501213a)
  • digikam/utils/albumsettings.cpp (b838479)
  • digikam/utils/tooltipfiller.h (d384b82)
  • digikam/utils/tooltipfiller.cpp (57206cb)
  • libs/models/imagesortsettings.h (8db56e7)
  • libs/models/imagesortsettings.cpp (cf01336)
  • libs/widgets/itemview/itemviewimagedelegate.h (d1c3fd0)
  • libs/widgets/itemview/itemviewimagedelegate.cpp (1a1632c)
  • utilities/setup/setupalbumview.cpp (f2d5e7c)
  • utilities/setup/setuptooltip.cpp (b1ac814)

View Diff

File Attachments


_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

Re: Review Request 108574: Turned on sorting by pixel size and added sorting by aspect ratio

Gilles Caulier-5
In reply to this post by Bugzilla from stoothman@yahoo.com
This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/108574/

This change has been marked as submitted.


Review request for Digikam.
By Stephen Toothman.

Updated April 21, 2013, 4:13 p.m.

Description

I turned on sorting by pixel size (width * height) and added sorting by aspect ratio (width and height divided by greatest common denominator).  I use digikam to manage pictures and wallpapers I download and create, So these are two essential tools to help me stay organized.  

Testing

I have tested the functionality and both work as expected.  I have used these patches since v2.5 and I am finally ready to share them as I think they are stable.
Bugs: none

Diffs

  • digikam/items/digikamimagedelegate.cpp (3e65a85)
  • digikam/items/imagedelegate.cpp (6dfad4f)
  • digikam/items/imagedelegatepriv.h (0ffa233)
  • digikam/main/digikamapp.cpp (f9a1206)
  • digikam/utils/albumsettings.h (501213a)
  • digikam/utils/albumsettings.cpp (b838479)
  • digikam/utils/tooltipfiller.h (d384b82)
  • digikam/utils/tooltipfiller.cpp (57206cb)
  • libs/models/imagesortsettings.h (8db56e7)
  • libs/models/imagesortsettings.cpp (cf01336)
  • libs/widgets/itemview/itemviewimagedelegate.h (d1c3fd0)
  • libs/widgets/itemview/itemviewimagedelegate.cpp (1a1632c)
  • utilities/setup/setupalbumview.cpp (f2d5e7c)
  • utilities/setup/setuptooltip.cpp (b1ac814)

View Diff

File Attachments


_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
12