Opened 14 years ago
Closed 14 years ago
#6646 closed Bug (fixed)
text inputs and areas lack context menu if they are present in the initial content
| Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Sa'ar Zac Elias |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5.1 |
| Component: | General | Version: | 3.4 |
| Keywords: | Cc: |
Description
Follow up from #6046
That ticket fixed the context menu for those elements when they are inserted, but if they are present in the original content, then the isReadOnly() check returns the element and so they aren't editable.
Reproduction steps:
- Insert a text input, check that the context menu for editing it appears.
- Switch to source and back to design.
- The context menu entry is missing.
(be careful if checking with IE due to #5522)
Attachments (2)
Change History (9)
comment:1 Changed 14 years ago by
| Status: | new → confirmed |
|---|
Changed 14 years ago by
| Attachment: | 6646.patch added |
|---|
comment:2 Changed 14 years ago by
| Milestone: | → CKEditor 3.5.1 |
|---|---|
| Owner: | set to Garry Yao |
| Status: | confirmed → review |
comment:3 follow-up: 6 Changed 14 years ago by
| Status: | review → review_failed |
|---|
The Value field is empty when trying to edit the control.
Changed 14 years ago by
| Attachment: | 6646_2.patch added |
|---|
comment:4 Changed 14 years ago by
| Owner: | changed from Garry Yao to Sa'ar Zac Elias |
|---|---|
| Status: | review_failed → review |
comment:5 Changed 14 years ago by
| Status: | review → review_passed |
|---|
comment:6 Changed 14 years ago by
Replying to tobiasz.cudnik:
The Value field is empty when trying to edit the control.
Missing value attribute is an known IE7 bug and we've already had a ticket for it.
comment:7 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [6289].
Note: See
TracTickets for help on using
tickets.
One thing need to be reviewed is whether it works with [some hacks http://dev.fckeditor.net/browser/CKEditor/trunk/_source/plugins/forms/dialogs/textfield.js#L177 ] we introduced in IE for inputs.