Bug 290196 - PATCH: Improvements for Levels adjust tool
Summary: PATCH: Improvements for Levels adjust tool
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-Editor-Levels (show other bugs)
Version: 2.5.0
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-12-30 18:11 UTC by Matthias Welwarsky
Modified: 2017-08-07 19:03 UTC (History)
1 user (show)

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


Attachments
Patch implementing the described improvement (6.49 KB, patch)
2011-12-30 18:11 UTC, Matthias Welwarsky
Details
Updated patch (5.87 KB, patch)
2011-12-30 18:16 UTC, Matthias Welwarsky
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Matthias Welwarsky 2011-12-30 18:11:08 UTC
Created attachment 67257 [details]
Patch implementing the described improvement

Version:           unspecified (using KDE 4.7.3) 
OS:                Linux

    Enable Color Channels histogram type in Levels tool
    
    This patch enables the "Color Channels" histogram type in the Levels
    tool. It is helpful when adjusting luminosity to observe if any of the
    color channels clip, which leads to false highlight colors.
    
    The patch also implements a second function:
    if Color Channels is selected, Black and White tone pickers work on all
    of the color channels simultaneously instead of having to iterate over
    R, G and B. It is very helpful for removing the color mask of a scanned
    film negative.


Reproducible: Didn't try



Expected Results:  
Vastly improved functionality of Levels adjustment tool after applying the patch
:-)

The patch applies to the master branch of digigkam-sc core.git. It was created with "git format-patch" and can simply be applied with "git am 0001-Enable-Color-Channels-histogram-type-in-Levels-tool.patch"
Comment 1 Matthias Welwarsky 2011-12-30 18:16:42 UTC
Created attachment 67258 [details]
Updated patch

Updated patch, removing the change to adjustlevelstool.h, which is superfluous.
Comment 2 caulier.gilles 2011-12-30 21:18:39 UTC
Matthias,

I tested your patch and it work fine for me. Let's go to commit your patch in git /master...

Gilles Caulier
Comment 3 caulier.gilles 2011-12-31 13:57:33 UTC
Git commit 654ad777755c8e70c5702a257f26b7d602109e8f by Gilles Caulier.
Committed on 31/12/2011 at 14:56.
Pushed by cgilles into branch 'master'.

polish patch for Matthias Welwarsky and close relevant file in bugzilla

M  +45   -38   imageplugins/color/adjustlevelstool.cpp
M  +1    -1    imageplugins/color/adjustlevelstool.h

http://commits.kde.org/digikam/654ad777755c8e70c5702a257f26b7d602109e8f