Fix disabled in IE8

[abombss]

Fixes bug in IE8 with attr 'disabled' introduced in #67 not working properly, the recommended jquery solution is to use prop instead of attr.

[pfiller]

Using the prop() function would cause jQuery 1.4 to no longer be supported. A more backwards-compatible methodology would be to access the property directly:

@is_disabled = @form_field_jq[0].disabled

Most references to attr in Chosen should probably be removed as a result of this method change in jQuery 1.6. I've gone ahead and committed this change in 9bc5fdc. Let me know if you're still seeing problems in IE8.

[abombss]

I actually just ran some more tests on my behavior and I see it was an
issue with how I was setting the disabled attribute in IE. I was using
$.attr('disabled', '') which worked in chrome & FF but not IE. I need to
explicitly be using $.removeAttr('disabled')

This issue can be closed.

On Wed, Nov 2, 2011 at 11:46 AM, Patrick Filler <
reply@reply.github.com>wrote:

Using the prop() function would cause jQuery 1.4 to no longer be
supported. A more backwards-compatible methodology would be to access the
property directly:

@is_disabled = @form_field_jq[0].disabled

Most references to attr in Chosen should probably be removed as a result
of this method change in jQuery 1.6. I've gone ahead and committed this
change in 9bc5fdc. Let me know if you're still seeing problems in IE8.

Reply to this email directly or view it on GitHub:
#327 (comment)