fabricjs / fabric.js

Javascript Canvas Library, SVG-to-Canvas (& canvas-to-SVG) Parser
http://fabricjs.com
Other
29.16k stars 3.52k forks source link

Handles for text object don't get zoomed #8051

Closed brianp70x closed 2 years ago

brianp70x commented 2 years ago

when zooming a SVG from within the loadSVGFromString function, Text objects handles stay at original location.

https://jsfiddle.net/brianp70/xybgpcz2/167/ Just select any of the lower txt, notice the handles are above and to the left.

I'm new to JavaScript. Can I restructure the code to work properly? when zooming outside the loadSVGFromString, zoom seems to work fine, but since the SVG has not yet loaded I have no values to calculate it from. thanks for the time and patience!

ShaMan123 commented 2 years ago

Discussions are meant for these kinds of questions


https://jsfiddle.net/uv102sjy/

I suggest you work on v6 (master branch if you want to work with objects nested in groups)

brianp70x commented 2 years ago

thanks ill check out v6. i was not looking to group the svg i need to edit the text on the fly.

ShaMan123 commented 2 years ago

This is why I pointer out v6

brianp70x commented 2 years ago

Looking at the v6-alpha file… @.***

Shouldn’t version be 6.0.0a or something like that… this make it tough to know which version I have in use.

Thanks bpp

From: Shachar @.> Sent: Thursday, July 7, 2022 9:10 AM To: fabricjs/fabric.js @.> Cc: Brian Pate @.>; Author @.> Subject: Re: [fabricjs/fabric.js] Handles for text object don't get zoomed (Issue #8051)

This is why I pointer out v6

— Reply to this email directly, view it on GitHubhttps://github.com/fabricjs/fabric.js/issues/8051#issuecomment-1177589146, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZNJTQHOHMIO7CWFKR5X6PDVS3JLNANCNFSM522RWW7Q. You are receiving this because you authored the thread.Message ID: @.**@.>>

ShaMan123 commented 2 years ago

Well yes... you are right.