My Orientation Change polyfill module

Orientation.js

var OrientationManager = (function () {
    "use strict";

    var supportsOrientationChange = window.hasOwnProperty("onorientationchange")
      , orientationEvent = supportsOrientationChange ? "orientationchange" : "resize"
      , currentOrientation = window.orientation || 0
      , currentWidth = window.outerWidth;

    function orientationChanged(callBack) {
      var newWidth = window.outerWidth
        , newOrientation = window.orientation || (newWidth !== currentWidth) ? 1 : 0;

      // only change orientation and call provided callback if necessary
      if (newOrientation !== currentOrientation) {
          currentOrientation = newOrientation;
          currentWidth = newWidth;
          if (callBack) { callBack(); }
      }
    }

    // exports
    return {
        supportsOrientationChange: supportsOrientationChange
        , orientation: currentOrientation
        , orientationEvent: orientationEvent
        , orientationChanged: orientationChanged
    };
} ());

// jQuery binding usecase
$(function($){	
    $(window).on(orientationEvent, function (){OrientationManager.orientationChanged(myCallback);});
});

This script/polyfill also assumes 2 things about the page - html5 doctype and the viewport constrained.

I don't think code will run as is. Line 32 tries to use the orientationEvent variable but it is outside of the closure that that variable is declared in. I think it should instead use OrientationManager.orientationEvent.

Looks good. Simple, I like it. I have a few suggestions:

1. Make the OrientationManager.orientationChanged() event support multiple callbacks.

2. When the OrientationManager.orientationChanged() event fires the callback, either pass in the OrientationManager as the context this (i.e. callBack.apply(OrientationManager)) or pass the OrientationManager, or the newOrientation as an argument to the callBack().

   I think this would make it slightly easier for the consumer since they won't have to turn around and call OrientationManager.currentOrientation in their callback function.

Hmm. Question. Is there a reason you don't set window.orientation, if this is meant as a polyfill?

I created a repo under orientation-change-polyfill.
🍺 Feel free to contribute.