Opened 14 years ago
Closed 14 years ago
#8007 closed Bug (fixed)
onBeforeUnload event is fired in IE by switchin modes
Reported by: | Alfonso Martínez de Lizarrondo | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.1 |
Component: | General | Version: | 3.6.1 |
Keywords: | Cc: |
Description
Modify the replacebyclass example adding this code:
<script type="text/javascript"> window.onbeforeunload = function() { if (typeof(CKEDITOR)!='undefined') { var oEditor = CKEDITOR.instances.editor1 ; if (oEditor.checkDirty()) return "The text has been modified, do you want to save it before closing?."; } return undefined; } </script>
Now load the page and switch to source and back to design.
Expected: nothing special.
Results: the onBeforeUnload is fired on each change, and the checkDirty() returns true (#6274)
Tested with IE8. Firefox works fine, only shows the alert if the content has been modified and you want to close the page.
Attachments (1)
Change History (7)
comment:1 Changed 14 years ago by
Changed 14 years ago by
Attachment: | 8007.patch added |
---|
comment:3 follow-up: 5 Changed 14 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
This ticket just reveal some evilness of [6971], so re-propose a fix for #188 as I tried to do on that ticket.
comment:4 Changed 14 years ago by
I don't think unselectable should be applied recursively, the purpose of the selectable call is to prevent text selection from starting on the specified element, and a single attribute turns out to be enough unless you cite me an evidence that I'm wrong.
comment:5 Changed 14 years ago by
Status: | review → review_failed |
---|
Replying to garry.yao:
This ticket just reveal some evilness of [6971], so re-propose a fix for #188 as I tried to do on that ticket.
The way you put seem sounds like this ticket was logically predictable, which was not (there is no logic for it). In any case the proposed patch already have R- on #188, so there is no reason to have it differently now.
Replying to garry.yao:
I don't think unselectable should be applied recursively, the purpose of the selectable call is to prevent text selection from starting on the specified element, and a single attribute turns out to be enough unless you cite me an evidence that I'm wrong.
That thing is there since I can remember it. There is no correlation between that change and the fix. If we have that code, I'm sure there is a good reason for it. We'll not spend time on looking for the reasons to justify changing stable code.
To conclude, the right way to close this ticket is simply reverting [6971] and reopening #188. I'll do that now.
comment:6 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | review_failed → closed |
Correction: this happens with almost any toolbar button.
The problems started with [6971]
This is a critical bug for anyone that has set a onBeforeUnload handler and is using IE, I suggest to nominate it for 3.6.1