Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#14332 closed defect (fixed)

editing a link ends up creating new link (Editor_LinkDialog.html failure on IE9)

Reported by: bill Owned by: Jared Jurkiewicz
Priority: high Milestone: 1.6.2
Component: Editor Version: 1.7.0
Keywords: dohfail Cc:
Blocked By: Blocking:

Description

On IE9, under the following modes, load test_LinkDialog.html:

  • Browser mode: IE9 compatibility mode, Document Mode: IE9 (page default)
  • Browser mode: IE8, Document Mode: IE9 (page default)

Select and edit the example link, modifying the values in the link TooltipDialog. After saving, inspect the DOM. There are two <a>'s in the document, the old one, without any text, and the new one.

Change History (8)

comment:1 Changed 8 years ago by Jared Jurkiewicz

Milestone: 1.8

comment:2 Changed 8 years ago by Jared Jurkiewicz

Owner: set to Jared Jurkiewicz
Resolution: fixed
Status: newclosed

In [27234]:

Addin 'empty link' removal logic to the link dialog plugin. fixes #14332

comment:3 Changed 8 years ago by bill

Thanks, test is passing now on IE9.

comment:4 Changed 8 years ago by bill

In [27240]:

Reduce code. Could reduce a little further by using NodeList.orphan() but that requires including NodeList-dom, so it's a toss-up. Refs #14332 !strict.

comment:5 Changed 7 years ago by bill

In [30260]:

Backport [27234] and [27240] to 1.7 branch, fixing LinkDialog on IE9, fixes #14332 on 1.7 branch !strict.

comment:6 Changed 7 years ago by bill

Milestone: 1.81.7.5

comment:7 Changed 7 years ago by Ed Chatelain

In [30446]:

Backport [27234] and [27240] to 1.6 branch, fixing LinkDialog? on IE9, fixes #14332 on 1.6 branch !strict.

comment:8 Changed 7 years ago by bill

Milestone: 1.7.51.6.2
Note: See TracTickets for help on using tickets.