The HTML5 shiv has been updated to v3 and the InnerShiv script is dead. From http://jdbartlett.com/innershiv/:

STOP! Don't use innerShiv!
html5shiv now patches for the innerHTML issue! Update html5shiv and you won't have to use innerShiv anymore:

http://html5shiv.googlecode.com/

That still leaves the monkey patch. In its current form, it doesn't work since it uses a function that isn't in the v3 of the HTML5 shiv.

Also, the monkeypatch is not needed for jQuery 1.7, but the jQuery Update module doesn't use 1.7 yet: #1386294: Release jQuery 1.7 for Drupal 7. Do we require the jQuery Update module once its updated? Or do we not? And what do we do in the interim?

Comments

danny_joris’s picture

danny_joris commented
ericduran’s picture

ericduran commented

Hmm, I haven't looked into this yet.

But it seems instead of doing innerShiv(response.data) we can do html5.shivDocument(response.data) until the jquery_update project updates to 1.7. Then after that we can just required 1.7 in order to properly support html5 elements.

ericduran’s picture

ericduran commented
Status: Active » Fixed

Yea, I just updated to the latest html5shiv and added a jquery_update dependency.

People should really be using jquery 1.7 :)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

mossy2100’s picture

mossy2100
Location Brisbane, Australia
commented

It would be useful to newcomers to this project to remove the reference to innershiv from the project page.