From d55d363112ee96941a1bb253c2e896400ffc422b Mon Sep 17 00:00:00 2001
From: fkloft <felix.kloft+github@gmail.com>
Date: Sat, 21 Feb 2015 14:19:33 +0100
Subject: [PATCH] [distance-to-portal] add vendor-prefixes to transformations
 and implement updating the distance without re-rendering all portal details

Firefox supports 'transform' without prefix, but Android still requires '-webkit-'. To be sure, include properties with '-moz-', '-webkit-' and without prefix.
---
 plugins/distance-to-portal.css     |  4 +++
 plugins/distance-to-portal.user.js | 47 ++++++++++++++++++------------
 2 files changed, 33 insertions(+), 18 deletions(-)

diff --git a/plugins/distance-to-portal.css b/plugins/distance-to-portal.css
index 5f2e223e..fe66b84d 100644
--- a/plugins/distance-to-portal.css
+++ b/plugins/distance-to-portal.css
@@ -19,10 +19,14 @@
 	top: 0.15em;
 	left: 0.15em;
 	transform: skewY(-30deg);
+	-moz-transform: skewY(-30deg);
+	-webkit-transform: skewY(-30deg);
 }
 #portal-distance-bearing:after {
 	left: auto;
 	right: 0.15em;
 	transform: scaleX(-1) skewY(-30deg);
+	-moz-transform: scaleX(-1) skewY(-30deg);
+	-webkit-transform: scaleX(-1) skewY(-30deg);
 }
 
diff --git a/plugins/distance-to-portal.user.js b/plugins/distance-to-portal.user.js
index cc00d65a..a32b4274 100644
--- a/plugins/distance-to-portal.user.js
+++ b/plugins/distance-to-portal.user.js
@@ -22,9 +22,24 @@
 // use own namespace for plugin
 window.plugin.distanceToPortal = function() {};
 
-window.plugin.distanceToPortal.addDistance = function(info) {
+window.plugin.distanceToPortal.addDistance = function() {
+  var div = $('<div>')
+    .attr({
+      id: 'portal-distance',
+      title: 'Double-click to set/change current location',
+    })
+    .on('dblclick', window.plugin.distanceToPortal.setLocation);
 
-  var ll = info.portal.getLatLng();
+  $('#resodetails').after(div);
+
+  window.plugin.distanceToPortal.updateDistance();
+};
+
+window.plugin.distanceToPortal.updateDistance = function() {
+  if(!(selectedPortal && portals[selectedPortal])) return;
+  var portal = portals[selectedPortal];
+
+  var ll = portal.getLatLng();
 
   var text;
 
@@ -42,22 +57,19 @@ window.plugin.distanceToPortal.addDistance = function(info) {
     var bearing = window.plugin.distanceToPortal.currentLoc.bearingTo(ll);
     var bearingWord = window.plugin.distanceToPortal.currentLoc.bearingWordTo(ll);
 
-    text = 'Distance: '+dist+' <span id="portal-distance-bearing" style="transform: rotate('+bearing+'deg)"></span> '
-      + zeroPad(bearing, 3) + '° ' + bearingWord;
+    $('#portal-distance')
+      .text('Distance: ' + dist + ' ')
+      .append($('<span>')
+        .attr('id', 'portal-distance-bearing')
+        .css({
+          'transform': 'rotate('+bearing+'deg)',
+          '-moz-transform': 'rotate('+bearing+'deg)',
+          '-webkit-transform': 'rotate('+bearing+'deg)',
+        }))
+      .append(document.createTextNode(' ' + zeroPad(bearing, 3) + '° ' + bearingWord));
   } else {
-    text = 'Location not set';
+    $('#portal-distance').text('Location not set');
   }
-
-  var div = $('<div>')
-    .attr({
-      id: 'portal-distance',
-      title: 'Double-click to set/change current location',
-    })
-    .html(text)
-    .on('dblclick', window.plugin.distanceToPortal.setLocation);
-
-  $('#resodetails').after(div);
-
 };
 
 
@@ -90,8 +102,7 @@ window.plugin.distanceToPortal.setLocation = function() {
 
     localStorage['plugin-distance-to-portal'] = JSON.stringify({lat:window.plugin.distanceToPortal.currentLoc.lat, lng:window.plugin.distanceToPortal.currentLoc.lng});
 
-    // bit nasty, but easiest way of refreshing the distance after marker is moved
-    if (selectedPortal) renderPortalDetails(selectedPortal);
+    if (selectedPortal) window.plugin.distanceToPortal.updateDistance();
   });
 
   map.addLayer(window.plugin.distanceToPortal.currentLocMarker);