-
Notifications
You must be signed in to change notification settings - Fork 182
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
Social sharing feature #398
Conversation
- Added link shortening on social sharing click, if there is an error, the default worldview link will be passed. - Formatted sharing text on each platform. - Added schema.org meta tags to index page for passing in information about the page and image to be used.
- Removed warning messages related to not being able to shorten link when clicking a share icon because if it fails, it will just use the full link. No warning needed since there is no direct user action.
- Created new image of share button & replaced font icon within tour.
// If selected during the animation, the cursor will go to the | ||
// end of the input box | ||
var updateLink = function() { | ||
$('#permalink_content').val(models.link.get()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wouldn't do any selecting of react components outside of react. e.g.
Pass a state into the component that Focuses the element. This way we are decoupling the component from Worldview and Jquery (maybe we can dispose of it in the future) @ZachTRice
this.state = {
focused: true,
selected: 'idk'
}
- Moved URL Shortener component back into WV until dialog boxes are converted to react.
@ZachTRice looks good! Just need to release a new version of WVC and update the package.json with that version. |
Connects to #140
Connects to #139
Connects to #60
**This feature needs to be tested on Master to ensure URL shortening is working properly. On UAT, Twitter & Email links will appear as the full link in the message fields but on Master these should be shortened links.