[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-bugs-dist
Subject: [krita] [Bug 348444] Currently active preset highlighting lost, by switching from tag "All" > to any
From: mvowada <freebox64 () tiscali ! it>
Date: 2015-06-30 22:19:54
Message-ID: bug-348444-17878-TVvcwZBWYO () http ! bugs ! kde ! org/
[Download RAW message or body]
https://bugs.kde.org/show_bug.cgi?id=348444
mvowada <freebox64@tiscali.it> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|FIXED |---
Status|RESOLVED |REOPENED
--- Comment #7 from mvowada <freebox64@tiscali.it> ---
(In reply to Stefano Bonicatti from comment #6)
> Fix selection bugs in the preset docker
>
> Now is possible to select again a brush that is the only one in the
> selected tag category.
>
> When switching tag categories if the selected preset is present in the new
> category, it will be selected, otherwise no preset will be, visually,
> selected,
> but the active one won't be changed, so the user can still paint.
>
> Added several checks to avoid unneeded executions and event emitting.
> Especially now it shouldn't be possible to break the previous preset
> selection/shortcut anymore.
> Related: bug 348120
> Differential Revision: https://phabricator.kde.org/D124
>
> M +25 -6 krita/ui/kis_paintop_box.cc
> M +22 -13 libs/widgets/KoResourceItemChooser.cpp
> M +1 -1 libs/widgets/KoResourceItemChooser.h
> M +5 -6 libs/widgets/KoResourceModel.cpp
> M +2 -2 libs/widgets/KoResourceModel.h
>
> http://commits.kde.org/calligra/f5df93f0f2017e12f2b5670ad3df4f53969c5f28
Hello Stefano, thank you for this commit :-)
I've double checked with a more recent version of Krita: 2.9.5 (git 9fe0557).
Unfortunately it seems that both these issues are still reproducible: Bug
348444 and Bug 348120.
Please see the video below. Reopening.
--
You are receiving this mail because:
You are watching all bug changes.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic