Merge pull request #55 from DERAlfons/master

Move brushPreview with canvas when panning with mousewheel
This commit is contained in:
Nicola 2021-07-22 13:16:32 +02:00 committed by GitHub
commit 12ddba3924
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 4 deletions

View File

@ -37,6 +37,7 @@ window.addEventListener("mousedown", function (mouseEvent) {
}
}
else if (mouseEvent.which == 2) {
tool.pan.brushSize = currentTool.brushSize;
currentTool = tool.pan;
}
else if (currentTool.name == 'pencil' && mouseEvent.which == 3) {
@ -171,6 +172,8 @@ window.addEventListener("mouseup", function (mouseEvent) {
currentTool = currentToolTemp;
currentTool.updateCursor();
var cursorLocation = getCursorPosition(mouseEvent);
currentTool.moveBrushPreview(cursorLocation);
}, false);
@ -214,8 +217,6 @@ function draw (mouseEvent) {
//if a document hasnt yet been created or the current layer is locked, exit this function
if (!documentCreated || dialogueOpen || !currentLayer.isVisible || currentLayer.isLocked) return;
// Moving brush preview
currentTool.moveBrushPreview(cursorLocation);
// Hiding eyedropper, will be shown if it's needed
eyedropperPreview.style.display = 'none';
@ -304,6 +305,9 @@ function draw (mouseEvent) {
for (let i=1; i<layers.length; i++) {
layers[i].copyData(layers[0]);
}
// Updating cursorLocation with new layer position
lastMouseMovePos = getCursorPosition(mouseEvent);
cursorLocation = lastMouseMovePos;
}
else if (currentTool.name == 'eyedropper' && dragging && mouseEvent.target.className == 'drawingCanvas') {
let selectedColor = getEyedropperColor(cursorLocation);
@ -417,6 +421,9 @@ function draw (mouseEvent) {
}
currentLayer.updateLayerPreview();
}
// Moving brush preview
currentTool.moveBrushPreview(cursorLocation);
}
if (mouseEvent.target.className == 'drawingCanvas')

View File

@ -18,10 +18,10 @@ class Tool {
if (options.imageCursor) this.cursor = "url(\'/pixel-editor/"+options.imageCursor+".png\'), auto";
this.currentBrushSize = 1;
this.previousBrushSize = 1;
if (options.brushPreview) {
this.brushPreview = true;
this.currentBrushSize = 1;
this.previousBrushSize = 1;
}
//add to tool object so it can be referenced