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

Fix a bug where the selection of compound path children doe not get drawn when the ctx.currentPath optimisation is in use.

Browse source
This commit is contained in:
Jürg Lehni 2014-04-10 22:28:21 +02:00
parent 1bede6f7ec
commit 8ab4df1979
1 changed files with 13 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/project/Project.js
View file

@ -433,8 +433,8 @@ var Project = PaperScopeItem.extend(/** @lends Project# */{
// set this.
updateMatrix: true
});
for (var i = 0, l = this.layers.length; i < l; i++)
this.layers[i].draw(ctx, param);
for (var i = 0, layers = this.layers, l = layers.length; i < l; i++)
layers[i].draw(ctx, param);
ctx.restore();
// Draw the selection of the selected items in the project:
@ -444,8 +444,17 @@ var Project = PaperScopeItem.extend(/** @lends Project# */{
var size = this._scope.settings.handleSize,
half = size / 2;
for (var id in this._selectedItems) {
var item = this._selectedItems[id];
if (item._updateVersion === this._updateVersion
// Check the updateVersion of each item to see if it got drawn
// in the above draw loop. If the version is out of sync, the
// item is either not in the DOM anymore or is invisible.
var item = this._selectedItems[id],
parent = item._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 updateVersion.
versionItem = parent instanceof CompoundPath ? parent : item;
if (versionItem._updateVersion === this._updateVersion
&& (item._drawSelected || item._boundsSelected)) {
// Allow definition of selected color on a per item and per
// layer level, with a fallback to #009dec