Opened 6 years ago

Closed 6 years ago

#7084 closed Bug (fixed)

SCAYT options menu is empty / incorrect work of "scayt_uiTabs" config param for multiple CKEditor instances

Reported by: SpellChecker.net Owned by:
Priority: Normal Milestone: CKEditor 3.5.3
Component: UI : Spell Checker Version: 3.1
Keywords: HasPatch, Review? Cc: tech@…

Description

Bug 1. "SCAYT options menu is empty for multiple CKEditor instances"

Steps to reproduce:

  1. Open "replace by code" sample.
  2. Start SCAYT for both of editors.
  3. Open SCAYT options menu for bothe of editors.

Expected result:
Options, Languages, Dictionaries, About Us tabs have content for both editors.

Actual result:
Options, Languages, Dictionaries, About Us tabs have content for only one of the editors.


Bug 2. "Incorrect work of "scayt_uiTabs" config param for multiple CKEditor instances"

Steps to reproduce:

  1. Open code of "replace by code" sample.
  2. For first editor set
    CKEDITOR.replace( 'editor1', { scayt_uiTabs: '1,0,1' });  
    


For second editor set

CKEDITOR.replace( 'editor2', { scayt_uiTabs: '0,0,0' });  


  1. Open page and start SCAYT for both editors

Expected result:
scayt_uiTabs param from second editor doesn't affect menu of first editor

Actual result:
All options in first editor are not active. When "About us" tab is opened and than "Cancel" is clicked, an error is shown: "document.forms.optionsbar is undefined [Break On This Error] return document.forms.optionsbaroptions?; options.js (line 27)"

Attachments (2)

7084.patch (17.7 KB) - added by SpellChecker.net 6 years ago.
7084_2.patch (17.6 KB) - added by wwalc 6 years ago.

Download all attachments as: .zip

Change History (6)

Changed 6 years ago by SpellChecker.net

comment:1 Changed 6 years ago by SpellChecker.net

Here is patch which fixes both issues.

comment:2 Changed 6 years ago by wwalc

  • Milestone set to CKEditor 3.5.2
  • Status changed from new to confirmed
  • Version changed from 3.5 to 3.1

Changed 6 years ago by wwalc

comment:3 Changed 6 years ago by wwalc

(updated patch against current trunk)

comment:4 Changed 6 years ago by Saare

  • Resolution set to fixed
  • Status changed from confirmed to closed

Fixed with [6518].

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy