Opened 14 years ago
Closed 14 years ago
#6278 closed Bug (fixed)
comment tags get moved to top
Reported by: | MBcode | Owned by: | Paweł Horzela |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.1 |
Component: | Core : Output Data | Version: | 3.4 |
Keywords: | Cc: |
Description (last modified by )
Hey there, In Source editing mode paste original default Text:
%header%<br /> <br /> <!--start:problem[0]--><br /> <!--stop:problem[0]--><br /> <!--start:result[0]--><br /> <!--stop:result[0]--><br /> <!--start:info[0]--><br /> <!--stop:info[0]--><br /> <!--start:end[0]--><br /> <!--stop:end[0]--><br /> <br /> <!--start:sorry[0]--><br /> <!--stop:sorry[0]--><br /> <br /> %footer%
Switch to WYSIWYG and back to source editing. it gets formatted to:
%header%<br /> <br /> <!--start:problem[0]--><!--stop:problem[0]--><!--start:result[0]--><!--stop:result[0]--><!--start:info[0]--><!--stop:info[0]--><!--start:end[0]--><!--stop:end[0]--><!--start:sorry[0]--><!--stop:sorry[0]--><br /> <br /> <br /> <br /> <br /> <br /> <br /> <br /> <br /> <br /> <br /> <br /> <br /> %footer%
This isnt rly usefull because I like to use Comments to Insert with Javascript at the given position.
Please fix that! Would be really great.
Attachments (1)
Change History (9)
comment:1 Changed 14 years ago by
Component: | General → UI : Source View |
---|---|
Description: | modified (diff) |
Keywords: | comment tags removed |
Status: | new → confirmed |
comment:2 Changed 14 years ago by
Owner: | set to Paweł Horzela |
---|---|
Status: | confirmed → assigned |
comment:3 Changed 14 years ago by
Status: | assigned → review |
---|
comment:4 Changed 14 years ago by
It looks like regression, introduced in 3.1 (revision 5103): http://dev.ckeditor.com/changeset/5103
by fixing the ticket #4886: http://dev.ckeditor.com/ticket/4886 http://rev.ckeditor.com/index.php?pageID=10
comment:5 Changed 14 years ago by
Is there any ETA for the fix? Seems that the patch not work. I also noticed latest comment, so there wont be a fix? :(
patch -p0 < 6278.patch (Stripping trailing CRs from patch.) patching file _source/core/htmlparser/fragment.js Hunk #1 FAILED at 386. 1 out of 1 hunk FAILED -- saving rejects to file _source/core/htmlparser/fragment.js.rej
comment:6 Changed 14 years ago by
Milestone: | → CKEditor 3.5.1 |
---|
comment:7 Changed 14 years ago by
Status: | review → review_passed |
---|
comment:8 Changed 14 years ago by
Component: | UI : Source View → Core : Output Data |
---|---|
Resolution: | → fixed |
Status: | review_passed → closed |
Fixed with [6284].
Confirmed on 3.4.1 Nightly & Demo. Source formating is lost in above case. I changed formating of issue.