Bug 399065

Summary:	Halftone filter broken and Stamp option in predefined brush editor using wrong bounds due bug with KisPaintDevice::crop
Product:	[Applications] krita	Reporter:	wolthera <griffinvalley>
Component:	Tools	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	dimula73
Priority:	NOR
Version:	git master (please specify the git hash!)
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed In:

Description wolthera 2018-09-25 17:26:50 UTC

SUMMARY
Okay, this one's one I researched myself.

I was trying to figure out why Stamp in the predefined brushes looked weird.

STEPS TO REPRODUCE
1. Open an image
2. make a selection smaller than the total canvas size.
3. go to brush editor(f5)
3.1. go to brush tip
3.2. go to predefined brushes
4. Select stamp.
5. Observe that the ratio is all wrong.

STEPS TO REPRODUCE 2:
1. Open an image.
2. Apply filters->artistic->halftone filer.
3. Observe it looking weird.

After some investigation this is caused by KisPaintDevice::crop(QRect) not actually cropping the device boundaries. Using KisPaintDevice::crop(int x, int y, int w, int h) does work. The two functions also do something different.

So the reason I am not actually just writing a patch for this is because I feel that crop(QRect) is named falsely, and should be a different function. Especially as it seems it's functionality used to be the same as KisPaintDevice::crop(int x, int y, int w, int h). However, I cannot determine why it changed, so I cannot come up with a new name.

Comment 1 Dmitry Kazakov 2018-09-26 11:13:01 UTC

Hi, Wolthera!

Could you tell what exactly does it mean "3. Observe it looking weird."? On my build the filter result looks correct and the only problem of the stamp brush I see is that the thumbnail has wrong aspect ratio. But the brush itself paints correctly...

Comment 2 Dmitry Kazakov 2018-09-26 11:33:58 UTC

Git commit 8dcacba3ed052911bc41df9cac96ee4e7a95057e by Dmitry Kazakov.
Committed on 26/09/2018 at 11:33.
Pushed by dkazakov into branch 'master'.

Make Stamp brush preview be scaled correctly

M  +14   -7    plugins/paintops/libpaintop/kis_custom_brush_widget.cpp
M  +1    -0    plugins/paintops/libpaintop/kis_custom_brush_widget.h

https://commits.kde.org/krita/8dcacba3ed052911bc41df9cac96ee4e7a95057e

Comment 3 Dmitry Kazakov 2018-09-26 11:36:44 UTC

Git commit 96bc1223072279dea70a1cca204a2b7cb1366434 by Dmitry Kazakov.
Committed on 26/09/2018 at 11:36.
Pushed by dkazakov into branch 'krita/4.1'.

Make Stamp brush preview be scaled correctly

M  +14   -7    plugins/paintops/libpaintop/kis_custom_brush_widget.cpp
M  +1    -0    plugins/paintops/libpaintop/kis_custom_brush_widget.h

https://commits.kde.org/krita/96bc1223072279dea70a1cca204a2b7cb1366434

Comment 4 wolthera 2018-09-26 16:29:41 UTC

I can't reproduce the halftone problem today, how weird.