| Summary: | display RGB histograms in 3 separate diagrams | ||
|---|---|---|---|
| Product: | [Applications] digikam | Reporter: | Fabien <fabien.ubuntu> |
| Component: | ImageEditor-Canvas | Assignee: | Digikam Developers <digikam-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | wishlist | CC: | caulier.gilles, marcel.wiesweg |
| Priority: | NOR | ||
| Version: | 2.4.0 | ||
| Target Milestone: | --- | ||
| Platform: | Ubuntu | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | 2.4.0 | |
| Sentry Crash Report: | |||
| Attachments: |
screenshot of sidebar
screenshot of photoshop histograms patch to display RGB histogram at the same time as photoshop. Screenshot of new Colors tab from right sidebar update patch to compile with git master |
||
|
Description
Fabien
2008-03-28 13:06:37 UTC
Fabien, I not agree to use 3 separate view for that, and for a simple reason : it take too more space. A better way is to make a semi transparent histogram, like photoshop do for ex. This can be done easily with Qt4. Marcel, your viewpoint... Gilles Caulier Created attachment 24103 [details]
screenshot of sidebar
screenshot to show sidebar space below histogram
Gilles, I understand your opinion, but I think it should be configurable. Look at my screenshot made with 20" wide LCD screen (1680x1050 resolution). There's plenty of space for that... Even my 2.5" lcd screen camera can display the 3 histograms. I find that so useful that it's a pity to not have it in digiKam. Created attachment 24105 [details]
screenshot of photoshop histograms
Here's a screenshot of photoshop histograms
I added another screenshot of photoshop histograms. I still think displaying 3 different diagrams is much easier to read than having a combined one... And, it doesn't take some much space :) Digikam features should fit into 800x600 window. I think it is possible but barely. My viewpoint: In the default case, just as Mikolaj said, we cannot use three histograms. It is too large. Transparent drawing in the combined view is a good idea as far as I see. Three separate histograms as an option? I dont object if there is user demand. Please, not another option. No need to add another option, we just need another menu entry. Right now, there's channel : - luminosity - Red - Green - Blue - Alpha - Colors Just add "RGB channels" and that's it :) I like histogram channels selection from RawTherapee : http://www.flickr.com/photos/digikam/5348758734/sizes/o/in/photostream/ We only just 5 new tool buttons (as histogram scale) to show relevant color channel together : Luminosity Red Green Blue Alpha And that all... Gilles Caulier Created attachment 65936 [details]
patch to display RGB histogram at the same time as photoshop.
The patch display 3 others separate RGB histogram into Color tab from right sidebar.
Gilles Caulier
Created attachment 65937 [details]
Screenshot of new Colors tab from right sidebar
Please tests and report. My patch is against git/master. Let's me hear if this solution is suitable for you. It's not configurable. Gilles Caulier No feedback yet about my patch and my screenshot ? Gilles Caulier Created attachment 66052 [details]
update patch to compile with git master
Comment on attachment 65937 [details] Screenshot of new Colors tab from right sidebar new screenshot of RGB histograms patch in action. http://www.flickr.com/photos/digikam/6396713165/sizes/o/in/photostream/ Git commit f8437b9ac0bba2fea327da9b4465388e74dafa4b by Gilles Caulier. Committed on 25/11/2011 at 12:50. Pushed by cgilles into branch 'master'. Colors tab from right sidebar : display RGB histograms in 3 separate diagrams for better comparison, channel by channel. BUGS: 160013 M +5 -6 libs/dimg/filters/levels/histogrambox.cpp M +4 -2 libs/dimg/filters/levels/histogrambox.h M +6 -0 libs/dimg/filters/levels/histogramwidget.cpp M +1 -0 libs/dimg/filters/levels/histogramwidget.h M +111 -9 libs/imageproperties/imagepropertiescolorstab.cpp M +1 -0 libs/imageproperties/imagepropertiescolorstab.h http://commits.kde.org/digikam/f8437b9ac0bba2fea327da9b4465388e74dafa4b |