[CKEditor Dev] #7839: Pasting multi level Numbered list from word not working properly
CKEditor
---------------------+-------------------
Reporter: satya | Type: Bug
Status: new | Priority: Normal
Component: General | Version:
Keywords: IBM |
---------------------+-------------------
'''To reproduce the defect:'''
Open CK Editor,copy the list from attached doc and paste in to editor.
'''Expected Result:''' Multilevel Numbered list in the document should
paste properly.
'''Actual Result:''' Nesting between the outer and inner lists is removed
and also the ordering of list items is not maintained. when we try to
insert new list items it is not updating the order of other list items at
the same level.
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839>
CKEditor <http://ckeditor.com/>
The text editor for the Internet
CKEditor
Reporter: satya | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone:
Component: General | Version: 3.5.3
Resolution: | Keywords: IBM
----------------------+-----------------------
Changes (by j.swiderski):
* status: new => confirmed
* version: => 3.5.3
Comment:
This is reproducible for all versions of IE.
1. Open sample file and copy the whole list
2. You don't have to care about word formatting options in config.js
(leave them to true)
a. If you paste the list using CRTL+V the result is as described
above.[[BR]]
b. If you use "paste from word" dialog and **allow for access** (when IE
message pops out) this will work in IE6 and IE7 but for IE8 and IE9 list
will be broken.[[BR]]
c. If you use "paste from word" dialog and **disallow for access** (when
IE message pops out) this will work for all IE versions. List will be
pasted correctly [[BR]]
In my opinion lists from word should be pasted with "paste from word"
button.
The only thing that you might not like is the point b. But this is
probably due to how older and newer versions of IE see it.
If any one else thinks this is not a bug please close it.
This has occurred in version 3.5.3 rev [6616] which is a progress in
pasting lists comparing to [6615].
The only major problem with IE6&7 and [6616] is that those browsers can
break down when modifying list pasted from word. I have created new ticket
for this. Please refer to #7843
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:1>
CKEditor
Reporter: satya | Owner:
Type: Bug | Status: confirmed
Priority: Normal | Milestone:
Component: General | Version: 3.5.3
Resolution: | Keywords: IBM
----------------------+-----------------------
Comment (by satya):
Replying to [comment:1 j.swiderski]:
> This is reproducible for all versions of IE.
>
> 1. Open sample file and copy the whole list
> 2. You don't have to care about word formatting options in config.js
(leave them to true)
>
> a. If you paste the list using CRTL+V the result is as described
above.[[BR]]
> b. If you use "paste from word" dialog and **allow for access** (when
IE message pops out) this will work in IE6 and IE7 but for IE8 and IE9
list will be broken.[[BR]]
> c. If you use "paste from word" dialog and **disallow for access**
(when IE message pops out) this will work for all IE versions. List will
be pasted correctly [[BR]]
>
> In my opinion lists from word should be pasted with "paste from word"
button.
> The only thing that you might not like is the point b. But this is
probably due to how older and newer versions of IE see it.
> If any one else thinks this is not a bug please close it.
>
> This has occurred in version 3.5.3 rev [6616] which is a progress in
pasting lists comparing to [6615].
>
> The only major problem with IE6&7 and [6616] is that those browsers can
break down when modifying list pasted from word. I have created new ticket
for this. Please refer to #7843
@ j.swiderski even when i use the option b lists in level 1 & level 2
getting combined as single list. i think we should not close this defect,
since this was a regression and it was working in earlier versions.
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:2>
CKEditor
Reporter: satya | Owner:
Type: Bug | Status: new
Component: General | Version: 3.5.3
Resolution: | Keywords: IBM
----------------------+-------------------
* status: confirmed => new
Comment:
Sorry
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:3>
CKEditor
Reporter: satya | Owner:
Type: Bug | Status: confirmed
Component: General | Version: 3.5.3
Resolution: | Keywords: IBM
Changes (by j.swiderski):
* status: new => confirmed
* version: => 3.5.3
Comment:
I have deleted my first comment as I brought to much confusion. I have got
a little mixed up with this issue. Here is the new comment:
This is reproducible for all versions of IE.
1. Open sample file and copy the whole list
2. You don't have to care about word formatting options in config.js
(leave them to true)
Before revision [6585], CRTL+V, paste from word “allow” and “disallow”
produced the same correct result. Since that revision lists pasted from
word were flatten (from mulit-level to one level)
In [6616] there was a slight improvement:
1. If you paste the list using CRTL+V the result is as described by
@satya.
2. If you use "paste from word" dialog and **allow for access** (when
IE message pops out) this will work in IE6 and IE7 but for IE8 and IE9
list will be broken.
3. If you use "paste from word" dialog and **disallow for access**
(when IE message pops out) this will work for all IE versions. List will
be pasted correctly.
The only major problem with IE6&7 and [6616] is that those browsers can
break down when modifying list pasted from word. I have created new ticket
for this. Please refer to #7843
To sum up – this worked correctly in IE till [6585] (exclusive).
Having said all that, I also think that bug #7843 is the result of [6585].
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:3>
CKEditor
Reporter: satya | Owner: garry.yao
Type: Bug | Status: review
Priority: Normal | Milestone:
Component: Core : Pasting | Version: 3.5.3
Resolution: | Keywords: IBM
-----------------------------+-----------------------
Changes (by garry.yao):
* owner: => garry.yao
* status: confirmed => review
* component: General => Core : Pasting
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:4>
CKEditor
Reporter: satya | Owner: garry.yao
Type: Bug | Status: review_failed
Component: Core : Pasting | Version: 3.5.3
Resolution: | Keywords: IBM
Changes (by fredck):
* status: review => review_failed
Comment:
After patch, I can confirm that the list is properly pasted with Firefox
and IE9 in IE9 Mode.
With IE9 in Compatibility Mode instead, I'm having the "It was not
possible to clean up the pasted data duo to an internal error" error.
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:5>
CKEditor
Type: Bug | Status: review
Component: Core : Pasting | Version: 3.5.3
Resolution: | Keywords: IBM
Changes (by garry.yao):
* status: review_failed => review
Comment:
Replying to [comment:5 fredck]:
> With IE9 in Compatibility Mode instead, I'm having the "It was not
possible to clean up the pasted data duo to an internal error" error.
Looked for a Array::indexOf again in IE7 ;(
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:6>
CKEditor
Reporter: satya | Owner: garry.yao
Type: Bug | Status: review_passed
Component: Core : Pasting | Version: 3.5.3
Resolution: | Keywords: IBM
Changes (by fredck):
* status: review => review_passed
Comment:
The new patch is ok, but the debug changes from line 1321 are also
included on it. This is a serious problem, so please be sure to revert it
when committing.
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:7>
CKEditor
Type: Bug | Status: review_passed
Resolution: | Keywords: IBM
Changes (by fredck):
* milestone: => CKEditor 3.6.1
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:8>
CKEditor
Reporter: satya | Owner: garry.yao
Component: Core : Pasting | Version: 3.5.3
-----------------------------+----------------------------
Changes (by garry.yao):
* status: review_passed => closed
* resolution: => fixed
Comment:
Fixed with [6977].
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:9>
CKEditor
Reporter: satya | Owner: garry.yao
Type: Bug | Status: closed
Priority: Normal | Milestone: CKEditor 3.6.1
Resolution: fixed | Keywords: IBM
---------------------------------------+----------------------------
Changes (by fredck):
* component: Core : Pasting => Plugin : Paste from Word
--
Ticket URL: <http://dev.ckeditor.com/ticket/7839#comment:10>