-
Notifications
You must be signed in to change notification settings - Fork 39
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
Added pinterest support, facebook share dialog, google snippets, reactivity #32
Conversation
Added pinterest support. Modified api so that users have control over order and size of social icons. Loop to reduce code duplication.
Facebook now uses share dialog when there's a valid app_id Changed sites array to an object for future configurations Pattern match test instead of comparing for possible port changes
Extended site configuration Added explicit site order array Added reactivity when data context changes
Doc is intentionally left incomplete for less conflicts when changes will be merged back.
Silence that annoying fb-root warning
@queso You should merge this in 👍 |
@rgoomar have you tested it in an app yet? |
Not just yet, but from going through the code, the logic seems to be in place and I don't see why it should break anything really. I can test it later tonight though. |
That would be awesome, my main focus is velocity atm and I will be wrapped up in that all night. |
+1 |
Added pinterest support, facebook share dialog, google snippets, reactivity
Need to add to the README that init is now explicitly required. Plus googleplus color is broken. |
Changes:
Let me know what you think works and what doesn't. Thanks :)