Смещение при рисовании на холсте - PullRequest
Новая
       32

Смещение при рисовании на холсте

3 голосов
/ 01 января 2012

Простое приложение для рисования. Проблема в координатах (функция redraw): они должны быть мышью, но находятся рядом с мышью в 2 раза. В чем проблема в коде? 

<html>
<head>
    <style type="text/css" media="screen">
body{
    background-color: green;
}

#workspace{
    width: 700px;
    height:420px;
    margin: 40px auto 20px auto;
    border: black dashed 1px;
}

#canvas{
    background: white;
    width: 100%;
    height: 100%;
}
    </style>
<script type="text/javascript" src="jquery-1.7.1.js"></script> 
<script type="text/javascript">
$(document).ready(
    function() {
        var context = document.getElementById('canvas').getContext("2d");
        var clickX = new Array();
        var clickY = new Array();
        var clickDrag = new Array();
        var paint;

        $('#canvas').mousedown(function(e){
            var mouseX = e.pageX - this.offsetLeft;
            var mouseY = e.pageY - this.offsetTop;
            paint = true;
            addClick(mouseX,mouseY, false);
            redraw();
        });

        $('#canvas').mousemove(function(e){
            if(paint){
                addClick(e.pageX - this.offsetLeft, e.pageY - this.offsetTop, true);
                redraw();
            }
        });

        $('#canvas').mouseup(function(e){
            paint = false;
        });

        $('#canvas').mouseleave(function(e){
            paint = false;
        });

        function addClick(x, y, dragging)
        {
            clickX.push(x);
            clickY.push(y);
            clickDrag.push(dragging);
        }

        function redraw(){
            canvas.width = canvas.width; // Clears the canvas

            context.strokeStyle = "#df4b26";
            context.lineJoin = "round";
            context.lineWidth = 5;

            for(var i=1; i < clickX.length; i++)
            {
                context.beginPath();
                if(clickDrag[i] && i){
                    context.moveTo(clickX[i-1], clickY[i-1]);
                }else{
                    context.moveTo(clickX[i], clickY[i]);
                }
                context.lineTo(clickX[i], clickY[i]);
                context.closePath();
                context.stroke();
            }

            console.log(clickX[clickX.length-1]+" "+clickY[clickX.length-1]);
        }
    });
</script>
</head>
<body>
<div id="workspace">
<canvas id="canvas"/>
</div>
</body>
</html>

1 Ответ

12 голосов
/ 01 января 2012

Вы не должны устанавливать ширину / высоту холста через CSS. Это заставляет холст растягиваться, а не повышать разрешение. Это означает, что, хотя координаты верны, они визуально окажутся где-то еще.

Например, координата х 100 будет растянута, чтобы визуально была координатой х 200 (или что-то еще; по крайней мере, она будет больше 100, поскольку она была растянута).

Вместо этого используйте: 

<canvas id="canvas" width="700" height="420" />

и удалите width: 100% в CSS.

http://jsfiddle.net/euXJC/1/

...