Flipped the logic for #587

thexerteproject · Mar 2, 2017 · a1c736e · a1c736e
1 parent 436c784
commit a1c736e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 12 deletions.
diff --git a/modules/xerte/parent_templates/Nottingham/models_html5/menu.html b/modules/xerte/parent_templates/Nottingham/models_html5/menu.html
@@ -53,7 +53,7 @@
 				}
 				var pageNum = x_params.tocNumbers == "false" ? "" : (i+1) + ") ";
 				$thisItem.html(pageNum + x_pages[i+numOffset].getAttribute("name"));
-				$thisItem.append(' <i class="fa fa-check-circle" aria-hidden="true"></i>');
+				$thisItem.append(' <i class="fa fa-check-circle notvisited" aria-hidden="true"></i>');
 			}
 
 			$("#menuItems .menuItem")
@@ -128,7 +128,7 @@
 			}
 
 			$("#menuItems .menuItem").each(function(i) {
-				if (x_pageInfo[i].built) $(this).find('i').addClass('visited');
+				if (x_pageInfo[i].built) $(this).find('i').removeClass('notvisited');
 			});
 		}
 
@@ -166,6 +166,10 @@
 		margin-bottom:	5px;
 		text-align:		left;
 	}
+
+	#menuItems .menuItem .notvisited {
+		display: none;
+	}
 
 	#menuItems {
 		width:	100%;

diff --git a/themes/Nottingham/btnTopPurple/btnTopPurple.css b/themes/Nottingham/btnTopPurple/btnTopPurple.css
@@ -90,16 +90,6 @@ h3 {
     border-color: #D4CB43 !important;
 }
 
-#menuItems .menuItem i {
-		display: none;
-}
-
-#menuItems .menuItem i.visited {
-    display: block;
-    float: right;
-    margin: 2px -8px 0 0;
-}
-
 .ui-dialog #menuItems .ui-state-default {
 	background:	none;
 	color:	black;