Hi!
I´m trying on ie6 and ie7 and it won´t oen the frame, it works just like a normal link...
It does work on firefox and chrome though...
Thanks for your help!

Comments

Rosamunda’s picture

Rosamunda commented
Title: Won´t work on internet explorer? » Won´t work on internet explorer

I´ve tried in different machines no, it won´t work. Please someone confirm this (or help me out) :)
Thanks!

rc2020’s picture

rc2020 commented

I can confirm this issue in IE7, no word about IE6 because I don't support it - although I'm inclined to believe it doesen't work there either.

Rosamunda’s picture

Rosamunda commented

IE is a headache.
I´ve opened a new issue in Modal Frame API module.

rc2020’s picture

rc2020 commented

+1 on IE being a headache. Just for the record.

markus_petrux’s picture

markus_petrux commented

I think the problem is located in automodal.js, here:

    var settings = {
      autoResize: true,
      autoFit: true,
      width: 600,
      height: 400,         // <--- This comma causes the problem in IE!!!
      //onSubmit: function (a, b) {} 
    }

Without that comma, it probably works. So it should look like this:

    var settings = {
      autoResize: true,
      autoFit: true,
      width: 600,
      height: 400         // <--- javascript in IE does not accept commas at end of list like in PHP.
      //onSubmit: function (a, b) {} 
    }
ludo1960’s picture

ludo1960 commented

Netbeans 6.8 reports the trailing comma in automodal.js is not supported in IE

Rosamunda’s picture

Rosamunda commented
Priority: Normal » Critical
Status: Active » Reviewed & tested by the community

Yay! I can confirm that it was the problem there :)
It works on IE7 (it won´t work on IE6 though, but then again, I don´t want to support that buggy, nasty and fishy version).

I think the world would be nicer without the IE whatsoever!

THANKS FOR ALL YOUR HELP! Thank you very much to ludo1960 and markus_petrux!!!!!!

Hope this gets commited very soon!

Rosamunda

dublin drupaller’s picture

dublin drupaller commented
Status: Reviewed & tested by the community » Active

just tried the solution at #5 in IE8 and it doesn't work I'm afraid. It tried to open up the modal frame, but, then just breaks out and displays it in the normal browser window.

anyone found a workaround?

jcmarco’s picture

jcmarco commented

I have tested automodal with IE8 and it works fine,
but in my case with the patch from http://drupal.org/node/767128#comment-2833562

mfer’s picture

mfer commented
Status: Active » Fixed

Fixed. Thanks. http://drupal.org/cvs?commit=408584

Status: Fixed » Closed (fixed)

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