Placeholders supplementary, warn against polyfill #215

Merged
merged 3 commits into from Jan 2, 2013

Projects

None yet

3 participants

@zachleat
Contributor

Per discussion included in #158

@nimbupani
Member

Looks great! Can you remove the polyfillurls meta value from the file? Given we are suggesting people do not use the polyfills.

Also I think we should strongly recommend use, given this is supplementary, so their presence or absence shouldn't have an impact?

@zachleat
Contributor

Do you mean recommend use or not use? I'm getting mixed signals here.

@nimbupani
Member

I mean recommend use :/ I mean if they are supplementary there is no harm 'using' them no? Our usage of the word 'use' is merely an indication that there is no threat to breaking across all browsers that do not implement this feature when used.

@addyosmani
Contributor

@zachleat if you get a chance could you look at @nimbupani's feedback above and adjust accordingly?

@zachleat
Contributor
zachleat commented Jan 2, 2013

Done!

@addyosmani
Contributor

Thanks!

@addyosmani addyosmani merged commit bbbae7f into h5bp:master Jan 2, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment