search

sathomas / acc-wizard

Wizard implementation for bootstrap based on accordion
216 stars 76 forks source link

Prev and Next Buttons failing in IE8+ (Only using 1 <form>) #10

Closed curtismorte-xx closed 10 years ago

curtismorte-xx commented 11 years ago

I have been doing some extensive browser testing with this and for the most part it is holding up rather well. (Only Failed in IE8+)

However, when in IE8+ (yes IE6 & IE7 work), the previous and next buttons fail after the first accordion body. So you can click the next button and advance to the second accordion body, but when you click the next or prev buttons in the second body the form submits instead of going to the next accordion boyd.

My actual .accordion division is wrapped in a form tag to let everyone know.

Let me know if anyone has any information on this.

sathomas commented 11 years ago

Unfortunately, I don't have the ability to test IE8 myself. Pull requests welcome.

curtismorte-xx commented 11 years ago

So I just tested MaBasic's fork where you don't append #hastags to the URL. It worked flawlessly with IE8+ https://github.com/mabasic/acc-wizard

You could use that as a reference to compare the differences.

I would like to note that your Demo does work perfectly with IE. This is because you use 4 < form > tags and make each accordion its own form essentially. In my working version, I am wrapping the .accordion division with a form tag.

So for users who are wrapping with a form tag and changing the element to which the buttons are added I would recommend using MaBasics. However if you are submitting each form individually or creating a clever work around your ACC Wizard files will work flawlessly.

Hopefully that can give you some insight. Cheers for a great repo!

sathomas commented 11 years ago

Thanks for the insight!

sathomas commented 10 years ago

closing with link to MaBasic's fork. no interest in adding ie8 support myself