Opened 8 years ago
Closed 3 years ago
#41020 closed enhancement (duplicate)
Use target="_blank" for links in WordPress news and events and updates
Reported by: | altonius | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 4.8 |
Component: | Administration | Keywords: | has-patch needs-testing |
Focuses: | Cc: |
Description
After upgrading to 4.8 Evan I clicked on the links to display info about the upgrade, however. When I did this it opened the link in the same window/tab.
Is it possible to open this in a new tab by adding the target="_blank" attribute to the link. Likewise most of the links in the Admin Dashboard under "News and Events" open in the same window/tab - would you consider opening any of these links in a separate window/tab?
To me this makes sense as they're external sites and though related, not directly related to the dashboard - and as you can head down a rabbit-hole when exploring new features / bugfixes it can be annoying to have to click back through multiple pages to get back to the dashboard.
Attachments (2)
Change History (12)
#2
@
7 years ago
- Summary changed from Use target="_blank" for links in Wordpress news and events and updates to Use target="_blank" for links in WordPress news and events and updates
#3
@
7 years ago
Following the lead of the lower links in this widget, I've added target="_blank" to the event name links, but I've also added the external link icon and associated screen reader text so that users will be aware of the external link. This is keeping in line with #23432 which promotes informing users if a link does open in an external window.
@
7 years ago
Added target="_blank", associated external icon and screen reader text along with styling so icon is in line.
#5
@
7 years ago
There's a ticket #37941 about add rel="noopener noreferrer" to any target="_blank" for security reasons.
@shedonist I would think that it should also be the case for the target="_blank"
in 41020.1.patch ?
#6
@
7 years ago
Great, thank you, @birgire. I've updated the patch to include
rel="noopener noreferrer"
to the event title links as well as the links in the footer of the widget (which also open externally).
@
7 years ago
Added rel="noopener noreferrer" to the event title links and the external links in the footer of the same widget.
#7
follow-up:
↓ 8
@
7 years ago
According to #41337 adding target="_blank"
for the event links is a wontfix.
#8
in reply to:
↑ 7
@
7 years ago
Replying to swissspidy:
According to #41337 adding
target="_blank"
for the event links is a wontfix.
Bummer that this is a wontfix since it leaves the widget completely inconsistent within itself.
#10
@
3 years ago
- Milestone Awaiting Review deleted
- Resolution set to duplicate
- Status changed from new to closed
Given the topic for this ticket is target=_blank
, I think it can be closed of as a duplicate of #41337. I know the other ticket later but has more discussion on it.
I think the suggestion of including rel="noopener noreferrer"
on links in the dashboard included in 41020.2.patch is a good privacy enhancement for external links in the dashboard, so I will open a separate ticket for that.
Related: #23432