joseeight / requestr Goto Github PK

• Create comparison of total file size loading via normal method and what Requestr and the API deliver
• Also include test for embedded data URIs

Some browsers, like Chrome, interpret and therefore change the original CSS rules, so it is best to parse for the URLs in the interpreted string but then use the original CSS string and replace the URLs in that string and not the interpreted one as it is currently happening. This will avoid interpretation errors and assure only the URLs are changed.

Since IE 11 wasn't released until after the initial build, it was never tested, and it is broken. The library breaks loading, and this needs to be fixed, could be a quirk of IE, as IE10 is fine.

Error:

SCRIPT438: Object doesn't support property or method 'addEventListener'
SCRIPT438: Object doesn't support property or method 'querySelector'

Seems like there is something happening with the window object.

Reloading the page seems to be broken, probably due to the API update, works fine on Chrome, but not on IE 10 or Firefox 31.

Getting error:

TypeError: j.Data is undefined

This should include adding a test case to ensure it won't happen again.

To eliminate a delay upon loading, the library should create a XMLHttpRequest to the service end-point before the library is even parsed by the browser. This will ensure the OPTIONS request is made as soon as possible allowing the library to immediate request resources without the delay of the OPTIONS check (when using CORS).

When parsing requesting elements, Requestr should exclude elements which values are data URIs as they do not need to be resolved and should not be requested from the API. Obvious oversight here, just need a filter to check.

Can see one item in the array requesting a URI from the asset server here.

http://cl.ly/image/050V211D1B3k