KDE Bug Tracking System
Home
Report New Wish or Bug
Query Existing Reports
First
Last
Prev
Next
No search results available
Search page
Bug
141390
:
Scaling with selections is broken
P
roduct
:
krita
Co
m
ponent
:
transform
Status
:
RESOLVED
Resolution
:
FIXED
Target
:
---
Version
:
1.6.1
Pr
i
ority
:
NOR
Severity
:
normal
V
otes
:
0
Description
:
Opened:
2007-02-08 11:00
Last Changed:
2007-05-17 18:16:52
Two situations: * Create a non-rectangular selection (say, two overlapping circles). Choose layer/rotate. Chose a non-90 degree rotation value. The selection is rotated correctly, but the non-selected pixels are erased. Expected behaviour: non-selected pixels are not touched. * Create another non-rectangular selection. Choose image/rotate. Chose a non-90 degree rotation value. The selection is rotated, the non selected pixels are erased (same bug as above), but additionally, the image is enlarged even if the rotated pixels do not exceed the image boundaries. Expected behaviour: non-selected pixels are not to be touched and the image stays the same size.
Comment
#1
boud valdyas org 2007-02-08 11:00:30
This one is of Casper, for when he has time again.
Comment
#2
Casper Boemann 2007-02-08 11:51:45
Well, as it's Cyrilles commit that broke it, he might be a better candidate to fix it. Not reassigning, but feel free to take over
Comment
#3
Cyrille Berger 2007-02-08 12:11:48
reverting my change does not fix the problem. but to avoid us pingponging bugs to the rotate/scaling transform worker, once 1.6.2 is released next week, I will commit the revert of my patch and (re)open the relevant bug.
Comment
#4
Casper Boemann 2007-05-17 15:18:26
SVN commit 665629 by boemann: Hopefully fix BUG:141390 M +6 -5 kis_transform_worker.cc --- branches/koffice/1.6/koffice/krita/core/kis_transform_worker.cc #665628:665629 @@ -430,17 +430,18 @@ { Q_UINT8 *data; - data = srcIt.rawData(); - memcpy(&tmpLine[i*pixelSize], data, pixelSize); + data = srcIt.rawData(); + memcpy(&tmpLine[i*pixelSize], data, pixelSize); + if(srcIt.isSelected()) + { // XXX: Should set alpha = alpha*(1-selectedness) cs->setAlpha(data, 0, 1); - if(srcIt.isSelected()) - { tmpSel[i] = 255; } - else + else { tmpSel[i] = 0; + } ++srcIt; i++; }
Comment
#5
boud valdyas org 2007-05-17 18:16:52
Fixed!
P
latform
:
Unlisted Binaries
O
S
:
Linux
K
eywords
:
People
Reporter
:
Assigned To
:
Casper Boemann
Related actions
View Bug Activity
Format For Printing
XML
Clone This Bug
Note
You need to
log in
before you can comment on or make changes to this bug.
Attachments
Add an attachment
(proposed patch, testcase, etc.)
Depends on
:
B
locks
:
Show dependency tree
-
Show dependency graph
First
Last
Prev
Next
No search results available
Search page
Actions
Reports
Requests
Reports
Bugs reported today
Bugs reported in the last 3 days
Bug reports with patches
Weekly Bug statistics
The most hated bugs
The most severe bugs
The most frequently reported bugs
The most wanted features
Junior Jobs
Report ownership counts and charts
My Account
New Account
Log In