Skip to content

include addTest by default #619

Closed
paulirish opened this Issue Jun 20, 2012 · 3 comments

3 participants

@paulirish
Modernizr member

see a lot of errors like http://stackoverflow.com/questions/11075507/modernizr-causes-errors-with-jquery-isotope reported in the wild.

and since we're moving to a 100% addtest api seems good to just say that method is core and required.

@SBoudrias

+1 !!

@paulirish
Modernizr member

@SlexAxton what changes need to happen at modulizr level to deal with this? I'd like to get this as part of 2.6 which shoudl ideally ship in the next week

@SlexAxton SlexAxton was assigned Jun 20, 2012
@SlexAxton
Modernizr member

We would just unwrap the addTest stuff. Modulizr can have unknown names and still work, as far as I know.

@drublic drublic referenced this issue in h5bp/html5-boilerplate Jul 8, 2012
Closed

Google Cache snapshot Issue #1086

@paulirish paulirish closed this in 570eef6 Jul 16, 2012
@phistuck phistuck pushed a commit that referenced this issue Aug 31, 2012
PhistucK addTest should be included by default. fixes #619 8e002b9
@patrickkettner patrickkettner pushed a commit to patrickkettner/Modernizr that referenced this issue Feb 22, 2015
@paulirish paulirish addTest should be included by default. fixes #619 2a81ff6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.