Disable mode doesn't work in IE.. the button become inactive but the form not submitted

CommentFileSizeAuthor
#7 680932_ie9_support_6x.patch840 bytesgreggles

Comments

optalgin’s picture

optalgin commented
Title: Disable mode doesn't work in IE.. the button become inactive but the form not submitted » Disable mode doesn't work in IE.. the button disabled but the form not submitted
YK85’s picture

YK85 commented
Status: Active » Fixed

i just installed and tested this module and it works in IE
please re-open if you still experience the problem

optalgin’s picture

optalgin commented

Please note that "disabled" mode is "experimental"
There are some issues with it which I haven't solved yet
Especially with forms that have more then one submit button (i.e. save / review / save as draft)

YK85’s picture

YK85 commented
Status: Fixed » Active

oh I see now, thanks for the explanation
changing back to 'active'

i'm not a programmer but please let me know if there is a patch to resolve this issue and i can help test =)
awesome module!!

optalgin’s picture

optalgin commented
Status: Active » Postponed
arijeet.bannerjee’s picture

arijeet.bannerjee commented

I have used the following code to resolve this issue for IE9.

--- hide_submit.js     
+++ hide_submit.js      
@@ -15,8 +15,8 @@
       .attr("value", $obj.attr("value"));
     $obj
       .attr("disabled", "disabled")
-      .siblings('input:submit').attr("disabled", "disabled")
-      .parents("form").submit();
+      .siblings('input:submit').attr("disabled", "disabled");
+      $obj.parents("form").submit();
greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Postponed » Needs review
StatusFileSize
new 680932_ie9_support_6x.patch840 bytes

Here is a patch of #6 along with a little extra cleanup.

I don't maintain the 6.x version, really, so I don't plan to commit this unless there is a wide request for it and *testing* on a large number of platforms.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

Thanks for submitting this issue.

The 6.x branch of this module is not supported. This issue was filed against the 6.x version so I'm closing it because it doesn't seem relevant to the 7.x version.

If the issue is relevant for the 7.x version: I apologize for closing it. Can you please reopen it and adjust the version number?