Opened 5 years ago

Last modified 5 years ago

#5854 confirmed Bug

SCAYT's class apears in the body tag during save operation

Reported by: SpellChecker.net Owned by:
Priority: Normal Milestone:
Component: UI : Spell Checker Version: 3.4.1
Keywords: HasPatch, Review? Cc:

Description

  1. Set option CKEDITOR.config.fullpage="true" for the sample page.
  1. Load the sample page.
  1. Click on the "Source" button to view source HTML code

--> Body doesn't contain any SCAYT classes.

  1. Click on the "Source" button again to return to normal mode
  1. Click on the "Save" icon to view the result

Actual result:

<body class="scayt-enabled">

Expected result:

<body>

Attachments (2)

5854.patch (2.0 KB) - added by SpellChecker.net 5 years ago.
5854.2.patch (493 bytes) - added by SpellChecker.net 5 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 5 years ago by SpellChecker.net

  • Component changed from General to UI : Spell Checker
  • Version set to 3.4.1 (SVN - trunk)

Changed 5 years ago by SpellChecker.net

comment:2 Changed 5 years ago by SpellChecker.net

  • Keywords HasPatch Review? added

The SpellChecker.net team proposes this patch in order to fix this bug. It also fixes situation when form is submitted with a "submit" button etc.

The patch was tested by the SpellChecker.net QA team.

comment:3 Changed 5 years ago by fredck

  • Keywords HasPatch Review? removed
  • Status changed from new to confirmed

This is the wrong way for this fix. I think the data processor should be used, adding a new rule for the body element.

Changed 5 years ago by SpellChecker.net

comment:4 Changed 5 years ago by SpellChecker.net

  • Keywords HasPatch Review? added

OK, here is the new patch, which adds a new rule for processing 'class' attribute of an element object.

comment:5 Changed 5 years ago by wwalc

#6858 marked as duplicate

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