WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 years ago

#39529 closed feature request (wontfix)

Need to add target blank for external site and RSS

Reported by: sagarprajapati Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Widgets Keywords: has-patch target-blank
Focuses: accessibility Cc:
PR Number:

Description

Hi,

We should add target="_blank" for the meta widget sidebar RSS link and wordpress.org link. Because after click on these links user leaves the site.

Thanks

Attachments (2)

attachment.png (6.5 KB) - added by sagarprajapati 3 years ago.
39529.patch (1.6 KB) - added by sagarprajapati 3 years ago.

Download all attachments as: .zip

Change History (7)

#1 @sagarprajapati
3 years ago

  • Keywords has-patch added

Hi,

I have attached patch for the above suggestion.

Thanks

Last edited 3 years ago by sagarprajapati (previous) (diff)

#2 @afercia
3 years ago

  • Focuses accessibility added
  • Keywords target-blank added
  • Version 4.7 deleted

#3 @swissspidy
3 years ago

Citing #38680 (another ticket with the target-blank keyword):

About the front-end, personally I think WordPress shouldn't encourage the usage of target="_blank" so it shouldn't be included in the default output.

I tend to agree with that, especially since these links have always been like that and we're getting of target="_blank" in the admin as well.

However, the ongoing discussion has focused mainly on the admin screens so far, so there's the need to start taking into consideration what is the better option for the front-end too. Discussion can happen on this ticket or also on Slack, any feedback welcome.

+1.

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.


3 years ago

#5 @afercia
3 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

Closing as wontfix for the reasons explained here and in the related ticket #38680.

Note: See TracTickets for help on using tickets.