Bug 24520 - Share via Twitter option on OPAC is out of line with rest of Share options
Summary: Share via Twitter option on OPAC is out of line with rest of Share options
Status: RESOLVED DUPLICATE of bug 26069
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Lucas Gass
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-01-27 15:28 UTC by Kelly McElligott
Modified: 2020-09-04 21:05 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Kelly McElligott 2020-01-27 15:28:57 UTC
It would appear as most Share options, email, facebook, linkedin are Font Awesome - however, the Tweet option is not.  Therefore when the Tweet Options appears next to the other share options, it is out of line with them.

It would be nice to make the Tweet option also Font Awesome to keep them consistent and look nice.
Comment 1 Owen Leonard 2020-01-27 15:45:53 UTC
The tweet link is a Font Awesome icon, but there is also some JavaScript which loads Twitter's version of the button in its place. I must have it blocked in my browser because I didn't see it until I switched to another one.

The solution is to remove the Twitter-specific JS:

Line 1410:

[% IF ( SocialNetworks ) %]
    <script>!function(d,s,id){var js,fjs=d.getElementsByTagName(s)[0];if(!d.getElementById(id)){js=d.createElement(s);js.id=id;js.src="//platform.twitter.com/widgets.js";fjs.parentNode.insertBefore(js,fjs);}}(document,"script","twitter-wjs");</script>
[% END %]
Comment 2 Lucas Gass 2020-09-04 21:05:24 UTC
This problem is fixed by Bug 26069.

Making this resolved

*** This bug has been marked as a duplicate of bug 26069 ***