Bug 457744

Summary:	Changing opacity in the toolbar set opacity in pop-up palette brush to 1.0
Product:	[Applications] krita	Reporter:	thetwo <thetwo222>
Component:	Usability	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	dimula73, halla
Priority:	NOR
Version:	5.1.0-RC1
Target Milestone:	---
Platform:	Other
OS:	Other
Latest Commit:	https://invent.kde.org/graphics/krita/-/commit/50295544108c06fd19a0b303b0dc1e28dd434173	Version Fixed In:
Sentry Crash Report:
Attachments:	gif gif

Description thetwo 2022-08-11 04:44:21 UTC

Created attachment 151240 [details]
gif

SUMMARY
https://krita-artists.org/t/opacity-slider-in-pop-up-palette-brush-editor-is-inaccurate/45509

A user of Ka forum encountered this problem on 5.1 beta 2. I can confirm it on RC1.

Comment 1 Halla Rempt 2022-08-11 08:08:48 UTC

I think we're going to have to postpone this until https://invent.kde.org/graphics/krita/-/merge_requests/1334 gets implemented -- the hud uses the widget from the same properties widget that is used in the preset editor, which does track the global opacity correctly...

Comment 2 thetwo 2024-08-18 07:13:45 UTC

Created attachment 172721 [details]
gif

To supplement this bug:

When I make changes to data on the brush interface, the 'pop-up palette' cannot synchronize correctly

I found that the reason is that 'pop-up palette' will round up the values passed from other places. So 'size' doesn't seem like a big problem. The term 'spacing' is an integer ranging from 1 to 10. And 'opacity' and 'flow' only have 0 and 1

Comment 3 Freya Lupen 2024-08-22 19:33:27 UTC

Git commit f7fa3831aa0164218f0715e4863654fe7ca098bc by Freya Lupen.
Committed on 22/08/2024 at 19:32.
Pushed by freyalupen into branch 'master'.

Fix On-Canvas Brush Editor's decimal sliders

The externally changed values were being converted to integers
by mistake.
Related: bug 447800

M  +1    -1    plugins/dockers/brushhud/kis_uniform_paintop_property_widget.cpp

https://invent.kde.org/graphics/krita/-/commit/f7fa3831aa0164218f0715e4863654fe7ca098bc

Comment 4 Freya Lupen 2024-08-22 19:45:46 UTC

Git commit 50295544108c06fd19a0b303b0dc1e28dd434173 by Freya Lupen.
Committed on 22/08/2024 at 19:45.
Pushed by freyalupen into branch 'krita/5.2'.

Fix On-Canvas Brush Editor's decimal sliders

The externally changed values were being converted to integers
by mistake.
Related: bug 447800

M  +1    -1    libs/ui/brushhud/kis_uniform_paintop_property_widget.cpp

https://invent.kde.org/graphics/krita/-/commit/50295544108c06fd19a0b303b0dc1e28dd434173