Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

We've moved the forum!

Please use forum.silverstripe.org for any new questions (announcement).
The forum archive will stick around, but will be read only.

You can also use our Slack channel or StackOverflow to ask for help.
Check out our community overview for more options to contribute.

Archive /

Our old forums are still available as a read-only archive.

Moderators: martimiz, Sean, Ed, biapar, Willr, Ingo

Bug in TinyMCE2


Go to End


9 Posts   5438 Views

Avatar
Markus

Google Summer of Code Hacker, 152 Posts

8 August 2007 at 3:48am

Edited: 08/08/2007 3:59am

There is a bug in TinyMCE which makes it unusable for IE. I attached a patch since I have not the rights to commit to the TinyMCE code... I also filed the issue at their Sourceforge tracker

Patch:

Index: tiny_mce_src.js
===================================================================
--- tiny_mce_src.js	(revision 39628)
+++ tiny_mce_src.js	(working copy)
@@ -1571,7 +1571,7 @@
 			return;
 		}
 
-		if (tinyMCE.isRealIE && window.event.type == "readystatechange" && document.readyState != "complete")
+		if (tinyMCE.isRealIE && window.event && window.event.type == "readystatechange" && document.readyState != "complete")
 			return true;
 
 		if (tinyMCE.isLoaded)

Avatar
Markus

Google Summer of Code Hacker, 152 Posts

9 August 2007 at 3:00am

Thanks for patching it Andy.

Avatar
elijahlofgren

Google Summer of Code Hacker, 222 Posts

9 August 2007 at 9:05am

Avatar
Markus

Google Summer of Code Hacker, 152 Posts

10 August 2007 at 3:47am

Edited: 10/08/2007 3:48am

No problem :-) I didn't look at those tickets but yes, it should fix both issues.

There is another javascript bug (at least in IE7):
The confirmation dialog ("Are you sure you want to navigate away from this page...") appears also if no changes where made.

There was somewhere a discussion about that but I can't find it.. have you an idea where the bug resides Elijah? Otherwise I'll open a ticket for it.

Avatar
elijahlofgren

Google Summer of Code Hacker, 222 Posts

10 August 2007 at 7:15am

> There is another javascript bug (at least in IE7):
The confirmation dialog ("Are you sure you want to navigate away from this page...") appears also if no changes where made.
> There was somewhere a discussion about that but I can't find it.. have you an idea where the bug resides Elijah? Otherwise I'll open a ticket for it.

I think this is the bug you are looking for: Ticket #1372 - Change Detection Broken in TinyMCE 2.1.1.1 cause Confirm dialogue every time page is edited :)

Avatar
Markus

Google Summer of Code Hacker, 152 Posts

10 August 2007 at 7:55am

Right.. I'm talking about that bug.. are you already working on it Elijah?
At the moment I'm busy but I could help you by the end of next week..

Avatar
elijahlofgren

Google Summer of Code Hacker, 222 Posts

10 August 2007 at 8:00am

> Right.. I'm talking about that bug.. are you already working on it Elijah?

No, I'm not. I did find a hack to work around the problem though: http://open.silverstripe.com/attachment/ticket/1372/Hack-To-Fix-TinyMCE-Change-Detection-cms-gsoc-r38861.diff

> At the moment I'm busy but I could help you by the end of next week..

It'd be great if you would work on this. This bug is quite annoying, and I don't enjoy working on TinyMCE very much. ;)

Avatar
Markus

Google Summer of Code Hacker, 152 Posts

13 August 2007 at 8:33pm

OK, I'll see what I can do.. I plan to work on this on Friday.. so if you find a solution or more information about this bug in the meantime tell me!

Go to Top