Make WordPress Core

Opened 5 years ago

Closed 3 years ago

#55308 closed enhancement (maybelater)

Update WordPress version URL target

Reported by: gazthink3 Owned by:
Priority: normal Milestone:
Component: Upgrade/Install Version:
Severity: minor Keywords: needs-patch
Cc: Focuses: ui, accessibility, administration

Description

It's always frustrating when the WordPress version URL opens up in the same tab; we can open the $version_url in a new tab by changing the target. It's a small change but will be helpful for some people.

Change History (11)

This ticket was mentioned in PR #2376 on WordPress/wordpress-develop by gazjones00.


5 years ago
#1

  • Keywords has-patch added

It's always frustrating when the WordPress version URL opens up in the same tab; we can open the $version_url in a new tab by changing the target. It's a small change but will be helpful for some people.

Please see - https://core.trac.wordpress.org/ticket/55308#ticket

#2 @gazthink3
5 years ago

  • Severity normalminor

joedolson commented on PR #2376:


5 years ago
#3

We've been intentionally removing instances of target=_blank from core as much as possible. While this might help some people, it will also harm some people. Adding target=blank takes choice away from the user, so for accessibility reasons we strongly recommend that it should be avoided unless in a case where opening in the same tab is likely to cause data loss.

I don't think this is one of those cases.

costdev commented on PR #2376:


5 years ago
#4

We've been intentionally removing instances of target=_blank from core as much as possible. While this might help some people, it will also harm some people. Adding target=blank takes choice away from the user, so for accessibility reasons we strongly recommend that it should be avoided unless in a case where opening in the same tab is likely to cause data loss.

I don't think this is one of those cases.

The nag can appear on all screens except the upgrade screen, but activating the link should prompt the user if there are forms with modifications, so I think you're right that this shouldn't cause any data loss.

gazjones00 commented on PR #2376:


5 years ago
#5

We've been intentionally removing instances of target=_blank from core as much as possible. While this might help some people, it will also harm some people. Adding target=blank takes choice away from the user, so for accessibility reasons we strongly recommend that it should be avoided unless in a case where opening in the same tab is likely to cause data loss.
I don't think this is one of those cases.

The nag can appear on all screens except the upgrade screen, but activating the link should prompt the user if there are forms with modifications, so I think you're right that this shouldn't cause any data loss.

Interesting, what if we were to implement something in a modal window similar to how "View details" works with plugins. I'm just trying to think of a method of implementation without disrupting the flow of things.

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


5 years ago

#7 @sabernhardt
5 years ago

  • Component GeneralUpgrade/Install
  • Focuses administration added
  • Type defect (bug)enhancement
  • Version trunk

Changing to enhancement, to consider the modal implementation (or something else other than target="_blank")

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


4 years ago

#9 @alexstine
4 years ago

  • Keywords needs-patch added; 2nd-opinion has-patch removed

At this point, it is unlikely that the links will still open in a new tab. Added needs-patch to see if the modal solution gets movement. Discussed in Accessibility Team meeting on Slack.

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


3 years ago

#11 @joedolson
3 years ago

  • Milestone Awaiting Review
  • Resolutionmaybelater
  • Status newclosed

We may consider adding this modal at some point in the future, so feel free to reopen if you wish to add a patch and move this forward. However, it's not a priority at the moment.

Note: See TracTickets for help on using tickets.