Added first-run infographic screens for IT2Me and Me2Me.

remoting/webapp/ui_mode.js

Index: remoting/webapp/ui_mode.js
diff --git a/remoting/webapp/ui_mode.js b/remoting/webapp/ui_mode.js
index 30a7c44908005eb381f4fdfa4f9b6b0e6a4689f5..ddffe204595b66db36d2564bf77aba321ad74f79 100644
--- a/remoting/webapp/ui_mode.js
+++ b/remoting/webapp/ui_mode.js
@@ -119,3 +119,32 @@ remoting.setMode = function(mode) {
 remoting.getMajorMode = function() {
   return remoting.currentMode.split('.')[0];
 };
+
+remoting.showOrHideIt2MeUi = function() {
+  var show = !!window.localStorage.getItem('it2me-visited');

[simonmorris, 2012/03/29 16:34:38]

It would be better to have constants for the names of localStorage items, to
avoid problems with typos.

But it's probably better to have a separate CL that converts all uses of item
names, and puts them in one place.

[simonmorris, 2012/03/29 16:34:38]

I think 'visited' is a clearer name for this variable than 'show'.

[Jamie, 2012/03/29 18:01:47]

Done.

[Jamie, 2012/03/29 18:01:47]

Yes, I didn't bother because all the uses are so close together, but a repo of
all local storage keys might be useful.

+  document.getElementById('it2me-first-run').hidden = show;
+  document.getElementById('it2me-content').hidden = !show;
+};
+
+remoting.showOrHideMe2MeUi = function() {
+  var show = !!window.localStorage.getItem('me2me-visited');
+  document.getElementById('me2me-first-run').hidden = show;
+  document.getElementById('me2me-content').hidden = !show;
+};
+
+remoting.showIt2MeUiAndSave = function() {
+  window.localStorage.setItem('it2me-visited', true);
+  remoting.showOrHideIt2MeUi();
+};
+
+remoting.showMe2MeUiAndSave = function() {
+  window.localStorage.setItem('me2me-visited', true);
+  remoting.showOrHideMe2MeUi();
+};
+
+remoting.resetInfographics = function() {
+  window.localStorage.removeItem('it2me-visited');
+  window.localStorage.removeItem('me2me-visited');
+  remoting.showOrHideIt2MeUi();
+  remoting.showOrHideMe2MeUi();
+}