Opened 15 years ago
Closed 15 years ago
#6869 closed Bug (fixed)
data-cke-nostyle disappears from source
| Reported by: | Wiktor Walc | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5.1 |
| Component: | Core : Output Data | Version: | 3.5 |
| Keywords: | Cc: |
Description
Using the code suggested here:
http://dev.ckeditor.com/ticket/6103#comment:30
switch to source mode and back.
Result:
My name is <span>%NAME%</span>
Note: data-cke-nostyle="1" is gone.
Attachments (2)
Change History (10)
Changed 15 years ago by
| Attachment: | 6869.patch added |
|---|
comment:1 Changed 15 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → review |
comment:2 Changed 15 years ago by
| Status: | review → review_failed |
|---|
comment:3 Changed 15 years ago by
| Keywords: | Discussion added |
|---|
We have some problems here:
- Considering that this attribute is defined in a plugin, it would be bad to have a special case for it on core. We could have similar issues with other plugins.
- It looks like "data-cke-nostyle" was a bad name for it, because we're leaving the "data-cke-" prefix to CKEditor internal stuff.
Based on the above, wouldn't it be better to have the "data-cke-nostyle" changed to something like "data-nostyle"?
I know this would break backwards compatibility, but considering that we have introduced it on our very last version, the impact may be low. It would be a way for us to fix the mistake we've done on the previous release.
comment:5 Changed 15 years ago by
| Keywords: | Discussion removed |
|---|
Ok, so let's do it earlier than later.
Changed 15 years ago by
| Attachment: | 6869_2.patch added |
|---|
comment:6 Changed 15 years ago by
| Status: | review_failed → review |
|---|
comment:7 Changed 15 years ago by
| Status: | review → review_passed |
|---|
Watch out for a corrupted changelog entry.
comment:8 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [6308].

An edge case, but easily fixable: attributes whose name starts with "data-cke-nostyle" (e.g. "data-cke-nostyles") are not removed.