| Summary: | Add shortcuts and sliders for Fade and Scatter in Auto brush mode | ||
|---|---|---|---|
| Product: | [Applications] krita | Reporter: | Tyson Tan <tysontanx> |
| Component: | Brush engines | Assignee: | Krita Bugs <krita-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | wishlist | CC: | tamtamy.tymona, tomtomtomreportingin |
| Priority: | NOR | ||
| Version First Reported In: | nightly build (please specify the git hash!) | ||
| Target Milestone: | --- | ||
| Platform: | Other | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
Tyson Tan
2021-03-31 12:04:24 UTC
I'm fine with all of that except for the 3), I'm not sure if we can just remove an option because it's not useful for a group of people. We could make it more accessible though, easier to set up in the default case while allowing other users to access the advanced case as well. The Fade X/Y options are locked by default anyway. But it's not straight forward because people are so used to look for A SINGLE SLIDER for Fade. I never discovered that option until I learned it from the Documentation, and not without intense experiment to understand what it really is -- there are so many other completely unrelated, different options with the name "Fade" after all. I think the best solution is to show a single X/Y unified slider by default, and only show separated sliders after checking a box. https://invent.kde.org/graphics/krita/-/merge_requests/1037 Here's an MR with action shortcuts for Fade and Scatter. After this, sliders for Scatter would likely be simple but Fade sliders might be a bit tricky with their dual-value nature and how their values are baked into the brush definition. Git commit 97c6084a2c36a56348030a905e114eec867c1116 by Dmitry Kazakov, on behalf of Tom Tom Tom. Committed on 23/02/2022 at 13:22. Pushed by dkazakov into branch 'master'. Add action shortcuts for configuring Flow, Fade, and Scatter M +72 -0 krita/krita.action M +51 -0 libs/image/brushengine/kis_paintop_settings.cpp M +20 -0 libs/image/brushengine/kis_paintop_settings.h M +4 -2 libs/resources/KoCanvasResourcesIds.h M +2 -0 libs/ui/KisViewManager.cpp M +85 -0 libs/ui/kis_canvas_controls_manager.cpp M +9 -0 libs/ui/kis_canvas_controls_manager.h M +1 -0 libs/ui/kis_canvas_resource_provider.cpp M +50 -0 libs/ui/kis_derived_resources.cpp M +18 -0 libs/ui/kis_derived_resources.h https://invent.kde.org/graphics/krita/commit/97c6084a2c36a56348030a905e114eec867c1116 Thank you guys! :D Tyson, can we close this report now or is there something more you want to get done here? Nothing really at this point. I thought about requesting optional sliders for Fade and Scatter in Krita's toolbar. But on second thought, we need to rely on Brush Editor's preview to control the effect. Toolbar sliders alone wouldn't be enough. It'd be nice if the toolbar sliders have built-in brush previews ... But let's not get too ahead of ourselves. :P Setting to Fixed, then. |