scratchfoundation/paper.js
1
0
Fork 0
mirror of https://github.com/scratchfoundation/paper.js.git synced 2025-01-19 14:10:14 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

The internal Item#_position is no longer a LinkedPoint.

Browse source 
Make sure it stays that way when it's kept up-to-date in Item#transform().
This commit is contained in:
Jürg Lehni 2011-12-20 22:35:06 +01:00
parent 0592038b9f
commit 8888ca912a
1 changed files with 4 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/item/Item.js
View file

@ -1852,19 +1852,13 @@ function(name) {
var rect = bounds[key];
// Transform without notifying the item of changes
bounds[key] = matrix._transformBounds(rect, rect, true);
// If we have cached 'bounds', update _position again, by
// linking it to it
// TODO: If LinkedPoint would not just sync writes, but reads
// too, we could do this: this._position = position;
// This is a bug currently in Paper.js, that should be fixed,
// but can only really be handled properly using versioning...
// If we have cached 'bounds', update _position again
if (key == 'bounds')
this._position = rect.getCenter();
this._position = rect.getCenter(true);
}
} else if (position) {
// Transform position as well. Do not notify _position of
// changes, since it's a LinkedPoint and would cause recursion!
this._position = matrix._transformPoint(position, position, true);
// Transform position as well.
this._position = matrix._transformPoint(position, position);
}
// PORT: Return 'this' in all chainable commands
return this;