search

prototypejs / prototype

Prototype JavaScript framework
http://prototypejs.org/
Other
3.54k stars 639 forks source link

adding simple JSONP support #294

Open jwestbrook opened 9 years ago

jwestbrook commented 9 years ago

For issue #197 , this should add transparent JSONP support to all browsers that PrototypeJS supports.

I added unit tests for a cross domain url but was exactly sure how to make WEBrick do it correctly.

savetheclocktower commented 9 years ago

Let's pencil this in for 1.8.

jwestbrook commented 9 years ago

no worries - had this PR sitting on my laptop and seeing all the updates reminded me to open this.

savetheclocktower commented 7 years ago

Despite jQuery's way of handling this, I'm hesitant to make it part of Ajax.Request, since the JSONP transport is completely different from your ordinary XMLHttpRequest. I would prefer a separate class like Ajax.JSONRequest. Thoughts?

jwestbrook commented 7 years ago

actually that would make it very clear if the dev was expecting JSONP to use the JSONPRequest method (and don't name it JSONRequest thats confusing)