Skip to content

agx: clarify in toe/shoulder warning#20133

Closed
kofa73 wants to merge 1 commit intodarktable-org:masterfrom
kofa73:agx-toe-shoulder-warning-tooltip-update
Closed

agx: clarify in toe/shoulder warning#20133
kofa73 wants to merge 1 commit intodarktable-org:masterfrom
kofa73:agx-toe-shoulder-warning-tooltip-update

Conversation

@kofa73
Copy link
Contributor

@kofa73 kofa73 commented Jan 11, 2026

Tooltip update to address misunderstandings

…ot an error condition, the module is still working fine
@kofa73
Copy link
Contributor Author

kofa73 commented Jan 11, 2026

@TurboGit: trying to send smaller, more focused PRs to be merged to master later

"without inverting the shoulder (forcing it to bend upwards), it would be\n"
"impossible to reach target white with the selected contrast and pivot position.\n"
"this is not an error, the module is still working correctly.\n"
"if you see artifacts, or wish to regain control over the shoulder, try the following:\n"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See also #20040. I'd recommend turning this final paragraph into a bullet list for readability.

@TurboGit TurboGit added this to the 5.6 milestone Jan 13, 2026
@TurboGit TurboGit added feature: enhancement current features to improve scope: translation / strings make darktable international scope: UI user interface and interactions and removed feature: enhancement current features to improve labels Jan 13, 2026
@TurboGit
Copy link
Member

TurboGit commented Feb 9, 2026

Needs conflict resolution. TIA.

@kofa73
Copy link
Contributor Author

kofa73 commented Feb 13, 2026

Superseded by #20040

@kofa73 kofa73 closed this Feb 13, 2026
@kofa73 kofa73 deleted the agx-toe-shoulder-warning-tooltip-update branch February 13, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: translation / strings make darktable international scope: UI user interface and interactions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants