[Bug 290927] New: Color management does not apply change and refresh the current preview image

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

[Bug 290927] New: Color management does not apply change and refresh the current preview image

DrSlony-3
https://bugs.kde.org/show_bug.cgi?id=290927

           Summary: Color management does not apply change and refresh the
                    current preview image
           Product: digikam
           Version: 2.5.0
          Platform: Gentoo Packages
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Created an attachment (id=67560)
 --> (http://bugs.kde.org/attachment.cgi?id=67560)
No color profile information.

Version:           2.5.0 (using KDE 4.7.4)
OS:                Linux

If you enable/disable and apply color management, digiKam should update the
photo you are viewing with the change. It doesn't. You have to click on another
photo and back to this one to see the difference.

Reproducible: Always

Steps to Reproduce:
1- Disable color management
2- Click on a photo with a non-sRGB color profile (so that you can see the
difference, as sRGB will be assumed if the profile metadata is missing from the
photo)
3- Enable color management and apply it
4- You should see the colors change, but they don't.
5- Click on another photo and back to this one. Now you can see what it should
look like with color management.


Expected Results:  
digiKam should refresh the preview when you enable or disable color management
the moment you hit apply.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 290927] Color management does not apply change and refresh the current preview image

DrSlony-3
https://bugs.kde.org/show_bug.cgi?id=290927





--- Comment #1 from DrSlony <bugs seeitmyway org>  2012-01-08 03:15:35 ---
Created an attachment (id=67561)
 --> (http://bugs.kde.org/attachment.cgi?id=67561)
RT_sRGB color profile

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 290927] Color management does not apply change and refresh the current preview image

DrSlony-3
In reply to this post by DrSlony-3
https://bugs.kde.org/show_bug.cgi?id=290927





--- Comment #2 from DrSlony <bugs seeitmyway org>  2012-01-08 03:17:08 ---
With no color management, these two attached photos will look identical. With
color management enabled, noicm.jpg will appear lighter than the RT_sRGB.jpg
photo.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 290927] Color management does not apply change and refresh the current preview image

Gilles Caulier-4
In reply to this post by DrSlony-3
https://bugs.kde.org/show_bug.cgi?id=290927


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
          Component|general                     |Color Management




--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 290927] Color management does not apply change and refresh the current preview image

Dik Takken-2
In reply to this post by DrSlony-3
https://bugs.kde.org/show_bug.cgi?id=290927

Dik Takken <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]

--- Comment #3 from Dik Takken <[hidden email]> ---
Also, when you change display profiles, they are not applied immediately. You
need to close DigiKam, load it again, and only then the new display profile is
in effect.

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 290927] Color management does not apply change and refresh the current preview image

Marcel Wiesweg
In reply to this post by DrSlony-3
https://bugs.kde.org/show_bug.cgi?id=290927

Marcel Wiesweg <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/digi
                   |                            |kam/b2754413698a001b38ef448
                   |                            |74869c9f73dbe8bf3

--- Comment #4 from Marcel Wiesweg <[hidden email]> ---
Git commit b2754413698a001b38ef44874869c9f73dbe8bf3 by Marcel Wiesweg.
Committed on 21/07/2012 at 15:29.
Pushed by mwiesweg into branch 'master'.

If color management settings change, reload the preview image.

M  +2    -1    NEWS
M  +14   -0    libs/widgets/graphicsview/dimgpreviewitem.cpp
M  +2    -0    libs/widgets/graphicsview/dimgpreviewitem.h

http://commits.kde.org/digikam/b2754413698a001b38ef44874869c9f73dbe8bf3

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[Bug 290927] Color management does not apply change and refresh the current preview image

Gilles Caulier-4
In reply to this post by DrSlony-3
https://bugs.kde.org/show_bug.cgi?id=290927

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Version Fixed In|                            |2.8.0

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel