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

Fixes Issues #305 & #304 #309

Closed
wants to merge 4 commits into from
Closed

Fixes Issues #305 & #304 #309

wants to merge 4 commits into from

Conversation

erwstout
Copy link

@erwstout erwstout commented May 9, 2016

Description

Fixes Issues #305 & #304 to allow successful npm install and Chrome svg issues.

PR Checklist

  • Clean commits
  • Passing Tests ( Testing removed, but gulp build runs correctly. )

@erwstout erwstout changed the title Fixes Issues [#305](https://github.com/carrot/share-button/issues/305) & [#304](https://github.com/carrot/share-button/issues/304] Fixes Issues #305 & #304 May 9, 2016
@drorn-aprcd
Copy link

please, someone, resolve this PR

@refi64
Copy link

refi64 commented Apr 30, 2017

refi64@c49bf2e

@erwstout
Copy link
Author

erwstout commented May 1, 2017

@kirbyfan64 whats the difference between your link and the PR I submitted?

@refi64
Copy link

refi64 commented May 1, 2017

@erwstout I added it to a fork I made... :D

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.

4 participants