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

--- Comment #19 from Henrik Fehlauer <rk...@lab12.net> ---
Git commit 899423ad41611b94848ed8fbdc877bd49f9cd408 by Henrik Fehlauer.
Committed on 30/01/2018 at 23:57.
Pushed by rkflx into branch 'master'.

Update view after changing rendering intent config

Summary:
This is a follow-up patch to D10076, so after changing the rendering
intent in Gwenview's config dialog the displayed image is updated
immediately after clicking on {nav OK} or {nav Apply}. Before, it was
necessary to {nav Reload} the image manually or pan around.

Currently this is only implemented for raster images. In case someone
figures out colour management for SVGs, this patch should be extended to
SVGs too.

Depends on {D10076}.

Test Plan:
Set up an ICC profile as described in D10076#197598 and open an
out-of-gamut image. Change the rendering intent in Gwenview's config
dialog, click {nav Apply} and observe that the image's colours change
immediately. Saving to the config file still works.

Reviewers: ngraham, muhlenpfordt

Reviewed By: ngraham

Differential Revision: https://phabricator.kde.org/D10187

M  +11   -14   lib/documentview/rasterimageview.cpp
M  +2    -0    lib/documentview/rasterimageview.h
M  +1    -0    lib/documentview/rasterimageviewadapter.cpp
M  +4    -8    lib/renderingintent.h

https://commits.kde.org/gwenview/899423ad41611b94848ed8fbdc877bd49f9cd408

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

Reply via email to