Opened 14 years ago
Closed 14 years ago
#6859 closed Bug (fixed)
Full page mode: Spell check dialog adds extra empty style tags
| Reported by: | Wiktor Walc | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5 |
| Component: | UI : Spell Checker | Version: | 3.5 |
| Keywords: | Cc: | WebSpellChecker.net |
Description (last modified by )
When SCAYT is enabled and the spell check dialog is used to check the document, empty extra style tags are added to the document and a JavaScript error occurs (reported in #6863).
- Open _samples/fullpage.html
- Start with:
<html> <head> </head> <body> <p> Write your email message here schol.</p> <p> You can also...</p> </body> </html>
- Enable SCAYT
- Open the Check Spell dialog
- Click on Finish Checking
- Open the Check Spell dialog again
- Click on Finish Checking
- Result: empty style tags are now present in the document:
<html> <head> <title></title> <style cke_temp="2" type="text/css"> </style> <style cke_temp="2" type="text/css"> </style> <style cke_temp="2" type="text/css"> </style> <style cke_temp="2" type="text/css"> </style> </head> <body class="scayt-enabled"> <p> Write your email message here schol.</p> <p> You can also...</p> </body> </html>
JavaScript error:
Tried to register widget with id==scayt_2 but that id is already registered
http://svc.spellchecker.net/scayt26/_base.xd.js
Line 880
Confirmed in Safari 5.0.3 and Firefox 3.6.13. Looks like a regression introduced in 3.5
Change History (6)
comment:1 Changed 14 years ago by
| Status: | new → confirmed |
|---|
comment:3 Changed 14 years ago by
Issue with styles is our server issue, while JS error occurs because of WSC and SCAYT conflict (when they work together).
Also we propose to divide this ticket into two separate tickets: one about styles and another about JS error.
comment:5 Changed 14 years ago by
| Keywords: | Review? added |
|---|
We have updated SCAYT core with new attributes ("data-cke-temp" instead of "cke_temp") and restarted our hosting. Now SCAYT version should show 2.6.1 (224) and it contains the appropriate fix.
comment:6 Changed 14 years ago by
| Keywords: | Review? removed |
|---|---|
| Resolution: | → fixed |
| Status: | confirmed → closed |
I cannot reproduce the issue anymore.
We've unified the custom attributes starting with "data-cke-temp", please make update at your side.