Merge pull request #8868 from huffman/fix/away-overlays

Remove showing of overlays in away.js
This commit is contained in:
Chris Collins 2016-10-20 16:42:05 -07:00 committed by GitHub
commit 414a79d986

View file

@ -54,7 +54,6 @@ var AWAY_INTRO = {
var isEnabled = true;
var wasMuted; // unknonwn?
var isAway = false; // we start in the un-away state
var wasOverlaysVisible = Menu.isOptionChecked("Overlays");
var eventMappingName = "io.highfidelity.away"; // goActive on hand controller button events, too.
var eventMapping = Controller.newMapping(eventMappingName);
var avatarPosition = MyAvatar.position;
@ -178,12 +177,6 @@ function goAway(fromStartup) {
playAwayAnimation(); // animation is still seen by others
showOverlay();
// remember the View > Overlays state...
wasOverlaysVisible = Menu.isOptionChecked("Overlays");
// show overlays so that people can see the "Away" message
Menu.setIsOptionChecked("Overlays", true);
// tell the Reticle, we want to stop capturing the mouse until we come back
Reticle.allowMouseCapture = false;
// Allow users to find their way to other applications, our menus, etc.
@ -233,9 +226,6 @@ function goActive() {
hideOverlay();
// restore overlays state to what it was when we went "away"
Menu.setIsOptionChecked("Overlays", wasOverlaysVisible);
// tell the Reticle, we are ready to capture the mouse again and it should be visible
Reticle.allowMouseCapture = true;
Reticle.visible = true;