comic.js
comic.js copied to clipboard
balint42
issue with complex paths...
Hello, I love your library. However, with some complex curves, I came across an error on line 12685. The problem is that list == undefined and as such !list.length throws an error.
My fix was to edited the code to:
if(!list ) list = {};
else if(!list.length) list = list._list ;
Hey @mugendi, thanks a lot for your feedback, I'm happy to hear you like comic.js. Now as for the problem you reported:
You are right that if on line 1285 list should be undefined this will cause an error. However, for this reason on line 1284 there is list = path.pathSegList || newPath.pathSegList and before newPath is created on line 1281 if ! path.pathSegList. What happens here with near certainty is the Chrome bug described in this open issue.
The conclusion is that this is a sad situation for Chrome users and I will try to make a new release soon that includes an own replacement for the missing Chrome API.
