Opened 13 years ago
Closed 13 years ago
#6866 closed Bug (fixed)
Image link target field not correctly set when editing an existing image.
| Reported by: | Joe Kavanagh | Owned by: | Sa'ar Zac Elias |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4.3 |
| Component: | General | Version: | 3.4.3 |
| Keywords: | IBM | Cc: | Satya Minnekanti, Damian, jamcunni@… |
Description (last modified by )
- Open the image dialog.
- Fill in the image information.
- On the Link tab enter a URL and set the Target field to 'New Window'.
- Click OK to insert the image.
- Right click on the inserted image and choose Image Properties from the context menu.
- Click on the Link tab on the Image dialog.
Observe that the Target field is incorrectly set to '<not set>'. The anchor tag's target attribute does contain the correct value, '_blank'.
Attachments (1)
Change History (13)
comment:1 Changed 13 years ago by
| Milestone: | → CKEditor 3.4.3 |
|---|---|
| Status: | new → confirmed |
comment:2 Changed 13 years ago by
Changed 13 years ago by
| Attachment: | 6866.patch added |
|---|
comment:3 Changed 13 years ago by
| Owner: | set to Sa'ar Zac Elias |
|---|---|
| Status: | confirmed → review |
comment:4 Changed 13 years ago by
| Description: | modified (diff) |
|---|---|
| Status: | review → review_failed |
#6673 is still reproducible but just to add back the "if ( type == LINK )", as the dialog fields are setup twice.
comment:5 Changed 13 years ago by
| Status: | review_failed → review |
|---|
Neither me or Tobiasz can reproduce #6673 after the patch.
comment:6 Changed 13 years ago by
I can't reproduce #6673 anymore, but the TC of this ticket fails because of the second invocation of the setup, when it's value is correct, but the new value is null.
comment:7 Changed 13 years ago by
| Status: | review → review_failed |
|---|
comment:8 Changed 13 years ago by
| Owner: | changed from Sa'ar Zac Elias to Tobiasz Cudnik |
|---|---|
| Status: | review_failed → review |
I had some code changes when testing this TC, it works fine with 6866.patch.
comment:9 Changed 13 years ago by
| Status: | review → review_passed |
|---|
comment:10 Changed 13 years ago by
| Cc: | Satya Minnekanti Damian jamcunni@… added; satya damo removed |
|---|
comment:11 Changed 13 years ago by
| Owner: | changed from Tobiasz Cudnik to Sa'ar Zac Elias |
|---|---|
| Status: | review_passed → assigned |
Note: See
TracTickets for help on using
tickets.
Regression of [6136].