O teu código só está a adicionar um event handler ao elemento circle
, tens de adaptar para os elementos rect
também.
Podes fazer isso assim:
var rects = document.querySelectorAll('rect');
for (var i = 0; i < rects.length; i++) {
rects[i].addEventListener('mousedown', mousedown);
}
shape.addEventListener('mousedown', mousedown);
(ou podias usar delegação como no exemplo no final da resposta)
Depois tens de resolver outro problema, as cooredenadas. No setter e no getter uma vez que elementos rect
tem x
e y
, mas elementos circle
têm cx
e cy
.
Aqui fica uma sugestão:
function mousedown(evt) {
var evt = evt || window.event;
ddData.element = this;
ddData.initialX = evt.clientX;
ddData.initialY = evt.clientY;
ddData.originalX = parseFloat(ddData.element.getAttributeNS(null, this.tagName == 'rect' ? 'x' : 'cx'));
ddData.originalY = parseFloat(ddData.element.getAttributeNS(null, this.tagName == 'rect' ? 'y' : "cy"));
};
svg.onmousemove = function (evt) {
var evt = evt || window.event;
var el = ddData.element;
if (el) {
var posX = ddData.originalX + evt.clientX - ddData.initialX;
var posY = ddData.originalY + evt.clientY - ddData.initialY;
if (el.tagName == 'rect') {
//move object
el.setAttributeNS(null, "x", posX);
el.setAttributeNS(null, "y", posY);
} else {
el.setAttributeNS(null, "cx", posX);
el.setAttributeNS(null, "cy", posY);
}
}
}
###jsFiddle: http://jsfiddle.net/y2d5og5s/
Podes mesmo fazer isto com delegação (via event.target
) e assim nem precisas do ciclo for
. Neste exemplo em baixo uso .id
para decidir se o elemento pode ser movido ou não (assumindo que elementos com ID podem ser movidos), mas podias fazer isso com uma classe ou data-
field.
###jsFiddle: http://jsfiddle.net/g50L96e7/