Fix marker placement bug

Readme.md

@@ -148,6 +148,9 @@ Tests are carried out with [QUnit](http://qunitjs.com). To run tests, either ope
 
 ## Changelog
 
+v1.1.1 (July 17, 2015)
+
+- Fixed marker position bug
 
 v1.1.0 (July 17, 2015)
 

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "pinpoint",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "authors": [
     "ejb <[email protected]>"
   ],

dist/pinpoint.js

@@ -175,11 +175,13 @@ Pinpoint.prototype.addMarker = function(mopts, index){
     // L.marker([50.505, 30.57]).addTo(map);
 
     var miList = this.element.querySelectorAll('.marker-inner');
-    mi = miList[miList.length-1];
-	mi.style.marginLeft = -mi.outerWidth/2;
-	setTimeout((function(){
-    	mi.style.marginLeft = -mi.outerWidth/2;
-	}).bind(this),100);
+    for (var i = 0; i < miList.length; i++) {
+        var mi = miList[i];
+    	mi.style.marginLeft = (-mi.offsetWidth/2)+'px';
+    	setTimeout((function(){
+        	mi.style.marginLeft = (-mi.offsetWidth/2)+'px';
+    	}).bind(this),100);
+    }
 
     if (this.opts.markerdragend) {
         icon.on('dragend', this.opts.markerdragend);

src/js/pinpoint.js

@@ -175,11 +175,13 @@ Pinpoint.prototype.addMarker = function(mopts, index){
     // L.marker([50.505, 30.57]).addTo(map);
 
     var miList = this.element.querySelectorAll('.marker-inner');
-    mi = miList[miList.length-1];
-	mi.style.marginLeft = -mi.outerWidth/2;
-	setTimeout((function(){
-    	mi.style.marginLeft = -mi.outerWidth/2;
-	}).bind(this),100);
+    for (var i = 0; i < miList.length; i++) {
+        var mi = miList[i];
+    	mi.style.marginLeft = (-mi.offsetWidth/2)+'px';
+    	setTimeout((function(){
+        	mi.style.marginLeft = (-mi.offsetWidth/2)+'px';
+    	}).bind(this),100);
+    }
 
     if (this.opts.markerdragend) {
         icon.on('dragend', this.opts.markerdragend);

test/pinpoint.test.js

@@ -69,3 +69,31 @@ QUnit.test( "Dynamic height", function( assert ) {
     },200);
 });
 
+QUnit.test( "Marker placement", function( assert ) { 
+    var data = {
+        "lat": 51.5049378,
+        "lon": - 0.0870377,
+        "zoom": 4,
+        "aspect-ratio": "tall",
+        "hed": "Hi",
+        "dek": "You",
+        "markers": [{
+            "lat": 51.5049378,
+            "lon": - 0.0870377,
+            "label": ""
+        }]
+    };
+    data.el = '.test-map';
+
+    var p = new Pinpoint(data);    
+    assert.ok( ($('.marker-inner').offset().left / $('.map-inner').width()) < 0.5 );
+    p.remove();
+
+    data.markers[0].label = 'callout';
+    var p = new Pinpoint(data);
+    assert.ok( ($('.marker-inner').offset().left / $('.map-inner').width()) < 0.5 );
+    p.remove();
+
+});
+
+