WordPress.org

Make WordPress Core

#24520 closed defect (bug) (fixed)

TinyMCE - Edit Link - New Window bug

Reported by: sc0ttkclark Owned by: markjaquith
Milestone: 3.6 Priority: normal
Severity: minor Version: 3.5.1
Component: TinyMCE Keywords: has-patch
Focuses: Cc:

Description

The value of inputes.openInNewTab is set to true only if the target is _blank, but if it isn't, it isn't unchecked.

Steps to reproduce:

  1. Add two links, make the first one not open in new window, make the second open in new window
  1. Be sure to edit the link that opens in a new window, so the checkbox will be set to checked, close the edit prompt
  1. Now edit the link that doesn't open in a new window, the checkbox remains checked and hasn't been turned off

I'm attaching a proposed patch for the resolution.

Attachments (3)

24520.patch (18.7 KB) - added by sc0ttkclark 11 months ago.
Fixes the minor JS true/false check
24520.2.patch (18.6 KB) - added by sc0ttkclark 11 months ago.
Fixed patch to base at /trunk/
24520.3.patch (705 bytes) - added by sc0ttkclark 11 months ago.
Remove min file from patch

Download all attachments as: .zip

Change History (6)

sc0ttkclark11 months ago

Fixes the minor JS true/false check

comment:1 sc0ttkclark11 months ago

  • Keywords has-patch added

sc0ttkclark11 months ago

Fixed patch to base at /trunk/

sc0ttkclark11 months ago

Remove min file from patch

comment:2 markjaquith11 months ago

  • Milestone changed from Awaiting Review to 3.6
  • Owner set to markjaquith
  • Status changed from new to accepted

comment:3 markjaquith11 months ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 24419:

Properly uncheck the "new tab" box when editing a non-new-tab link.

Previously, if you had just edited a new-tab link, a subsequent edit of a non-new-tab link would mistakenly keep the checkbox checked.

props sc0ttkclark. fixes #24520.

Note: See TracTickets for help on using tickets.