Add hidebydefault setting to playermarkers component too

webbukkit · Aug 26, 2011 · f62d46c · f62d46c
1 parent bc357dd
commit f62d46c
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/src/main/resources/configuration.txt b/src/main/resources/configuration.txt
@@ -55,6 +55,8 @@ components:
     showplayerhealth: true
     # Option to make player faces small - don't use with showplayerhealth
     smallplayerfaces: false
+  #  # Optional - make player faces layer hidden by default
+  #  hidebydefault: true
   #- class: org.dynmap.ClientComponent
   #  type: digitalclock
   - class: org.dynmap.ClientComponent

diff --git a/web/js/map.js b/web/js/map.js
@@ -284,13 +284,15 @@ DynMap.prototype = {
 
 		me.selectMap(me.defaultworld.defaultmap);
 
+		var componentstoload = 0;
 		var configset = { };
 		$.each(me.options.components, function(index, configuration) {
-			if(!configset[configuration.type])
+			if(!configset[configuration.type]) {
 				configset[configuration.type] = [];
+				componentstoload++;
+			}
 			configset[configuration.type].push(configuration);
 		});
-		var componentstoload = configset.length;
 
 		$.each(configset, function(type, configlist) {
 			loadjs('js/' + type + '.js', function() {

diff --git a/web/js/playermarkers.js b/web/js/playermarkers.js
@@ -128,6 +128,7 @@ componentconstructors['playermarkers'] = function(dynmap, configuration) {
 		}
 	});
 	dynmap.playermarkergroup = new L.LayerGroup();
-	dynmap.map.addLayer(dynmap.playermarkergroup);
+	if(!configuration.hidebydefault)
+		dynmap.map.addLayer(dynmap.playermarkergroup);
 	dynmap.layercontrol.addOverlay(dynmap.playermarkergroup, 'Players');
 };