Skip to content

Conversation

@Phemisters
Copy link
Contributor

Relates to two darktable PRs: darktable-org/darktable#12882 and darktable-org/darktable#12887

Also relates to dtdocs issue: #527

Attempts to update the filmic doc page with changes to defaults and controls. It's quite a long page, and I don't think the PR comments mention all the changes. 🤞 I hope I caught enough of the changes, but let me know what you think.

@elstoc
Copy link
Contributor

elstoc commented Jan 26, 2026

#742 also attempts to make some changes to some of the same documentation. On reflection I think this is probably closer to what I would have done, but it doesn't mention the change to the default latitude. I think I'll go ahead and merge this one and we can perhaps revisit latitude in another PR.

Thanks!

@elstoc elstoc merged commit a5b9b64 into darktable-org:master Jan 26, 2026
1 check passed
@elstoc elstoc mentioned this pull request Jan 26, 2026
@Phemisters Phemisters deleted the update-filmic-defaults-and-controls-doc branch January 26, 2026 19:49
@Phemisters
Copy link
Contributor Author

#742 also attempts to make some changes to some of the same documentation. On reflection I think this is probably closer to what I would have done, but it doesn't mention the change to the default latitude. I think I'll go ahead and merge this one and we can perhaps revisit latitude in another PR.

Thanks!

Thank you - I had not read other PR.

@elstoc
Copy link
Contributor

elstoc commented Jan 26, 2026

I'm just working on the remaining changes now

@elstoc
Copy link
Contributor

elstoc commented Jan 26, 2026

Further updates in a643b0f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants