Closed Brennaleker closed 7 years ago
:+1: That looks really nice so far! I love how the colors seem to match the app.
Those options make sense to me, but that might be something to ask the client about when we show them the app–they might have some other favorites.
This looks good to me, too. What are the next steps in completing this feature, @Brennaleker?
@micahbales the functionality works as expected locally but when I looked at the changes on the staging site I noticed that while the icons are there the onclick functionality is not due to JS throwing an UncaughtReferenceError "SocialShareButton is not defined". I'd like to dig into that before merging. I should have some time tonight or tomorrow night to look into it.
Sounds good! Thanks, Brenna.
I chose email, Facebook and Twitter as sharing options but more can be found here