Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Small refactorings, performance enhancement and delicious button #73

Closed
wants to merge 14 commits into from

Conversation

czerasz
Copy link

@czerasz czerasz commented Jan 8, 2015

I added a Gruntfile.js which makes development easier.
It also adds support for autoprefixer and SVG sprites which enables browser caching.

@dbox
Copy link
Contributor

dbox commented Jan 8, 2015

Cool. We'll look this over. We definitely don't want to have to require Ruby or Node, but having grunt support is great.

Thanks for contributing!

Alt attributes are not supported on svg tags. Use role="img" aria-
label="" instead.
@czerasz
Copy link
Author

czerasz commented Jan 13, 2015

Hi @dbox,

Made an additional small change. Maybe we can take it with this pull request.

@dbox
Copy link
Contributor

dbox commented Jan 13, 2015

Can you resolve the conflicts and resubmit? Thanks!

@czerasz
Copy link
Author

czerasz commented Jan 13, 2015

Synced my feature/delicious with your recent master changes.

@czerasz
Copy link
Author

czerasz commented Jan 29, 2015

Hi,

Sorry for the late response.

Good question. I don't think you can edit the img/icons.svg with Illustrator because it contains only symbols.

Tried to goggle for "Illustrator import svg symbol" and also checked in the Illustrator's symbol panel but didn't find anything :-(.

But there is hope. One could add xmlns:xlink="http://www.w3.org/1999/xlink" to the svg header tag. And then also add <use> tags with 0 opacity.

Example below:

<use xlink:href="#icon-linkedin" opacity="0"/>

This will allow You to preview the svg in Illustrator. I'm not sure what happens after you save it.

@IanCaunce
Copy link
Contributor

It's also worth noting that using an svg spritesheet like this isn't supported in any version of Internet Explorer or Android 3.0 - 4.3

There is a polyfill for IE but not for Android.

https://github.com/jonathantneal/svg4everybody

@IanCaunce
Copy link
Contributor

Also with regards to illustrator, would the correct workflow be: edit the individual svg's and use the grunt task to recreate the icons.svg spritesheet rather than editing it directly

dbox added a commit that referenced this pull request May 3, 2015
- Shiny new gulp/browser-sync dev environment /via @connorwyatt
- Updated Facebook icon
- New delicious icon
- Added vk icon per #80 
- fixed color for hackernews
- Added delicious icon via @czerasz per #73 
- Optimized code and inline svgs
@dbox
Copy link
Contributor

dbox commented May 3, 2015

Hi @czerasz. We decided to go with gulp instead of grunt, but I was able to add your delicious button. Pull down the latest and check it all out. I think this will be better fit your workflow now. Thanks again!

@dbox dbox closed this May 3, 2015
@czerasz
Copy link
Author

czerasz commented May 3, 2015

Looks great. Well done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants