WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#54945 closed defect (bug) (fixed)

”Edit media”-links opening in new tabs with target="_blank"

Reported by: kebbet Owned by: audrasjb
Priority: normal Milestone: 6.0
Component: Media Version: 3.5
Severity: normal Keywords: has-patch commit dev-reviewed
Cc: Focuses: accessibility, administration

Description

The edit links should not open in new tabs. PR is on its way.

Could not find the changeset that introduced it, it is there in https://core.trac.wordpress.org/changeset/25001

Change History (8)

This ticket was mentioned in ​PR #2231 on ​WordPress/wordpress-develop by ​kebbet.
5 years ago #1

  • Keywords has-patch added; needs-patch removed

Trac ticket: https://core.trac.wordpress.org/ticket/54945

#2 @audrasjb
5 years ago

  • Milestone Awaiting Review → 6.0
  • Owner set to audrasjb
  • Status new → reviewing

Thanks for spotting this!
Moving for 6.0 consideration 👍

#3 @SergeyBiryukov
5 years ago

  • Version 3.7 → 3.5

Introduced in [22755] / #22083, setting the version accordingly.

Related: #23432.

#4 @audrasjb
5 years ago

  • Keywords commit added
  • Status reviewing → accepted

The patch looks good to me. Self-assigning for commit consideration.

#5 @audrasjb
5 years ago

  • Keywords dev-reviewed added

I was a bit afraid this change could break uploads of multiple files, but I tested the PR and it appears that even if the Edit link opens in the same tab, the rest of the media are successfully uploaded. Asynchronous uploads FTW 😎

We're good to go.

#6 @audrasjb
5 years ago

  • Resolution → fixed
  • Status accepted → closed

In 52646:

Media: Remove target blank attribute from media uploader edit links.

This change removes a dispensable target blank attribute from Edit links displayed in the asynchronous file uploader.

Follow-up to [22755].

Props kebbet, SergeyBiryukov.
Fixes #54945.
See #23432.

#8 @desrosj
4 years ago

#40125 was marked as a duplicate.

Note: See TracTickets for help on using tickets.