| Summary: | Custom color smudge engine brush doesn't blend while there is selection | ||
|---|---|---|---|
| Product: | [Applications] krita | Reporter: | soitismyankh |
| Component: | General | Assignee: | Dmitry Kazakov <dimula73> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | dimula73, janeelf, kapyia.art, lynx.mw+kde |
| Priority: | NOR | ||
| Version: | 4.2.8 | ||
| Target Milestone: | --- | ||
| Platform: | Microsoft Windows | ||
| OS: | Microsoft Windows | ||
| Latest Commit: | https://invent.kde.org/graphics/krita/commit/3f8f763bdbb9cf1b73cd365f1d972574b724f975 | Version Fixed In: | |
| Sentry Crash Report: | |||
| Attachments: | Brush that was used in reproducing the bug | ||
|
Description
soitismyankh
2020-07-03 20:15:18 UTC
I'm seeing it here with the bundled "k)_Blender_Blur" brush, no custom brush required. It hardly does anything inside a selection with 4.3 git (Linux)... I'm alsmost sure it used to work, screams for some regression testing. Sorry, I need to pull back the confirmation, at least if version 4.2.8 as affection version is correct. Bisecting revealed a similar problem caused by 2b5ed5a0bdb23812c4b64e4ed7d657a5acff85ca but this commit is not in any released version yet, and it broke several brush engines entirely. And I'm afaid something went wrong when attaching the brush preset, it says "Soft_smudge.kpp (image/png)" so it's not a .kpp file but just the brush icon, I can't reproduce the issue that way or figure out what's different about your brush. Also, try the latest krita release please. Okay I'm stupid, .kpp files are just PNGs with additional metadata, browser just confused me... So, due to the regression currently in git code, I tested with 4.3.0 appimage and eliminated all brush settings until I found the issue. The problem is clearly the Smudge Lenght => Strength parameter. When lowered to ~15% or lower, the difference with and wihout selection becomes very obvious with all smudge brushes that don't add color. Shape/size/type of the selection doesn't seem to matter, even selecting the whole canvase (Ctrl+A) drastically reduces the smudge effect. I guess it just hasn't been discovered earlier because such low values are not very common for smear/blender brushes and it's hard to notice at higher values and/or when the brush adds color too. Git commit a899f2f42800f546105e2fbe53d37e4b1b355b45 by Dmitry Kazakov. Committed on 27/07/2020 at 14:08. Pushed by dkazakov into branch 'master'. Fix painting with colorsmudge brush when a selection is active Selection crops destination rect, not the source one Related: bug 394439 M +17 -8 libs/image/kis_painter.cc https://invent.kde.org/graphics/krita/commit/a899f2f42800f546105e2fbe53d37e4b1b355b45 Git commit c4d0924a66b8dbc6e61f38aa5fb61cc5c03ac444 by Dmitry Kazakov. Committed on 27/07/2020 at 14:13. Pushed by dkazakov into branch 'krita/4.3'. Fix painting with colorsmudge brush when a selection is active Selection crops destination rect, not the source one Related: bug 394439 M +17 -8 libs/image/kis_painter.cc https://invent.kde.org/graphics/krita/commit/c4d0924a66b8dbc6e61f38aa5fb61cc5c03ac444 The commit fixed painting is general, though there is still a bit weird behavior, because selection makes smudging effect much lighter. I don't know what causes it yet. *** Bug 396479 has been marked as a duplicate of this bug. *** *** Bug 426929 has been marked as a duplicate of this bug. *** Git commit 4463699df4c5d8d9ddc9e056286568b40d5fbc99 by Dmitry Kazakov. Committed on 28/09/2020 at 09:43. Pushed by dkazakov into branch 'krita/4.3'. Fix double application of selection in ColorSmudge brush When doing bitBltWithFixedSelection() the passed mask is merged with the current selection. We shouldn't try to reuse d->paramInfo for the merge because it may have a custom opacity/flow set. M +30 -18 libs/image/kis_painter.cc https://invent.kde.org/graphics/krita/commit/4463699df4c5d8d9ddc9e056286568b40d5fbc99 Git commit 3f8f763bdbb9cf1b73cd365f1d972574b724f975 by Dmitry Kazakov. Committed on 28/09/2020 at 09:44. Pushed by dkazakov into branch 'master'. Fix double application of selection in ColorSmudge brush When doing bitBltWithFixedSelection() the passed mask is merged with the current selection. We shouldn't try to reuse d->paramInfo for the merge because it may have a custom opacity/flow set. M +30 -18 libs/image/kis_painter.cc https://invent.kde.org/graphics/krita/commit/3f8f763bdbb9cf1b73cd365f1d972574b724f975 |