-
Notifications
You must be signed in to change notification settings - Fork 381
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
Fix for Email Button, Added Tooltips on Buttons, Added Active CSS Class to Button Itself, and More #300
Comments
Downloaded your file, but the button is NOT displaying in Firefox nor Safari on El Capitan... :/ ` new share-button();<script src='dist/share-button.js'></script> <script>` The dist-folder is M.I.A also. |
Uh what are you talking about? The code you posted has nothing to do with how you're supported to use the share button... you need to load it Javascript side. |
Put this somewhere on the page in the HTML:
Then in a Javascript script:
|
Yes, and none of ´em are working for me! Tried a couple diff solution... Still no share button or network is displayed! Please see my #302 |
You're right! This share-button plugin is poorly written. I looked through the code, and it is using questionable CSS for calculations and complicated markup when it can be a lot simpler. As a result, a similar plugin was created which should provide the same functionality share-button provides in the form of a much simpler jQuery plugin. Please see here: https://github.com/earnolmartin/jquery_social_media_share_plugin Could you help me test this version? It should be in a decent state at the moment, and it works in IE9-11+, Firefox, Chrome, and Safari. |
First off, I'm not sure why your repository has a different version of share-button.js than the release does, so I cleaned this up in my repository. I don't want to create a pull request since it will complicate your branch.
So, to illustrate my changes, I committed your release version, and then I inserted my changes into it.
own3mall@1793428
I fixed the email button not working, added an active CSS class to the share button itself, made it so that clicking on a social media button doesn't toggle the share button close event, and added tooltips to the social media buttons. Hope this helps if any of my changes might make it into your branch.
Consider this an issue report for all of the things listed above except I've already fixed them in my own repo.
The text was updated successfully, but these errors were encountered: