Ticket #5854 (confirmed Bug)

Opened 3 years ago

Last modified 2 years ago

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

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

Change History

comment:1 Changed 3 years ago by SpellChecker.net

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

Changed 3 years ago by SpellChecker.net

comment:2 Changed 3 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 3 years ago by fredck

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

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 3 years ago by SpellChecker.net

comment:4 Changed 3 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 2 years ago by wwalc

#6858 marked as duplicate

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