Merge pull request #171 from RussTedrake/hideMenu

Adds support for query parameter "hideMenu" (attempt #2)

client-data/board.html

@@ -35,7 +35,7 @@
 
 	<div id="loadingMessage">{{translations.loading}}</div>
 
-	<div id="menu">
+	<div id="menu" {{#hideMenu}}style="display:none;"{{/hideMenu}}>
 		<div id="menuItems">
 			<ul id="tools" class="tools">
 				<li class="tool" tabindex="-1">

server/templating.js

@@ -68,6 +68,7 @@ class BoardTemplate extends Template {
     const boardUriComponent = parts[1];
     params["boardUriComponent"] = boardUriComponent;
     params["board"] = decodeURIComponent(boardUriComponent);
+    params["hideMenu"] = parsedUrl.query.hideMenu == "true" || false;
     return params;
   }
 }

tests/integration.js

@@ -97,6 +97,11 @@ function testBoard(browser) {
     page = testPencil(page);
     page = testCircle(page);
     page = testCursor(page);
+
+    // test hideMenu
+    browser.url(SERVER + '/boards/anonymous?lang=fr&hideMenu=true').waitForElementNotVisible('#menu');
+    browser.url(SERVER + '/boards/anonymous?lang=fr&hideMenu=false').waitForElementVisible('#menu');
+
     page.end();
 }