Minor js things

[jdalton]

I was browsing the js and noticed in controler.js

head = doc.head || doc.getElementsByTagName( "head" )[0] || docElem

is more weight than

head = doc.getElementsByTagName( "head" )[0] || docElem

for something that's executing once and being stored in a var it seems silly to html5 fork it.

Also curious about

//might need to wait until DOMReady in IE...
if( lastlink.nextSibling ){
  head.insertBefore(lk, lastlink.nextSibling );
} else {
  head.appendChild( lk );
}

I'm not sure of the lastlink.nextSibling check, is order really important?
Can't it all be done via the head.insertBefore(lk, lastlink.nextSibling ); without forking for appendChild?

[Wilto]

As explanation for the first thing: if it weren’t for over-, I’m not sure I’d do any engineering at all.

For the second, yeah, you might be right—I’m not sure why order would matter there. I’ll swap that later on today.

[mathiasbynens]

@jdalton Gah, you and your love/hate relationship with document.head! ;)

[Wilto]

I get it, man. I still lay awake at night thinking about <dialog>, and that’s not even a thing anymore.