Skip to content

Commit

Permalink
Rename id of orientation button to orientation-button (#10)
Browse files Browse the repository at this point in the history
In Firefox and possibly some versions of Chrome, having an element with an id of `orientation` replaced the `window.orientation` numerical value with a pointer to the element. This appears to be fixed in upcoming versions of Firefox, but I'm renaming it anyway to fix it in the current version and any other browsers where it may be an issue.

Orientation in Firefox is still wonky, but that's a separate matter.
  • Loading branch information
brianchirls committed Oct 4, 2015
1 parent ec651b2 commit 414a829
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion build/vr.dev.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion build/vr.js

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion build/vr.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/entry.js
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@
vrButton.className = 'unsupported';
vrButton.addEventListener('click', VR.requestVR, false);

orientationButton = svgButton(require('raw!open-iconic/svg/compass.svg'), 'orientation');
orientationButton = svgButton(require('raw!open-iconic/svg/compass.svg'), 'orientation-button');
orientationButton.setAttribute('title', 'Toggle Orientation');
orientationButton.className = 'unsupported';
orientationButton.addEventListener('click', toggleOrientation, false);
Expand Down

0 comments on commit 414a829

Please sign in to comment.