Opened on 01/07/2016 at 06:58:20 PM
Closed on 02/14/2018 at 02:10:48 PM
#3488 closed change (duplicate)
Don't hardcode social media links in first-run page
Reported by: | greiner | Assignee: | |
---|---|---|---|
Priority: | P3 | Milestone: | |
Module: | User-Interface | Keywords: | |
Cc: | Blocked By: | #2734 | |
Blocking: | Platform: | Unknown / Cross platform | |
Ready: | yes | Confidential: | no |
Tester: | Unknown | Verified working: | no |
Review URL(s): |
Description
Background
The links to our social media pages on the first-run page are currently hardcoded which means that we cannot easily update them in case they change. Usually, we use Utils.getDocLink() to create links that redirect to external pages. In #2734 we added such redirects for all of our social media pages so we should make use of those in the first-run page.
What to change
Use Utils.getDocLink() to dynamically create links to all of our social media pages and remove the existing hardcoded ones from firstRun.html.
Attachments (0)
Change History (2)
comment:1 Changed on 02/12/2018 at 10:33:42 AM by agiammarchi
comment:2 Changed on 02/14/2018 at 02:10:48 PM by greiner
- Resolution set to duplicate
- Status changed from new to closed
This is about the fallback links which are used when Adblock Plus is configured to block social media scripts. Those links are hardcoded in firstRun.html.
We're going to rewrite the first-run page in #5894 so we can tackle it as part of that one to ensure that we're not making the same mistake again.
I believe we are already using getDocLink within the click handler:
Should the link pre-set befre the click or can we just close this ticket?