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

Fix PointText#setPoint(point) and PointText#setPosition(point).

Browse source
This commit is contained in:
Jonathan Puckey 2011-08-02 14:31:35 +02:00
parent 0bbb4640ce
commit fe1b2ca8a0
1 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
src/text/PointText.js
View file

@ -37,7 +37,7 @@ var PointText = this.PointText = TextItem.extend(/** @lends PointText# */{
*/
initialize: function(point) {
this.base();
var point = Point.read(arguments);
point = Point.read(arguments);
this._point = LinkedPoint.create(this, 'setPoint', point.x, point.y);
this._matrix = new Matrix().translate(point);
},
@ -56,10 +56,17 @@ var PointText = this.PointText = TextItem.extend(/** @lends PointText# */{
* @bean
*/
getPoint: function() {
return this._point;
// this._point is a LinkedPoint, so we can use _x and _y.
// Do not cache LinkedPoints directly, since we would not be able to
// use them to calculate the difference in #setPoint, as when it is
// modified, it would hold new values already and only then cause the
// calling of #setPoint.
return LinkedPoint.create(this, 'setPoint', this._point._x,
this._point._y);
},
setPoint: function(point) {
this._changed(Change.GEOMETRY);
this._transform(new Matrix().translate(
Point.read(arguments).subtract(this._point)));
},
@ -67,7 +74,7 @@ var PointText = this.PointText = TextItem.extend(/** @lends PointText# */{
// TODO: Position should be the center point of the bounds but we currently
// don't support bounds for PointText.
getPosition: function() {
return this._point;
return this.getPoint();
},
setPosition: function(point) {