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
adds Twitter accounts and the hashtag (implements #1045) #1047
Conversation
egonw
commented
Feb 18, 2020
- to pages that already showed some info
- and details are now shown for events and event series too
- Twitter account
- hashtag, which points to the wmflabs hashtags-hub service
The current implementation does not indicate that the handle is for Twitter - perhaps add a Twitter logo (perhaps this t or this bird). |
Looks good to me now, also for hashtags. |
scholia/app/templates/base.html
Outdated
|
||
try { | ||
var twitterAccount = item.claims.P2002[0].mainsnak.datavalue.value; | ||
$( '#details' ).append( '<a href="https://twitter.com/' + twitterAccount + '">@' + twitterAccount + '</a> '); |
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 wonder if we might want a escape here, - just in case? I see that the above fullTextUrl does not have one...
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.
Escape of what? I think the Twitter account name is meant to give valid URLs.
I have begone to work a bit on the code here. |