https://bugs.kde.org/show_bug.cgi?id=383909

Tymond <tamtamy.tym...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/kde/
                   |                            |krita/commit/47a5716210f2f3
                   |                            |0c3ae4c9ff22e590d1a25b877a
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #3 from Tymond <tamtamy.tym...@gmail.com> ---
Git commit 47a5716210f2f30c3ae4c9ff22e590d1a25b877a by Agata Cacko.
Committed on 14/08/2019 at 19:49.
Pushed by tymond into branch 'master'.

Fix curve change despite 'Use same curve' checked

Before this commit, if you have a preset like Fill_circle
that has "Use the same curve" checkbox checked and you were
clicking on other sensors, the curve would be back to the default
curve (the one that other sensors had) if you switch back and forth
between some sensors (like Pressure -> X-Tilt-> Pressure).

Also before this commit if you have the default uprising curve
and you switched Rotation -> Tilt elevation, the curve would change.

This commit fixes both behaviours (first by adding transferCurve(),
second by switching the line in kis_curve_widget).

This commit also includes two slot functions instead of one function
that were used as a slot for two different signals. It helps with
debugging signals and keeping stuff clear and ordered.

There is one problem left: if you switch Tilt elevation -> Distance
(using the 'hill' curve), the preset is set to modified even though
the curve doesn't change.
BACKPORT:krita/4.2

M  +1    -1    libs/ui/widgets/kis_curve_widget.cpp
M  +18   -2    plugins/paintops/libpaintop/kis_curve_option_widget.cpp
M  +3    -0    plugins/paintops/libpaintop/kis_curve_option_widget.h

https://invent.kde.org/kde/krita/commit/47a5716210f2f30c3ae4c9ff22e590d1a25b877a

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to