Skip to content

Commit 3a51e4f

Browse files
jenshannoschwalmTurboGit
authored andcommitted
Fix auto applied presets for sraws and true monochromes
We use `dt_image_is_rawprepare_supported()` instead of `dt_image_is_raw()` in presets and modulegroups and should also do so while auto apply of modules for a fresh history. This fixes for example exposure not being applied correctly in sraws and true monochromes, clicking for reset to defaults worked as intended. This leaves out the current scene-referred tone mappers, they all auto-apply also for monochromes which i think is correct. If not they all should only have `FOR_MATRIX` instead.
1 parent e87f8b5 commit 3a51e4f

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/develop/develop.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1490,7 +1490,7 @@ static gboolean _dev_auto_apply_presets(dt_develop_t *dev)
14901490
dt_image_t *image = dt_image_cache_get(imgid, 'w');
14911491
if(!(image->flags & DT_IMAGE_AUTO_PRESETS_APPLIED)) run = TRUE;
14921492

1493-
const gboolean is_raw = dt_image_is_raw(image);
1493+
const gboolean is_raw = dt_image_is_rawprepare_supported(image);
14941494
const gboolean is_modern_chroma = dt_is_scene_referred();
14951495

14961496
// flag was already set? only apply presets once in the lifetime of
@@ -1569,7 +1569,7 @@ static gboolean _dev_auto_apply_presets(dt_develop_t *dev)
15691569
// set filters
15701570

15711571
int iformat = 0;
1572-
if(dt_image_is_raw(image))
1572+
if(is_raw)
15731573
iformat |= FOR_RAW;
15741574
else
15751575
iformat |= FOR_LDR;

0 commit comments

Comments
 (0)