-
Notifications
You must be signed in to change notification settings - Fork 318
Animated SVGs are well enough supported now that Skycons should be SVG. #11
Comments
good luck! |
👍 |
Was checking out the cloud in SVG: http://jsfiddle.net/s303b409/ anyone else tried to make svg icons? maybe we could get the |
@SergioCrisostomo Looks like the cloud is most of the way there... there's also a subtle stretch/shrink of each circle that makes up the cloud that'll need to go in too. Not sure when I'll have the chance to dig in, but I'll use your template as a starting place when I do! |
Just wondering if something like canvas2svg would be useful. |
@tech4him1 +1 and what's the status of this PR? @ironwallaby are we ever going to receive SVG support? |
Dark Sky is actively busy with other priorities at the moment, so this Issue is awaiting resources. We'd be happy to accept contributions, of course. |
May I ask, do animated versions of the icons have to exactly follow the canvas versions you made, or can they be slightly different? The reason I ask is that I'm currently working on animating some static icons based on the original static icons by Adam Whitcroft that the Skycons are based on. I wanted to ask if when I finish if you'd like to include the animated SVG's in this project specifically, of if you'd just like to mention it as a fork like you did the others? I'd love to be able to contribute to animating the SVG's for this, but I'm not sure if they have to be exactly the same as the canvas versions, or if they may be somewhat different. |
Way, way back when we originally made Skycons, the only way to make them was by drawing to a Canvas in JavaScript. These days, animated SVGs are well-enough supported that Skycons should really just be SVG.
This would fix several issues:
We've started a
skycons/svg
branch for working on these. (It's slow going, though, since I very rarely have the extra hours to work on it.)Already complete are the
clear_day
andclear_night
Skycons. The next big task is to implement that fancy cloud, since so many of the Skycons depend on it.The text was updated successfully, but these errors were encountered: