Opened 16 years ago
Closed 16 years ago
#4149 closed Bug (fixed)
Fix for core/editor test case
Reported by: | Artur Formella | Owned by: | Artur Formella |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.0 |
Component: | QA | Version: | |
Keywords: | Review+ | Cc: |
Description
test_config_inpage from core/editor.html fails because of SCAYT plugin:
beforeInit : function( editor ) { // Register own rbc menu group. editor.config.menu_groups = 'scayt_suggest,scayt_moresuggest,scayt_control,' + editor.config.menu_groups; },
Sometimes it works because of the random loading order. My idea is to disable this plugin in config.
Test case:
http://nightly.ckeditor.com/latest/ckeditor/_tests/core/editor.html
Attachments (2)
Change History (9)
comment:1 Changed 16 years ago by
Owner: | set to Artur Formella |
---|---|
Status: | new → assigned |
Changed 16 years ago by
Attachment: | 4149.patch added |
---|
comment:2 Changed 16 years ago by
Keywords: | Review? added |
---|
comment:3 Changed 16 years ago by
comment:4 Changed 16 years ago by
Keywords: | Review- added; Review? removed |
---|
I'm not able to reproduce it with FF3. It's not passing in IE7 though, but the patch is not fixing it, and it's actually making another tc to fail.
Other than that, as a side note, for such kinds of changes it's always good to add a comment explaining the reason for such thing in the code, possibly including the ticket number. In this way we understand why the scayt plugin is being removed.
Changed 16 years ago by
Attachment: | 4149_2.patch added |
---|
comment:5 Changed 16 years ago by
Keywords: | Review? added; Review- removed |
---|
We should temporarily disable SCAYT for the whole test suite.
comment:6 Changed 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
Easy to reproduce in FF3 with enabled cache.