Not sure if this is a WYSIWYG issue or a Node Embed issue (or maybe even a ckeditor version library issue!), but when I'm using:

WYSIWYG (7.x-2.1 OR 7.x-2.x-dev) (configured to use the CKEditor library, version 3.6.4.7575) and
Node Embed 7.x-1.1
IE9 - In compatibility Mode (will need to check real IE 7 version too) Confirmed same functionality occurs in real IE 7 browser

I get a blank WYSIWYG textarea. If I use IE9 normally, the textarea displays fine.

If I disable the node embed module, my WYSIWYG displays just fine.

See attached files. The only difference is that one has IE9 compatibility mode on (broken WYSIWYG), the other has it off (working WYSIWYG).

CommentFileSizeAuthor
#4 ie9-in-compatibility-mode-1794734.patch1.98 KBgmclelland
#2 nodeEmbedIE7-doublescrollbars.jpg61.26 KBladybug_3777
nodeEmbedIE9.jpg207.71 KBladybug_3777
nodeEmbedIE7.jpg179.34 KBladybug_3777
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ladybug_3777’s picture

ladybug_3777 CreditAttribution: ladybug_3777 commented

Additional details. IE is giving the following errors;

HTML1201: localjendrupal.com is a website you've added to Compatibility View.
localjendrupal.com
SEC7115: :visited and :link styles can only differ by color. Some styles were not applied to :visited.
localjendrupal.com
SCRIPT1028: Expected identifier, string or number
node_embed.js?max4iy, line 87 character 1
SCRIPT5007: Unable to get value of the property 'invoke': object is null or undefined
ckeditor-3.0.js?max4iy, line 169 character 9

ladybug_3777’s picture

ladybug_3777 CreditAttribution: ladybug_3777 commented
FileSize
nodeEmbedIE7-doublescrollbars.jpg61.26 KB

OH! I think I found the problem!

In the following file:

[path to node embed module]/plugins/js/node_embed.js
on line 87 there is an unecessary trailing comma. 

$('#buttonCancelDialog').click(function(e) {
      $(dialog).dialog('close');
    });
    
    $(dialog).dialog('open');
    // Generate HTML markup.
    
  }, <----- YOU WANT TO REMOVE THIS COMMA!
};
})(jQuery);

Once I removed the extra comma IE 7 (and IE9 in compatibility mode) worked!

There is however one additional small problem, in IE 7 (or IE9 Comp) I'm seeing a double scrollbar on the view. (see attached file) Not a huge deal, but would be annoying to users.

ladybug_3777’s picture

ladybug_3777 CreditAttribution: ladybug_3777 commented
Issue summary: View changes

confirmed IE7

humansky’s picture

humansky CreditAttribution: humansky commented
Issue summary: View changes
Status: Active » Reviewed & tested by the community

Can confirm, removing comma as per #2 instructions worked for me for IE9 on Windows 7.

gmclelland’s picture

gmclelland CreditAttribution: gmclelland commented
FileSize
ie9-in-compatibility-mode-1794734.patch1.98 KB

Untested, but here is a simple patch that removes the comma.

  • MrHaroldA committed 9d39a41 on 7.x-1.x 
    Issue #1794734 by ladybug_3777: remove trailing comma + refactor code...
MrHaroldA’s picture

MrHaroldA CreditAttribution: MrHaroldA commented
Status: Reviewed & tested by the community » Fixed

Fixed within two years!!! ;)

And, I didn't test it (yet), as I don't have Windows/IE.

gargsuchi’s picture

gargsuchi CreditAttribution: gargsuchi commented

Doesnt work on IE11. Havent tried other versions yet.

Status: Fixed » Closed (fixed)

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