Simon Hunt

GUI -- removed commented out tests.

Change-Id: I3b37263edefbf947846dd2f9b2d7cbd4295e0c0f
...@@ -115,6 +115,7 @@ ...@@ -115,6 +115,7 @@
115 115
116 function quickHelp(view, key, code, ev) { 116 function quickHelp(view, key, code, ev) {
117 // TODO: show quick help 117 // TODO: show quick help
118 + // delegate to QuickHelp service.
118 //libApi.quickHelp.show(keyHandler); 119 //libApi.quickHelp.show(keyHandler);
119 console.log('QUICK-HELP'); 120 console.log('QUICK-HELP');
120 return true; 121 return true;
...@@ -122,6 +123,7 @@ ...@@ -122,6 +123,7 @@
122 123
123 function escapeKey(view, key, code, ev) { 124 function escapeKey(view, key, code, ev) {
124 // TODO: plumb in handling of alerts and quick help dismissal 125 // TODO: plumb in handling of alerts and quick help dismissal
126 + // We will delegate to the Alert / QuickHelp Services as appropriate.
125 /* 127 /*
126 if (alerts.open) { 128 if (alerts.open) {
127 closeAlerts(); 129 closeAlerts();
......
...@@ -77,18 +77,6 @@ describe('factory: fw/util/keys.js', function() { ...@@ -77,18 +77,6 @@ describe('factory: fw/util/keys.js', function() {
77 element.dispatchEvent(ev); 77 element.dispatchEvent(ev);
78 } 78 }
79 79
80 - // === Theme related tests
81 - // TODO: fix these tests once we have ThemeService
82 -/*
83 - it('should start in light theme', function () {
84 - expect(ks.theme()).toEqual('light');
85 - });
86 - it('should toggle to dark theme', function () {
87 - jsKeyDown(elem, 84); // 'T'
88 - expect(ks.theme()).toEqual('dark');
89 - });
90 -*/
91 -
92 // === Key binding related tests 80 // === Key binding related tests
93 it('should start with default key bindings', function () { 81 it('should start with default key bindings', function () {
94 var state = ks.keyBindings(), 82 var state = ks.keyBindings(),
......