EffEPi / fabric.curvedText

Allows you to create curved text - extension to fabric.js
43 stars 41 forks source link

Group select causes other elements to shift. #23

Closed jhundley9109 closed 8 years ago

jhundley9109 commented 8 years ago

See http://jsfiddle.net/jhundley9109/z7dw9ekn/

By simply selecting all the elements on the canvas, some, actually all elements except one shifts. When clicking off the the group select the elements go back to place. The item that does not shift seems to be based on the index of the item in the canvas.getActiveObjects() array. This is why I put a obj.bringToFront() in the fiddle.

Jochuaf commented 8 years ago

Hi Effepi, Your code works really great except for grouping as mentioned by jhundley9109. Plus, rotation mess up after 2 tries. Is there any solution for this problem? Thanks in advance. ps: I mixed up 2 conversations and answered #20 by mistake.

jhundley9109 commented 8 years ago

I have figured out the cause of this issue. During the render function, the ctx.save() and ctx.restore() are commented out. These need to be uncommented. Also, the line with the this.clipTo&&ctx.restore() probably needs modified.

EffEPi commented 8 years ago

Can you test it again and maybe create a pull request for it?

EffEPi commented 8 years ago

I believe that the problem is fixed with the latest merge