Bug 441682

Summary: Smudge Radius slider range doesn't change to range of diff smudge algo when reloading preset while option is selected
Product: [Applications] krita Reporter: tomtomtomreportingin
Component: Brush enginesAssignee: Tiar <tamtamy.tymona>
Status: RESOLVED FIXED    
Severity: minor CC: tamtamy.tymona
Priority: NOR    
Version: 5.0.0-beta1   
Target Milestone: ---   
Platform: Appimage   
OS: Linux   
Latest Commit: https://invent.kde.org/graphics/krita/commit/f49541f6d5738cfed63bfacd75c1a2c8c6930d2f Version Fixed In:
Sentry Crash Report:

Description tomtomtomreportingin 2021-08-29 03:21:32 UTC 
SUMMARY
The old smudge algo has a Color Radius strength range of 0-300%, while the new algo has a range of 100%. When the user enables the new algo in Smudge Length and selects Smudge Radius option, then resets the brush, the slider still has a range of 0-100%. This happens in both directions, and I'm not sure if this impacts functionality of the brush itself. The range fixes itself when selecting a new option and going back to Smudge Radius.

STEPS TO REPRODUCE
1. Select i) Wet Knife.
2. Open Brush Settings, select Smudge Length option, enable new smudge algo.
3. Select Smudge Radius option.
4. Reset brush preset.

OBSERVED RESULT
The Smudge Radius slider still has a range of 0-100%.

EXPECTED RESULT
The Smudge Radius slider range should accommodate properly on reload.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: Debian sid
KDE Plasma Version: 5.21.5
KDE Frameworks Version: 5.83.0
Qt Version: 5.12.11 (Appimage)
Comment 1 Tiar 2022-01-13 15:21:40 UTC 
Git commit 494d089943d545ee96c5f29763355c8df225cb48 by Agata Cacko.
Committed on 13/01/2022 at 14:46.
Pushed by tymond into branch 'master'.

Fix wrong color smudge radius range

Before this commit, if you:
1) took a brush like `i) Wet Paint Details`,
2) went to Smudge Length and enabled New Algorithm for smudging,
3) went to Smudge Radius (the slider range would be up to 100% because
that's the range for the new algorithm)
4) reset the brush
the range of the Smudge Radius slider would be still up to 100%
even though the old algorithm (used for that brush) is up to 300%
and should be restored when resetting.

This commit fixes this behaviour and now the range matches the
algorithm.

M  +2    -0    plugins/paintops/colorsmudge/kis_colorsmudgeop_settings_widget.cpp

https://invent.kde.org/graphics/krita/commit/494d089943d545ee96c5f29763355c8df225cb48
Comment 2 Tiar 2022-02-22 23:14:48 UTC 
Git commit f49541f6d5738cfed63bfacd75c1a2c8c6930d2f by Agata Cacko.
Committed on 22/02/2022 at 22:51.
Pushed by tymond into branch 'krita/5.0'.

Fix wrong color smudge radius range

Before this commit, if you:
1) took a brush like `i) Wet Paint Details`,
2) went to Smudge Length and enabled New Algorithm for smudging,
3) went to Smudge Radius (the slider range would be up to 100% because
that's the range for the new algorithm)
4) reset the brush
the range of the Smudge Radius slider would be still up to 100%
even though the old algorithm (used for that brush) is up to 300%
and should be restored when resetting.

This commit fixes this behaviour and now the range matches the
algorithm.
(cherry picked from commit 494d089943d545ee96c5f29763355c8df225cb48)

M  +2    -0    plugins/paintops/colorsmudge/kis_colorsmudgeop_settings_widget.cpp

https://invent.kde.org/graphics/krita/commit/f49541f6d5738cfed63bfacd75c1a2c8c6930d2f