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

make helper mouse functions have consistent interface

Browse source
This commit is contained in:
DD Liu 2017-08-22 15:57:12 -04:00
parent 6ede893585
commit b55b23e446
3 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/containers/blob/blob.js
View file

@ -82,9 +82,9 @@ class Blobbiness {
blob.resizeCursorIfNeeded(event.point); blob.resizeCursorIfNeeded(event.point);
if (event.event.button > 0) return; // only first mouse button if (event.event.button > 0) return; // only first mouse button
if (blob.brush === Blobbiness.BROAD) { if (blob.brush === Blobbiness.BROAD) {
blob.broadBrushHelper.onBroadMouseDrag(event, blob.options); blob.broadBrushHelper.onBroadMouseDrag(event, blob.tool, blob.options);
} else if (blob.brush === Blobbiness.SEGMENT) { } else if (blob.brush === Blobbiness.SEGMENT) {
blob.segmentBrushHelper.onSegmentMouseDrag(event, blob.options); blob.segmentBrushHelper.onSegmentMouseDrag(event, blob.tool, blob.options);
} else { } else {
log.warn(`Brush type does not exist: ${blob.brush}`); log.warn(`Brush type does not exist: ${blob.brush}`);
} }
@ -102,7 +102,7 @@ class Blobbiness {
if (blob.brush === Blobbiness.BROAD) { if (blob.brush === Blobbiness.BROAD) {
lastPath = blob.broadBrushHelper.onBroadMouseUp(event, blob.tool, blob.options); lastPath = blob.broadBrushHelper.onBroadMouseUp(event, blob.tool, blob.options);
} else if (blob.brush === Blobbiness.SEGMENT) { } else if (blob.brush === Blobbiness.SEGMENT) {
lastPath = blob.segmentBrushHelper.onSegmentMouseUp(event); lastPath = blob.segmentBrushHelper.onSegmentMouseUp(event, blob.tool, blob.options);
} else { } else {
log.warn(`Brush type does not exist: ${blob.brush}`); log.warn(`Brush type does not exist: ${blob.brush}`);
} }

2
src/containers/blob/broad-brush-helper.js
View file

@ -30,7 +30,7 @@ class BroadBrushHelper {
this.lastPoint = this.secondLastPoint = event.point; this.lastPoint = this.secondLastPoint = event.point;
} }
onBroadMouseDrag (event, options) { onBroadMouseDrag (event, tool, options) {
const step = (event.delta).normalize(options.brushSize / 2); const step = (event.delta).normalize(options.brushSize / 2);
// Move the first point out away from the drag so that the end of the path is rounded // Move the first point out away from the drag so that the end of the path is rounded

2
src/containers/blob/segment-brush-helper.js
View file

@ -36,7 +36,7 @@ class SegmentBrushHelper {
this.lastPoint = event.point; this.lastPoint = event.point;
} }
onSegmentMouseDrag (event, options) { onSegmentMouseDrag (event, tool, options) {
if (event.event.button > 0) return; // only first mouse button if (event.event.button > 0) return; // only first mouse button
const step = (event.delta).normalize(options.brushSize / 2); const step = (event.delta).normalize(options.brushSize / 2);