Opened 15 years ago
Closed 15 years ago
#4988 closed Bug (fixed)
Safari breaks when pasting with forcePasteAsPlainText = true
| Reported by: | Frederico Caldeira Knabben | Owned by: | Alfonso Martínez de Lizarrondo |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.2 |
| Component: | Core : Pasting | Version: | 3.1 |
| Keywords: | Safari Confirmed Review+ | Cc: | diego@… |
Description
When pasting with Safari and forcePasteAsPlainText = true in the settings, the browser breaks and shutdown completely.
Confirmed with Safari 3.2.2 Win, and also reported here.
Attachments (1)
Change History (14)
comment:1 Changed 15 years ago by
comment:2 Changed 15 years ago by
| Cc: | diego@… added |
|---|
Confirmed also in Safari 4.0.4 Mac but without the nasty crash, simply it doesn't work (if you insist and try to paste more the one time, one or more textareas for each paste appears inside the editor).
comment:3 Changed 15 years ago by
And, yes, it works in Chrome 4 Mac (it seems a Safari-only problem, apart from webkit).
comment:5 Changed 15 years ago by
| Priority: | High → Normal |
|---|
WFM also with Safari3/Win, seems for me a Mac-Safari-only problem.
comment:6 Changed 15 years ago by
Yes, I confirm that the bug doesn't affect Safari 4.0.4 WIN, but does affect the same Safari version on Mac.
comment:7 Changed 15 years ago by
Confirmed in Safari 4.0.4/Mac OS 10.6.2.
Single paste results in a blank space being inserted. Further pastes result in textareas appearing within the editor.
comment:8 Changed 15 years ago by
This one may be related to #5095, which is an internal browser bug.
comment:9 Changed 15 years ago by
| Milestone: | CKEditor 3.2 → CKEditor 3.3 |
|---|
comment:10 Changed 15 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Alfonso Martínez de Lizarrondo |
| Status: | new → assigned |
In Webkit it's possible to get the content of the clipboard as plain text without any hack, so the patch uses that approach.
I've included also a change in the registration of the event that fixes #5095 as that might make easier to review all the code.
And after applying the patch it seems that #5144 is also fixed, although that bug is a little elusive and sometimes seems to work (or maybe it's problems with the cache)
comment:11 Changed 15 years ago by
| Keywords: | Review+ added; Review? removed |
|---|---|
| Milestone: | CKEditor 3.3 → CKEditor 3.2 |
Great job! Feel free to close #5144 also (please add changelog entries for both tickets).
WFM with Chrome 4.