jcarr
/
noVNC
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add missing log output for UI-specific errors 

Some UI errors and one warning didn't write to the log. We want all
errors to be logged.
This commit is contained in:
Samuel Mannehed 2016-10-24 13:36:05 +02:00
parent daca5b1727
commit 7d20158b4b
1 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
app/ui.js
View File

@ -385,7 +385,9 @@ var UI;
UI.showStatus("Disconnected"); UI.showStatus("Disconnected");
break; break;
default: default:
UI.showStatus("Invalid state", 'error'); msg = "Invalid UI state";
Util.Error(msg);
UI.showStatus(msg, 'error');
break; break;
} }
@ -967,7 +969,9 @@ var UI;
} }
if ((!host) || (!port)) { if ((!host) || (!port)) {
UI.showStatus("Must set host and port", 'error'); var msg = "Must set host and port";
Util.Error(msg);
UI.showStatus(msg, 'error');
return; return;
} }
@ -1020,6 +1024,7 @@ var UI;
if (typeof msg === 'undefined') { if (typeof msg === 'undefined') {
msg = "Password is required"; msg = "Password is required";
} }
Util.Warning(msg);
UI.showStatus(msg, "warning"); UI.showStatus(msg, "warning");
}, },
@ -1231,7 +1236,10 @@ var UI;
// The browser is IE and we are in fullscreen mode. // The browser is IE and we are in fullscreen mode.
// - We need to force clipping while in fullscreen since // - We need to force clipping while in fullscreen since
// scrollbars doesn't work. // scrollbars doesn't work.
UI.showStatus("Forcing clipping mode since scrollbars aren't supported by IE in fullscreen"); var msg = "Forcing clipping mode since scrollbars aren't" +
"supported by IE in fullscreen";
Util.Debug(msg);
UI.showStatus(msg);
UI.rememberedClipSetting = UI.getSetting('clip'); UI.rememberedClipSetting = UI.getSetting('clip');
UI.setViewClip(true); UI.setViewClip(true);
document.getElementById('noVNC_setting_clip').disabled = true; document.getElementById('noVNC_setting_clip').disabled = true;