Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#3935 closed Bug (fixed)

SCAYT ads

Reported by: Artur Formella Owned by:
Priority: Normal Milestone: CKEditor 3.3
Component: UI : Spell Checker Version:
Keywords: Confirmed Cc: WebSpellChecker.net

Description

When

CKEDITOR.config.scayt_maxSuggestions = 0.

Scayt banner has wrong position/z-index. Please see the attachment.

Attachments (1)

3935-ff3.jpg (44.4 KB) - added by Artur Formella 10 years ago.

Download all attachments as: .zip

Change History (11)

Changed 10 years ago by Artur Formella

Attachment: 3935-ff3.jpg added

comment:1 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Confirmed added

My suggestion to this one is to force having a minimum number of suggestions, which would not impact on the ads banner.

comment:2 Changed 10 years ago by Garry Yao

Cc: WebSpellChecker.net added

comment:3 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.1CKEditor 3.2

comment:4 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.2CKEditor 3.x

comment:5 Changed 10 years ago by WebSpellChecker.net

Resolution: fixed
Status: newclosed

patch is submitted in #5145

comment:6 Changed 10 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: closedreopened

It's ok to indicate that this one is supposed to be fixed by another tickets, but it should be closed only after confirming the fix one the relative ticket gets closed.

comment:7 Changed 10 years ago by Garry Yao

Forcing a minimum number of suggestions could somehow helps to workaround this problem while it doesn't work if user's using the following config:

config.scayt_maxSuggestions = 1;

It's necessary to re-design the AD banner location to avoid conflicting with the secondary context menu.

comment:8 Changed 10 years ago by WebSpellChecker.net

New version of SCAYT core allows to close banner manually. Those changes will available in CKEditor 3.2.1 and should help.

comment:9 Changed 10 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: reopenedclosed

I confirm this is not anymore an issue.

comment:10 Changed 10 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.xCKEditor 3.3
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy