Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Address minor JS changes recommended by @jdalton #61

Merged
merged 1 commit into from

2 participants

@addyosmani

As discussed here:

Fixes #31

@paulirish
Owner

oh i dont mind the doc.head thing but the rest lgtm

@paulirish paulirish merged commit eaca619 into h5bp:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 27, 2011
  1. @addyosmani
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 7 deletions.
  1. +3 −7 js/controller.js
View
10 js/controller.js
@@ -2,7 +2,7 @@
var doc = win.document,
docElem = doc.documentElement,
- head = doc.head || doc.getElementsByTagName( "head" )[0] || docElem,
+ head = doc.getElementsByTagName( "head" )[0] || docElem,
Modernizr = win.Modernizr;
// Supportin’ stuff.
@@ -81,12 +81,8 @@
lk.onload = respond.update;
}
- //might need to wait until DOMReady in IE...
- if( lastlink.nextSibling ){
- head.insertBefore(lk, lastlink.nextSibling );
- } else {
- head.appendChild( lk );
- }
+ head.insertBefore(lk, lastlink.nextSibling );
+
};
// Non-MQ browser, or in one of the two development modes? Exit here:
Something went wrong with that request. Please try again.