xuijs / xui

A tiny javascript framework for mobile web apps.
http://xuijs.com
788 stars 106 forks source link

XHR header should be X-Requested-With #51

Closed i8ramin closed 13 years ago

i8ramin commented 13 years ago

The correct header value is actually X-Requested-With, not X-Request-With.

http://en.wikipedia.org/wiki/List_of_HTTP_header_fields

filmaj commented 13 years ago

Thank you sir, someone's gotta pay attention to this stuff eh? ;)

i8ramin commented 13 years ago

No problem. Btw, I work for Etsy and we are using xui for our "mobile friendly" site. My team and I are interested in contributing to the project if you guys are looking for help and/or contributors.

We can continue to submit patches if that is the preferred method.

Thank you, Ramin

On Aug 29, 2011, at 11:33 AM, filmajreply@reply.github.com wrote:

Thank you sir, someone's gotta pay attention to this stuff eh? ;)

Reply to this email directly or view it on GitHub: https://github.com/xui/xui/pull/51#issuecomment-1932248

filmaj commented 13 years ago

yea sounds good, patches are always nice. There's no real 'roadmap', just what's laid out in the issues list on github pretty much. I think figuring out IE / Windows Phone support properly is a good idea (whether to support both in one profile or to split them out, for example)... other than that it's pretty open to suggestions