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

--- Comment #6 from Misha Aizatulin <ava...@hot.ee> ---
Here's a patch against 49a52e748904ae41b4991f6828ce50de9c9e05c9 (system is too
old to compile tip sadly). Obviously you'll need to adjust it.

I actually like the double-profile behaviour in about 10% of cases (when the
images are really dark), and I'm not sure how color management works on other
systems, so I made it a config option.

diff --git a/app/generalconfigpage.ui b/app/generalconfigpage.ui
index d874d8a5..1efe9d63 100644
--- a/app/generalconfigpage.ui
+++ b/app/generalconfigpage.ui
@@ -25,6 +25,13 @@
      </property>
     </widget>
    </item>
+   <item row="0" column="1">
+    <widget class="QCheckBox" name="kcfg_NoICC">
+     <property name="text">
+       <string>No ICC</string>
+     </property>
+    </widget>
+   </item>
    <item row="1" column="1">
     <spacer name="verticalSpacer">
      <property name="orientation">
diff --git a/lib/cms/cmsprofile.cpp b/lib/cms/cmsprofile.cpp
index 56f38d5a..eec7b6ef 100644
--- a/lib/cms/cmsprofile.cpp
+++ b/lib/cms/cmsprofile.cpp
@@ -28,6 +28,8 @@ Foundation, Inc., 51 Franklin Street, Fifth Floor, Cambridge,
MA 02110-1301, USA
 #include <iodevicejpegsourcemanager.h>
 #include <jpegerrormanager.h>

+#include <lib/gwenviewconfig.h>
+
 extern "C" {
 #include <cms/iccjpeg.h>
 }
@@ -212,7 +214,7 @@ Profile::Ptr Profile::getMonitorProfile()
     // Get the profile from you config file if the user has set it.
     // if the user allows override through the atom, do this:
 #ifdef HAVE_X11
-    if (QX11Info::isPlatformX11()) {
+    if (QX11Info::isPlatformX11() && !GwenviewConfig::noICC()) {
         // get the current screen...
         int screen = -1;

diff --git a/lib/gwenviewconfig.kcfg b/lib/gwenviewconfig.kcfg
index 417b4ad0..f991e945 100644
--- a/lib/gwenviewconfig.kcfg
+++ b/lib/gwenviewconfig.kcfg
@@ -222,6 +222,9 @@
         <entry name="ListVideos" type="Bool">
             <default>true</default>
         </entry>
+        <entry name="NoICC" type="Bool">
+            <default>false</default>
+        </entry>
     </group>

     <group name="Print">

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to