scratchfoundation/scratch-paint
1
0
Fork 0
mirror of https://github.com/scratchfoundation/scratch-paint.git synced 2024-12-22 21:42:30 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Merge pull request #87 from fsih/fixApplyStrokeColor

Browse source 
Fix color matching logic in apply stroke color
This commit is contained in:
DD Liu 2017-10-23 09:13:19 -04:00 committed by GitHub
commit 7e5c14a982
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/helper/style-path.js
View file

@ -50,6 +50,11 @@ const applyFillColorToSelection = function (colorString, onUpdateSvg) {
}
};
const _strokeColorMatch = function (item, incomingColor) {
return (!item.strokeColor && !incomingColor) ||
(item.strokeColor && incomingColor && item.strokeColor.toCSS() === new paper.Color(incomingColor).toCSS());
};
/**
* Called when setting stroke color
* @param {string} colorString New color, css format
@ -65,8 +70,7 @@ const applyStrokeColorToSelection = function (colorString, onUpdateSvg) {
if (child.children) {
for (const path of child.children) {
if (!path.data.isPGGlyphRect) {
if ((path.strokeColor === null && colorString) ||
path.strokeColor.toCSS() !== new paper.Color(colorString).toCSS()) {
if (!_strokeColorMatch(path, colorString)) {
changed = true;
path.strokeColor = colorString;
}
@ -80,14 +84,12 @@ const applyStrokeColorToSelection = function (colorString, onUpdateSvg) {
}
}
} else if (!item.data.isPGGlyphRect) {
if ((item.strokeColor === null && colorString) ||
item.strokeColor.toCSS() !== new paper.Color(colorString).toCSS()) {
if (!_strokeColorMatch(item, colorString)) {
changed = true;
item.strokeColor = colorString;
}
}
} else if ((item.strokeColor === null && colorString) ||
item.strokeColor.toCSS() !== new paper.Color(colorString).toCSS()) {
} else if (!_strokeColorMatch(item, colorString)) {
changed = true;
item.strokeColor = colorString;
}