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

CommentFileSizeAuthor
#2 with external links.png25.51 KBezraw
#2 without extermal links.png25.49 KBezraw

Comments

quicksketch’s picture

quicksketch
he/him
commented

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

ezraw’s picture

ezraw commented
StatusFileSize
new without extermal links.png25.49 KB
new with external links.png25.51 KB

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.

ezraw’s picture

ezraw commented

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.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
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?

ezraw’s picture

ezraw commented

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.

quicksketch’s picture

quicksketch
he/him
commented
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.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

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

Status: Fixed » Closed (fixed)

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

ronan’s picture

ronan commented

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