Ticket #3481 (closed Bug: fixed)
paste error IN Chrome
| Reported by: | toby941 | Owned by: | martinkou |
|---|---|---|---|
| Priority: | Normal | Milestone: | FCKeditor 2.6.5 |
| Component: | General | Version: | FCKeditor 2.6.4 |
| Keywords: | Confirmed WebKit Review+ | Cc: | toby941@… |
Description (last modified by martinkou) (diff)
- use chrome(version:1.0.154.36) 2.create a table
- paste something into table
- the format of the table is error
Attachments
-
fck.jpg
(25.4 KB) -
added by toby941 3 years ago.
- the error format
-
3481.patch
(2.4 KB) -
added by martinkou 3 years ago.
-
3481_2.patch
(2.5 KB) -
added by martinkou 3 years ago.
Change History
comment:2 Changed 3 years ago by arczi
- Keywords WorksForMe Pending added
I was unable to reproduce this bug on http://www.fckeditor.net/demo.
Could you tell me what did you paste?
comment:3 Changed 3 years ago by toby941
I use Chrome(2.0.168.0),on[ http://www.fckeditor.net/demo.] # click the insert table button,insert a table(default attribute ) # copy two letters,and then paste some times,the table format was break
comment:5 Changed 3 years ago by martinkou
- Status changed from new to closed
- Resolution set to worksforme
I'm not able to reproduce the bug as well. Closing as worksforme.
comment:6 Changed 3 years ago by martinkou
- Keywords Confirmed Chrome added; WorksForMe Pending removed
- Status changed from closed to reopened
- Resolution worksforme deleted
Oh, I've just got the bug reproduced.
comment:7 Changed 3 years ago by martinkou
To reliably reproduce this bug:
- Open http://www.fckeditor.net/demo in Chome 2.
- Create a default table.
- Copy two letters from anywhere.
- Put caret to a table cell.
- Press Ctrl-V twice or more times.
The first Ctrl-V works, the second, third, ... Ctrl-V will paste the two letters outside of table cells.
comment:8 Changed 3 years ago by martinkou
- Keywords WebKit added; Chrome removed
The same bug can also be reproduced in Safari 4 in Mac.
comment:10 Changed 3 years ago by martinkou
- Owner set to martinkou
- Status changed from reopened to new
comment:12 Changed 3 years ago by garry.yao
- Keywords Review- added; Review? removed
When you looked into the DOM after pasted:
<td><span class="Apple-style-span"
style="-webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; ">sa</span>{ZWNBSP}
</td>
You'll notice that the second ZWNBSP is still there, the removing of this node has failed, don't know why.
comment:13 Changed 3 years ago by martinkou
- Keywords Review? added; Review- removed
Something in the browser's paste logic seems to be merging the text nodes, and even worse, even if I use inline tags like <b> and <span> it would get merged too. I've found that Safari doesn't merge <a> tags but it would still clone it (i.e. the original DOM node reference we've got becomes invalid). So I've assigned an id to the <a> dummy to keep track of it.
comment:15 Changed 3 years ago by martinkou
- Status changed from assigned to closed
- Resolution set to fixed
Fixed with [3973].
Click here for more info about our SVN system.
comment:16 Changed 2 years ago by alfonsoml
This patch created #4960
