CSS for polygon no longer needed

kni-labs / rrssb

:arrow_right_hook: Ridiculously Responsive Social Sharing Buttons

https://rrssb.netlify.com/

MIT License

3.4k stars 424 forks source link

CSS for polygon no longer needed #132

Closed AdamPS closed 9 years ago

AdamPS commented 9 years ago

SVG have now been optimised to use path. Can drop polygon, saves a couple of kB of CSS.

I could do a pull request, but let's wait for the outcome of the discussion of the future of different branches first.

dbox commented 9 years ago

Thanks! Go ahead and pull request to dev and I'll get it merged up. (Just getting a little hairy in this in-between time. Don't want to have to recreate a bunch of work in 2.0)

dbox commented 9 years ago

So even more optimizations since the last pull request?

AdamPS commented 9 years ago

I've now added this change to my other pull request. I've just changed the SCSS so far. I don't use the tools installed to regenerate the CSS from that. I could do a simple search and replace by hand, or maybe you could rerun the regeneration process.