Opened 13 years ago

Closed 13 years ago

#7084 closed Bug (fixed)

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

Reported by: WebSpellChecker.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 WebSpellChecker.net 13 years ago.
7084_2.patch (17.6 KB) - added by Wiktor Walc 13 years ago.

Download all attachments as: .zip

Change History (6)

Changed 13 years ago by WebSpellChecker.net

Attachment: 7084.patch added

comment:1 Changed 13 years ago by WebSpellChecker.net

Here is patch which fixes both issues.

comment:2 Changed 13 years ago by Wiktor Walc

Milestone: CKEditor 3.5.2
Status: newconfirmed
Version: 3.53.1

Changed 13 years ago by Wiktor Walc

Attachment: 7084_2.patch added

comment:3 Changed 13 years ago by Wiktor Walc

(updated patch against current trunk)

comment:4 Changed 13 years ago by Sa'ar Zac Elias

Resolution: fixed
Status: confirmedclosed

Fixed with [6518].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy