Bug 390583 - Clicking/Selecting some of the parameters/options on the brush editor toggles the presets as modified
Summary: Clicking/Selecting some of the parameters/options on the brush editor toggles...
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: Brush engines (show other bugs)
Version: git master (please specify the git hash!)
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Scott Petrovic
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-02-16 20:14 UTC by Quiralta
Modified: 2018-03-03 13:46 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Quiralta 2018-02-16 20:14:00 UTC
On the brush editor, a simple clicking/selecting a parameter/option will change the preset status as modified, it seems random but mostly the ones that are already selected produce this behavior. The following video shows the issue.

https://drive.google.com/file/d/1AtGZg8q4DyVXa8viqRw1XyqoPJ_mKjxa/view?usp=sharing
Comment 1 Scott Petrovic 2018-03-02 18:10:08 UTC
Git commit d26f9f9706f06e833d2f72f14c57c2bd78b310c8 by Scott Petrovic.
Committed on 02/03/2018 at 18:09.
Pushed by scottpetrovic into branch 'master'.

M  +11   -3    libs/ui/kis_categorized_list_model.h
M  +4    -2    libs/ui/kis_paintop_settings_widget.cpp
M  +20   -26   libs/ui/widgets/kis_categorized_list_view.cpp
M  +2    -6    libs/ui/widgets/kis_categorized_list_view.h
M  +2    -2    libs/ui/widgets/kis_cmb_composite.cc
M  +1    -1    libs/ui/widgets/kis_paintop_list_widget.cpp

https://commits.kde.org/krita/d26f9f9706f06e833d2f72f14c57c2bd78b310c8
Comment 2 Scott Petrovic 2018-03-02 18:10:56 UTC
Git commit 489a854dd35961e6c8d1c17889f3976f5c4bb504 by Scott Petrovic.
Committed on 02/03/2018 at 18:10.
Pushed by scottpetrovic into branch 'krita/4.0'.

(cherry picked from commit d26f9f9706f06e833d2f72f14c57c2bd78b310c8)

M  +11   -3    libs/ui/kis_categorized_list_model.h
M  +4    -2    libs/ui/kis_paintop_settings_widget.cpp
M  +20   -26   libs/ui/widgets/kis_categorized_list_view.cpp
M  +2    -6    libs/ui/widgets/kis_categorized_list_view.h
M  +2    -2    libs/ui/widgets/kis_cmb_composite.cc
M  +1    -1    libs/ui/widgets/kis_paintop_list_widget.cpp

https://commits.kde.org/krita/489a854dd35961e6c8d1c17889f3976f5c4bb504
Comment 3 Scott Petrovic 2018-03-02 18:16:54 UTC
That bug should be fixed. 

Along with that fix, I also talked with some artists and made the "lock" status only available with a right click. It was previously done with double clicking, but that was accidentally done a lot, so you need to be more explicit now when setting the lock status.
Comment 4 Quiralta 2018-03-02 20:58:32 UTC
This bug still happening for me on the master branch, exactly the same way as originally stated, on a clean install, build.

Let me know if I should test it differently and whether to re-open or investigate further.
Comment 5 Quiralta 2018-03-03 13:46:15 UTC
Never mind previous comment, rebuild and now everything is fixed :)