Opened 8 years ago

Closed 8 years ago

Last modified 8 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 8 years ago.

Download all attachments as: .zip

Change History (11)

Changed 8 years ago by Artur Formella

Attachment: 3935-ff3.jpg added

comment:1 Changed 8 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 8 years ago by Garry Yao

Cc: WebSpellChecker.net added

comment:3 Changed 8 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.1CKEditor 3.2

comment:4 Changed 8 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.2CKEditor 3.x

comment:5 Changed 8 years ago by WebSpellChecker.net

Resolution: fixed
Status: newclosed

patch is submitted in #5145

comment:6 Changed 8 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 8 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 8 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 8 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: reopenedclosed

I confirm this is not anymore an issue.

comment:10 Changed 8 years ago by Frederico Caldeira Knabben

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