From 92693709ffe70ba0b2e61ff585d25b36860b4e49 Mon Sep 17 00:00:00 2001
From: Christopher Baines <mail@cbaines.net>
Date: Fri, 13 Mar 2015 14:46:50 +0000
Subject: Add some more comments and notes

---
 leaflet-indoor.js | 24 +++++++++++++++++-------
 1 file changed, 17 insertions(+), 7 deletions(-)

diff --git a/leaflet-indoor.js b/leaflet-indoor.js
index aede15c..835d66f 100644
--- a/leaflet-indoor.js
+++ b/leaflet-indoor.js
@@ -9,6 +9,9 @@
 L.Indoor = L.Class.extend({
 
     options: {
+        // by default the levels are expected to be in the level attribute in
+        // the feature properties, pass a replacement function in options if
+        // this is not the case.
         getLevel: function(feature) {
             return feature.properties.level;
         }
@@ -82,6 +85,8 @@ L.Indoor = L.Class.extend({
         if (this._level !== null) {
             if (this._level in this._layers) {
                 this._map.addLayer(this._layers[this._level]);
+            } else {
+                // TODO: Display warning?
             }
         }
     },
@@ -93,11 +98,9 @@ L.Indoor = L.Class.extend({
         this._map = null;
     },
     addData: function(data) {
-        var layers = this._layers;
-
-        var options = this.options;
-
-        var features = L.Util.isArray(data) ? data : data.features;
+        var layers = this._layers,
+            options = this.options,
+            features = L.Util.isArray(data) ? data : data.features;
 
         features.forEach(function (part) {
 
@@ -106,13 +109,18 @@ L.Indoor = L.Class.extend({
             var layer;
 
             if (typeof level === 'undefined' ||
-                level === null)
+                level === null) {
+                // TODO: Display warning
+
                 return;
+            }
 
             if (!("geometry" in part)) {
+                // TODO: Not sure if this is still needed/display warning
                 return;
             }
 
+            // if the feature is on mutiple levels
             if (L.Util.isArray(level)) {
                 level.forEach(function(level) {
                     if (level in layers) {
@@ -126,7 +134,7 @@ L.Indoor = L.Class.extend({
 
                     layer.addData(part);
                 });
-            } else {
+            } else { // feature is on a single level
                 if (level in layers) {
                     layer = layers[level];
                 } else {
@@ -194,6 +202,8 @@ L.Control.Level = L.Control.extend({
 
     options: {
         position: 'bottomright',
+
+        // used to get a unique integer for each level to be used to order them
         parseLevel: function(level) {
             return parseInt(level, 10);
         }
-- 
cgit v1.2.3