Make WordPress Core

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#39276 closed defect (bug) (fixed)

Link Editor bug - target="_blank" not removed

Reported by: mario62rus's profile Mario62rus Owned by: iseulde's profile iseulde
Milestone: 4.7.1 Priority: normal
Severity: normal Version: 4.7
Component: Editor Keywords: has-patch commit
Focuses: Cc:

Description

Here's my workflow:

  1. Setup clear WP 4.7, no themes or plugins.
  2. Edit post/page.
  3. Add some link.
  4. In the Link Manager uncheck "Open link in a new tab", press Update button and see tab "Text" - attribute target="_blank" added.
  5. Click "Edit link", uncheck "Open link in a new tab", press Update button and see tab "Text" - the attribute remains in place.

In version 4.6.1 this works correctly.

Attachments (1)

39276.patch (610 bytes) - added by iseuldebot 8 years ago.

Download all attachments as: .zip

Change History (8)

#1 @dd32
8 years ago

  • Component changed from General to Editor
  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 4.7.1
  • Summary changed from Link Manager bug - target="_blank" not removed to Link Editor bug - target="_blank" not removed

Hi @Mario62rus and thanks for reporting this!

I've reproduced this locally, I'm milestoning this for 4.7.1 pending a fix.

(Note: I renamed this from "Link Manager" as that's what we call another part of WordPress internally, just so we don't get tickets confused)

#2 @Mario62rus
8 years ago

OK, thx

@iseuldebot
8 years ago

#4 @iseulde
8 years ago

  • Keywords has-patch added; needs-patch removed

#5 @azaozz
8 years ago

  • Keywords commit added

39276.patch looks good.

#6 @iseulde
8 years ago

  • Owner set to iseulde
  • Resolution set to fixed
  • Status changed from new to closed

In 39601:

Editor: Remove target=_blank when unchecked in the link modal.

Fixes #39276 for trunk.

#7 @iseulde
8 years ago

In 39602:

Editor: Remove target=_blank when unchecked in the link modal.

Merges [39601] to the 4.7 branch.
Fixes #39276.

Note: See TracTickets for help on using tickets.