summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSamuel Mannehed <samuel@cendio.se>2018-07-30 10:02:44 +0200
committerSamuel Mannehed <samuel@cendio.se>2018-07-30 10:29:35 +0200
commit0b903af296d6b0626cee53a3b12b0e56d84b02d9 (patch)
tree5c7bf414db66e950f480de0b84b402d31327c8af
parent862967e08950c2a9c71529420321395737e2734d (diff)
downloadnovnc-0b903af296d6b0626cee53a3b12b0e56d84b02d9.tar.gz
Remove unnecessary code
The enableDisableViewClip call in the fullscreen code didn't have any effect and should have been removed when the special case for clipping in IE and Safari fullscreen was removed in b18ef81. The setViewDrag call claimed to disable view drag on UI state change. The UI states are: init, connecting, connected, reconnecting, disconnecting, disconnected The only state where the called function didn't immediately return was "connected" and that's the only state where enabling view drag is possible. Thus it could never have been enabled when changing to the "connected" state.
-rw-r--r--app/ui.js7
1 files changed, 1 insertions, 6 deletions
diff --git a/app/ui.js b/app/ui.js
index 4fe2a3f..552ac28 100644
--- a/app/ui.js
+++ b/app/ui.js
@@ -432,11 +432,7 @@ const UI = {
UI.keepControlbar();
}
- // State change disables viewport dragging.
- // It is enabled (toggled) by direct click on the button
- UI.setViewDrag(false);
-
- // State change also closes the password dialog
+ // State change closes the password dialog
document.getElementById('noVNC_password_dlg')
.classList.remove('noVNC_open');
},
@@ -1204,7 +1200,6 @@ const UI = {
document.body.msRequestFullscreen();
}
}
- UI.enableDisableViewClip();
UI.updateFullscreenButton();
},