I decided to forego support for IE6 and older -- IIRC, IE7 implemented XHR, but still uses the old event listener model.
Ah, nice catch on the for loops. Updated. Thanks! On Tue, Aug 2, 2011 at 10:30 AM, Xavier MONTILLET <[email protected]>wrote: > You don't use the activex version of xhr whereas you fall back to > attachevent if addeventlistener isn't here... > You can decide to drop ie or not but not half... > And in ie xmlhttprequest.prototype.whatever > The property you store in a variable at the beginning will throw an error > in old ie versions as xmlhttprequest wonlt be defined. > > And in you for loops, you don't declare the i (or j or whatever) variable > with the var operator. > > I'll take a deeper look later if needed becaise right now I'm on my > smartphone.. > On Aug 2, 2011 6:24 AM, "Skyler Brungardt" <[email protected]> wrote: > > Hey folks. I'd like to submit some code for your feedback and > > critique. > > > > Some (brief) context. At the organization for which I work, we're > > striving to meet the WCAG 2.0 AA spec (link: > http://www.w3.org/TR/WCAG20/). > > However, that usually precludes having many nifty CSS features and > > rich media, since the WCAG spec is fairly specific about how content > > can be presented -- specifically what kinds of contrast the content > > needs to have based upon its color and its background. > > > > You can see some examples of what's compliant and what isn't with this > > tool: http://snook.ca/technical/colour_contrast/colour.html > > > > Needless to say, our design team balked at having their color palette > > restricted to such a limited set of colors. So I wrote this JS > > library to let us have our cake and eat it too. > > > > Essentially, it provides a widget on a site for enabling high-contrast > > styling regardless of the default appearance, and for making fonts > > easier to read. I'd like to have it be as turn-key as possible, and > > there's still some work to be done in that realm I think. > > > > I have it up on github here: > https://github.com/StrictlySkyler/accessorizejs/blob/master/accessorize.js > > > > I welcome your feedback! > > > > Thanks, > > > > -Skyler > > > > -- > > To view archived discussions from the original JSMentors Mailman list: > http://www.mail-archive.com/[email protected]/ > > > > To search via a non-Google archive, visit here: > http://www.mail-archive.com/[email protected]/ > > > > To unsubscribe from this group, send email to > > [email protected] > > -- > To view archived discussions from the original JSMentors Mailman list: > http://www.mail-archive.com/[email protected]/ > > To search via a non-Google archive, visit here: > http://www.mail-archive.com/[email protected]/ > > To unsubscribe from this group, send email to > [email protected] > -- To view archived discussions from the original JSMentors Mailman list: http://www.mail-archive.com/[email protected]/ To search via a non-Google archive, visit here: http://www.mail-archive.com/[email protected]/ To unsubscribe from this group, send email to [email protected]
