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

Simplify Item#_isUpdate() code a bit.

Browse source
This commit is contained in:
Jürg Lehni 2014-05-07 17:27:53 +01:00
parent bb6e1e686f
commit 6aaeb81869
1 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/item/Item.js
View file

@ -3695,8 +3695,7 @@ var Item = Base.extend(Callback, /** @lends Item# */{
* anymore or is invisible.
*/
_isUpdated: function(updateVersion) {
var version = this._updateVersion,
parent = this._parent;
var parent = this._parent;
// For compound-paths, we need to use the _updateVersion of the parent,
// because when using the ctx.currentPath optimization, the children
// don't have to get drawn on each frame and thus won't change their
@ -3707,10 +3706,13 @@ var Item = Base.extend(Callback, /** @lends Item# */{
// _updateVersion of all its children will not be updated, but the
// children should still be considered updated, and selections should be
// drawn for them. Excluded are only items with _visible == false:
if (version !== updateVersion && parent && parent._visible
&& parent._isUpdated(updateVersion))
version = this._updateVersion = updateVersion;
return version === updateVersion;
var updated = this._updateVersion === updateVersion;
if (!updated && parent && parent._visible
&& parent._isUpdated(updateVersion)) {
this._updateVersion = updateVersion;
updated = true;
}
return updated;
},
_drawSelection: function(ctx, matrix, size, updateVersion) {