Opened 17 years ago
Closed 13 years ago
#3032 closed Bug (fixed)
Event 'instanceReady' doesn't work in certain case
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | General | Version: | 3.0 Beta 2 |
| Keywords: | Cc: |
Description
Specifically, the instanceReady was triggered twice when edito r created in 'append' mode and the editor isn't fully interacted in because of incorrect state of 'dirty' flag.
Attachments (3)
Change History (13)
Changed 17 years ago by
| Attachment: | test-editor-instanceReady.patch added |
|---|
comment:1 Changed 17 years ago by
| Priority: | Normal → High |
|---|
comment:2 Changed 17 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → assigned |
Changed 17 years ago by
| Attachment: | 3032.patch added |
|---|
comment:3 Changed 17 years ago by
| Keywords: | Review? added |
|---|
comment:4 Changed 17 years ago by
| Keywords: | Review- WorksForMe added; Review? removed |
|---|
I can't understand this issue fully. The ticket points to "instanceReady" being fired twice, and the test case is checking for the dirty state.
I've changed the replacebycode sample, making the following for the first instance:
CKEDITOR.replace( 'editor1',
{
on :
{
instanceReady : function( ev )
{
document.title += ' (' + ev.editor.name + ')';
}
}
} );
Everything works properly, and the event is always fired, once only.
Maybe this has been fixed by recent changes. Can you confirm it, giving a real example other than the test case?
comment:5 Changed 17 years ago by
The description is misleading and out-of-date, the problem handled by this ticket is only about:
Event listener for "instanceReady" doesn't grab a correct 'dirtyFlag' value, as the TC described, Since this is a internal error, it's hard to reproduce at UI level.
comment:6 Changed 17 years ago by
| Keywords: | WorksForMe removed |
|---|---|
| Priority: | High → Normal |
comment:7 Changed 16 years ago by
| Milestone: | CKEditor 3.0 → CKEditor 3.x |
|---|
comment:8 Changed 16 years ago by
| Version: | SVN (FCKeditor) → CKEditor 3.0 Beta 2 |
|---|
comment:10 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_failed → closed |
Expired. TC we currently have points to #9872.

Functional test case.