Jump to content
Search Community

Oddly named SVG element breaking MorphSVG.pathDataToBezier

gmullinix test
Moderator Tag

Go to solution Solved by GreenSock,

Warning: Please note

This thread was started before GSAP 3 was released. Some information, especially the syntax, may be out of date for GSAP 3. Please see the GSAP 3 migration guide and release notes for more information about how to update the code to GSAP 3's syntax. 

Recommended Posts

Ran into an interesting bug the other week. I was using DrawSVG to make an element follow an SVG path (technically a polyline), and Morph SVG's pathDataToBezier feature to grab and convert the polyline to path data.


I found that if I named my polyline with a specific naming convention, MorphSVG broke. The ID name I used was path_1_1_1 and for some reason the specific sequence of a repeated underscore and a number broke the plugin.


The console log reports "ERROR: malformed path data" but the path is fine; it's the _#_#_# name that breaks things.


Please see attached Codepen for a demo.


Changing my ID name fixed everything but it took a few hours of QA (and baffled coworkers) to resolve. Since I didn't see any online reports of this issue, I figured I'd post here to bring attention to it.

See the Pen YpMKRg by gem0303 (@gem0303) on CodePen

Link to comment
Share on other sites

  • Solution

Ah, very interesting. Yeah, the issue was that there's some logic in there to sense if the string you're passing in is actually path data (like "M0,0 C3,5...") or selector text. I assumed if there were 3 or more numbers in the string, it's likely path data. But obviously in your case you've got selector text with 3 numbers in it. I've added some extra logic to better sense this type of condition, and you can test it on codepen using this URL for the plugin: https://s3-us-west-2.amazonaws.com/s.cdpn.io/16327/MorphSVGPlugin.min.js


Let me know if that works for you and I'll add it to the official zips. 


Sorry about the extra debugging hassle. Thanks for reporting this. 

  • Like 2
Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
  • Recently Browsing   0 members

    • No registered users viewing this page.
  • Create New...