Opened 7 years ago

Closed 4 years ago

#6116 closed Bug (wontfix)

The contentEditable attribute is lost from textareas and inputs

Reported by: Sa'ar Zac Elias Owned by: Sa'ar Zac Elias
Priority: Normal Milestone:
Component: General Version: 3.4 Beta
Keywords: Cc:

Description

  • Load the following:
    <textarea contentEditable="false">Text</textarea>
    
  • Go to source mode.

Notice that the attribute is missing.

Attachments (1)

6116.patch (1.3 KB) - added by Sa'ar Zac Elias 7 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 7 years ago by Sa'ar Zac Elias

Milestone: CKEditor 3.4
Owner: set to Sa'ar Zac Elias
Status: newassigned

Changed 7 years ago by Sa'ar Zac Elias

Attachment: 6116.patch added

comment:2 Changed 7 years ago by Sa'ar Zac Elias

Status: assignedreview

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.4
Status: reviewreview_failed

I don't think it's worth having code changes because of this one. In fact, using contenteditable in form fields is a very uncommon practice. So, let's wait for some one coming with this need from the real world.

Additionally, this issue also impacts <input> elements.

Even when considering the fix needed, I would make it inside the protectReadOnly and unprotectReadyOnly functions directly, instead of bringing overhead to processor.

comment:4 Changed 4 years ago by Piotrek Koszuliński

Resolution: wontfix
Status: review_failedclosed

It would be a waste of time and unnecessary code bloating.

Note: See TracTickets for help on using tickets.
© 2003 – 2017 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy