Bug 394887

Summary:	Segment gradient editor sets opacity to 0~1% when a segment gets selected.
Product:	[Applications] krita	Reporter:	Yemachu
Component:	General	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	minor	CC:	halla
Priority:	NOR
Version First Reported In:	4.0.3
Target Milestone:	---
Platform:	unspecified
OS:	All
Latest Commit:	https://commits.kde.org/krita/d58a43e48d61382a3b2240efd69e113365519f59	Version Fixed/Implemented In:
Sentry Crash Report:

Description Yemachu 2018-05-31 10:31:54 UTC

Steps to reproduce:
 1. Click "Gradients" in the toolbar
 2. In the pop-up, click the button with the triangle pointing down (next to "+ Add...")
 3. From the options, select "Segmented Gradient"
Optional:
 4. Modify the value or leave it as is...
 5. Right click the gradient and select "Split Segment"
 6. Click on the new segment to see the opacity to be 1%.
 7. Click on the old segment to see the opacity to be 1% as well.
 8. Repeat steps 6 and 7 a few times, and the opacity becomes 0% instead.

Actual results:
When the editor window opens, the (segment of the) gradient is shown fully opaque, while the opacity on its "Left" and "Right" are set to 1% (or 0% in case of a low previous value). Switching back and forth between segments 

Expected results:
The initial opacity of the left and right side set to be 100%. When switching between segments, the value should remain unchanged.

Additional Information:
Looking at the code in the repository, the issue seems to stem from either dividing by 100 when the user modifies the value, or by not multiplying it by 100 when a new segment is selected. Found in the file "<PROJECT ROOT>/libs/ui/kis_autogradient.cc", on lines 75, 79, 142, and 155.

Comment 1 Halla Rempt 2018-06-05 08:07:26 UTC

Thanks, good catch and correct suggestion!

Comment 2 Halla Rempt 2018-06-05 08:08:02 UTC

Git commit 807aebf90487716ba39770cacf717071cdc1dc3e by Boudewijn Rempt.
Committed on 05/06/2018 at 08:07.
Pushed by rempt into branch 'krita/4.0'.

Show the correct opacity in the segment gradient editor

M  +2    -2    libs/ui/kis_autogradient.cc

https://commits.kde.org/krita/807aebf90487716ba39770cacf717071cdc1dc3e

Comment 3 Halla Rempt 2018-06-05 08:08:14 UTC

Git commit d58a43e48d61382a3b2240efd69e113365519f59 by Boudewijn Rempt.
Committed on 05/06/2018 at 08:08.
Pushed by rempt into branch 'master'.

Show the correct opacity in the segment gradient editor
(cherry picked from commit 807aebf90487716ba39770cacf717071cdc1dc3e)

M  +2    -2    libs/ui/kis_autogradient.cc

https://commits.kde.org/krita/d58a43e48d61382a3b2240efd69e113365519f59