Skip to content
This repository was archived by the owner on Aug 16, 2023. It is now read-only.

replace jquery-json with es5 equivalents #5

Closed
wants to merge 1 commit into from
Closed

Conversation

bcm
Copy link
Contributor

@bcm bcm commented Mar 11, 2013

fixes #3

@dchelimsky
Copy link
Owner

Thanks @bcm. Do you think, for the case in which the developer fails to require json2, there should be an explicit warning for the potential end-user (who doesn't know anything about javascript)? i.e. test for the existence of JSON and if it's not there alert the user to upgrade his/her browser?

@bcm
Copy link
Contributor Author

bcm commented Mar 11, 2013

I think that choice is best left to the application developer. I'm sure different people will want to handle that situation in different ways.

@dchelimsky
Copy link
Owner

Merged: 7f3d70c

@dchelimsky dchelimsky closed this Mar 14, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

consider using json2 for JSON functionality
2 participants