Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#390 alternative fix for IE8 #446

Merged
merged 1 commit into from
Feb 20, 2017
Merged

#390 alternative fix for IE8 #446

merged 1 commit into from
Feb 20, 2017

Conversation

nimasmi
Copy link
Contributor

@nimasmi nimasmi commented Dec 2, 2014

Alternative way to resolve #390

@Kevin-Hamilton
Copy link

This worked for me.

@malsup please consider merging this pull request. The current code as implemented in #390 (relying on a userAgent test) fails on IE9 when running in compatibility mode.

(Note: the reason it fails is because because in compatibility mode, IE9 passes the userAgent test, but $(form.elements) behaves as per IE8 and earlier - returning the HTMLFormElement rather than an HTMLFormControlsCollection)

@kevindb
Copy link
Contributor

kevindb commented Jan 15, 2017

This pull request was merged into the maintained fork of this plugin at #8

@nimasmi
Copy link
Contributor Author

nimasmi commented Jan 16, 2017

Many thanks.

@kevindb kevindb merged commit 49b46f5 into jquery-form:master Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IE8 ajaxForm issue. Incorrectly collect form data
3 participants