Incompatibility with Wysiwyg API + TinyMCE

ezraw - February 23, 2009 - 03:52
Project:External Links
Version:6.x-1.6
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

In IE 7, External Link conflicts with TinyMCE (3.2.1.1) with wysiwyg (1.0), rendering TinyMCE unusable.

#1

quicksketch - February 23, 2009 - 04:15

Could you provide more information (a screenshot would be awesome), about exactly how it conflicts?

#2

ezraw - February 23, 2009 - 05:03

Sorry to be unclear. None of the TinyMCE buttons show up, so you can't do any formatting or linking, etc. See attachments for example.

AttachmentSize
with external links.png 25.51 KB
without extermal links.png 25.49 KB

#3

ezraw - February 23, 2009 - 05:21

Would it help to know what other modules are installed? Let me know if you can't reproduce it locally and I can provide more info.

#4

sun - February 23, 2009 - 13:12
Title:Incompatibility with TinyMCE » Incompatibility with Wysiwyg API + TinyMCE
Component:Miscellaneous» Code
Status:active» postponed (maintainer needs more info)

Does that mean that just disabling External Links module makes TinyMCE work again? Did you test other editors as well?

#5

ezraw - February 23, 2009 - 13:30

That is correct. When I disable External Links, TinyMCE works again. Unfortunately I don't have any other editors installed right now to test with.

FWIW, in trying to isolate this, I found two other modules that cause the same problem with wysiwyg + TinyMCE -- Popups and Date Popup.

#6

quicksketch - March 14, 2009 - 05:29
Status:postponed (maintainer needs more info)» fixed

This should be fixed in the 1.7 version since we now use an alternate approach for adding the external icon. Previously we were adding a background image directly on the link (or tinyMCE button in your case). Now we check that the link is "display: inline" before adding the icon. Since tinyMCE buttons are all display: block, they should be excluded.

Separately, it's confusing why the links would be affected at all, since they're javascript: buttons and not external at all.

Regardless, I tested with WYSIWYG module on both FCKeditor and tinyMCE and it's fixed in the 1.7 version.

#7

sun - March 14, 2009 - 16:49

I wonder whether that test will have false positives in custom themes, but anyway - thanks for taking this on, quicksketch!

#8

System Message - March 28, 2009 - 16:50
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

#9

ronan - April 7, 2009 - 22:16

For what it's worth, I also had this issue, and it turns out the problem (at least in my case) was not the javascript, but rather that the extlink's CSS file was the straw that broke the camel's back of IE's 31 CSS file limit. TinyMCE was then unable to load it's own stylesheet.

The solution, in case anybody else has this issue and finds this thread. Is to reduce the number of CSS files you use or turn on CSS aggregation.

Hope this helps.
Ronan

 
 

Drupal is a registered trademark of Dries Buytaert.