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

Replace prop('readonly') with native readOnly #1311

Merged
merged 1 commit into from
Jun 27, 2013

Conversation

koenpunt
Copy link
Collaborator

Replaced use of prop method for native readOnly to be compatible with jquery 1.4

@davereid
Copy link

Confirmed this fixes errors happening with jQuery 1.4.*.

@pfiller
Copy link
Contributor

pfiller commented Jun 27, 2013

Shoot. I almost pushed back on the original PR to suggest just accessing the native property and I wasn't even thinking about backwards compatibility. I'd really like to see Chosen move towards less framework-based functions whenever possible.

Thanks for the quick update, @koenpunt

pfiller added a commit that referenced this pull request Jun 27, 2013
Replace prop for jQuery legacy support
@pfiller pfiller merged commit d480561 into harvesthq:master Jun 27, 2013
@koenpunt koenpunt deleted the replace-prop branch June 28, 2013 16:42
pfiller added a commit that referenced this pull request Jul 2, 2013
Replace prop for jQuery legacy support #1311
Prevent default on arrow down #1309
Explicitly add files for gh-pages #1307
Fix RTL Disabled Search Issue #1300
Fix scroll bug in Prototype version #1301
Read-only search field when search is disabled. #1294
Remove Grunt-Bump #1280
Make sure Chosen isn't disabled before showing results. #1282
Fix choice build after update #1286
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.

3 participants