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

No more need for special handling of 'subtract' overlaps.

Browse source 
This is now taken care of in the code that handles overlaps itself, and the additional code was causing additional issues.
This commit is contained in:
Jürg Lehni 2015-08-30 19:58:32 +02:00
parent 4379e0b0f0
commit 0651eee0c2
1 changed files with 3 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
src/path/PathItem.Boolean.js
View file

@ -175,6 +175,9 @@ PathItem.inject(new function() {
var seg = chain[j].segment,
inter = seg._intersection,
wind = winding;
// We need to handle the edge cases of overlapping curves
// differently based on the type of operation, and adjust the
// winding number accordingly:
if (inter && inter._overlap) {
switch (operation) {
case 'unite':
@ -185,25 +188,6 @@ PathItem.inject(new function() {
if (wind === 2)
wind = 1;
break;
case 'subtract':
// When subtracting, we need to reverse the winding
// number along overlaps.
// Calculate the new winding number based on current
// number and role in the operation.
var path = getMainPath(seg),
newWind = wind === 0 && path === _path1 ? 1
: wind === 1 && path === _path2 ? 2
: null;
if (newWind != null) {
// Check against the winding of the intersecting
// path, to exclude islands in compound paths, where
// the reversal of winding numbers below in overlaps
// is not required:
var pt = inter._segment._path.getInteriorPoint();
if (getWinding(pt, monoCurves) === 1)
wind = newWind;
}
break;
}
}
seg._winding = wind;