Bug 476924 - Brush editor: errors when toggling "Share curves across all settings"
Summary: Brush editor: errors when toggling "Share curves across all settings"
Status: REPORTED
Alias: None
Product: krita
Classification: Applications
Component: Usability (show other bugs)
Version: git master (please specify the git hash!)
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-11-13 06:54 UTC by paleh
Modified: 2023-11-13 06:54 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
video recording of the bug (475.64 KB, video/webm)
2023-11-13 06:54 UTC, paleh
Details

Note You need to log in before you can comment on or make changes to this bug.
Description paleh 2023-11-13 06:54:31 UTC
Created attachment 163107 [details]
video recording of the bug

STEPS TO REPRODUCE
1. Open the brush editor (e.g. for Basic-5 Size)
2. Select an active parameter (e.g. Size)
3. Select an active sensor (e.g. Pressure) and modify its curve.
4. Uncheck "Share curves across all settings"
5. Observe the curves for all the other sensors of that parameter.

OBSERVED RESULT
Step 4 makes the curve for the selected sensor revert to its initial state, which is different from the behavior of "share curve across all settings" in 5.1.5.
In step 5 we can see that the curves for some other sensors have unexpected shapes that we never defined. The set of affected sensors across different tries seems to be non-deterministic.
Re-activating "Share..." on an affected sensor creates a new unexpected curve for that sensor and, contrary to the purported function of that checkbox, doesn't affect some other curves.

EXPECTED RESULT
Toggling "Share..." on sensor X should not change the curve for sensor X.
Activating "Share..." on sensor X should set all the other sensors' curves to that of sensor X.
Deactivating "Share..." on sensor X should not affect any of the curves. The behavior in 5.1.5 is keeping the curves for all the active sensors intact, while replacing the curves for the inactive sensors by the default "f(x)=x".

SOFTWARE/OS VERSIONS
Linux: Ubuntu 22.04
Qt Version: 5.15.7
Appimage

ADDITIONAL INFORMATION
krita-5.3.0 (git d8e226389d)
appeared in 5.2.0