conditionally hide controls

[cvan]

this is dependent on the foundational work in PR #20 for allowing overrides via URL. I'd recommend reviewing and merging that one first. :smiley:

though issue #15 requests to "have properties we can set via JS," I think allowing these overrides from a URL is a good first step.

once there is a standalone bundle that's generated (see issue #21), I can see how it might be useful to initialise the player without the video controls via like so:

eleVRWebPlayer({
  controls: false
});

---

here is a list of the items addressed in this PR:

• removes unused #forkongithub in index.html and css/forkme.css stylesheet (was interfering with the height of the #video-container when the #video-controls were hidden)
• prevents video controls (in #video-container) and event listeners (controls.create()) from getting initialised if controls are to be hidden on page load (e.g., by passing ?controls=false or #{"controls": false})
  • admittedly, all the if statements in controls.js does add some cruft. could just simply hide the controls using CSS, but thought this would be a better approach - to just not expose the global references if they don't get rendered (though if the hash changes or a postMessage message is received that says to disable the controls, the global references are set and everything just works)
• fixes container dimensions so they get recalculated on window resize
• fixes case where querystring would persist when postMessage would set window.location.hash but would leave window.location.search, thereby confusing which settings to read from (could uplift change to PR #20)

  test case

to test how this would work from an iframe, drop this into a file like iframe.html in the root directory:

<iframe src="/?controls=false&amp;autoplay=1&amp;video=http://cdn2.vrideo.com/prod_videos/v1/mYNVcD6_480p_full.mp4" style="border: 0; position: absolute; left: 0; top: 0; height: 100%; width: 100%" allowfullscreen></iframe>

and to test muting, open the JS Console and run this command:

window.postMessage({autoplay: true, mute: true}, '*')

and to unmute:

window.postMessage({autoplay: true, unmute: true}, '*')

---

feedback very welcome. thanks! :smiley_cat:

[cvan]

updated patch to remove all conditional logic and just always call function to set up query selectors.

[cvan]

will update now that #20 has been merged

[cvan]

ready for re-review for this now that #20, #24, #25 are merged - and I just opened #26 to split unrelated things out of this PR.

also added sample usage to README.

[hawksley]

Yay! Thanks for updating the readme as well. :)