Opened 14 years ago
Last modified 14 years ago
#6395 review_failed Bug
Unable to set formating on empty page
Reported by: | Krzysztof Studnik | Owned by: | Martin |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Core : Styles | Version: | 3.4.1 |
Keywords: | Opera | Cc: |
Description (last modified by )
Found when checking #6370 In Opera
- set new empty page
- set text formating as "Formated"
- enter some text - formating is back to normal
- set formating again - enetered text format changes itself (video)
- result in CKE3.4.1
{{{{ <pre> test<span style="display: none"> </span></pre> <pre> <span style="display: none"> </span> </pre> }}} The same test case in CKE 3.4
<pre> <span _fck_bookmark="1" style="display: none"> </span>test</pre> <pre> <span _fck_bookmark="1" style="display: none"> </span> </pre> }}}[[BR]] [[BR]] [[BR]]
Attachments (2)
Change History (13)
Changed 14 years ago by
Attachment: | krst 15.56.15 .avi added |
---|
comment:1 Changed 14 years ago by
Milestone: | → CKEditor 3.4.2 |
---|
comment:2 Changed 14 years ago by
Status: | new → confirmed |
---|
comment:4 Changed 14 years ago by
Owner: | set to Paweł Horzela |
---|---|
Status: | confirmed → assigned |
comment:5 Changed 14 years ago by
Owner: | changed from Paweł Horzela to Martin |
---|
comment:6 Changed 14 years ago by
Status: | assigned → review |
---|
comment:7 Changed 14 years ago by
Status: | review → review_failed |
---|
@mani Ah...hard to believe this is a fix...the 'newBlock' may have contents from existing blocks.
comment:8 Changed 14 years ago by
I think you'll need to briefly explain a bit the cause behind such a fix.
comment:9 Changed 14 years ago by
Description: | modified (diff) |
---|
Look at the code:
//document.body.contentEditable = false document.designMode = 'on'; var sel = window.getSelection(); sel.removeAllRanges();
It is not working but should but when we uncomment first line and comment second everythink works.
Additionaly I can say when designMode == 'on', sel.AddRange() don`t work too so we can not modify selection.
Maybe this will be some tip to others.
comment:11 Changed 14 years ago by
Milestone: | CKEditor 3.4.2 |
---|
I don't quite understand the bug you're pointing to, the test case you've reported to Opera could present here on ticket.
- We're not using "designMode" for Opera;
- If native range manipulation doesn't work, does it affects also other scenarios?
Anyway, as we're at the time of preparing the release of 3.4.2, deferring it.
pre-formatted style is not opened properly, while other blocks are good, the cursor is not blinking after applying them.