-
Notifications
You must be signed in to change notification settings - Fork 9
Conversation
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.
According to https://www.npmjs.com/package/vue-social-sharing, "twitter" ShareNetwork
s take two arguments:
hashtags A list of comma-separated hashtags.
twitter-user Twitter user to mention.
The original issue asked to remove the #dandi
hashtag and replace it with the @DANDIarchive
user.
@dchiquito - I believe should be better now |
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.
@satra - could you please double check the twitter changes |
twitter looks good. could we also implement the other comment i had in the issue to use the share link in all viewports? |
ok, I moved the share icon to the title window |
@djarecka could you please merge this? If you'd like me to do it on your behalf, just let me know. |
fixes #913