Switch back to links for sharing

app/assets/javascripts/leaflet.share.js

@@ -29,39 +29,24 @@ L.OSM.share = function (options) {
         $('<h4>')
           .text(I18n.t('javascripts.share.title')));
 
-    var $share_link = $('<section>')
+    var $linkSection = $('<section>')
+      .attr('class', 'share-link')
       .appendTo($ui);
 
-    var $title = $('<h4>')
+    $('<h4>')
       .text(I18n.t('javascripts.share.link'))
-      .appendTo($share_link);
+      .appendTo($linkSection);
 
-    var $input = $('<input />')
+    var $shortLink, $longLink;
-      .attr('type', 'text')
-      .on('click', select)
-      .appendTo($share_link);
 
-    var $list = $('<ul>')
+    $('<ul>')
-      .appendTo($share_link);
+      .appendTo($linkSection)
-
+      .append($('<li>')
-    var $short_option = $('<li>')
+        .append($longLink = $('<a>')
-      .appendTo($list);
+          .text(I18n.t('javascripts.share.long_link'))))
-
+      .append($('<li>')
-    var $short_url_label = $('<label></label>')
+        .append($shortLink = $('<a>')
-      .attr('for', 'short_url')
+          .text(I18n.t('javascripts.share.short_link'))));
-      .appendTo($short_option);
-
-    var $short_url_input = $('<input />')
-      .attr('id', 'short_url')
-      .attr('type', 'checkbox')
-      .prop('checked', 'checked')
-      .appendTo($short_url_label)
-      .bind('change', function() {
-          options.short = $(this).prop('checked');
-          update();
-      });
-
-    $short_url_label.append(I18n.t('javascripts.share.short_url'));
 
     map.on('moveend layeradd layerremove', update);
 
@@ -71,13 +56,12 @@ L.OSM.share = function (options) {
       e.stopPropagation();
       e.preventDefault();
       options.sidebar.togglePane($ui);
-      $input.select();
+      update();
     }
 
     function update() {
-      $input.val(
+      $shortLink.attr('href', options.getShortUrl(map));
-          options.short ? options.getShortUrl(map) : options.getUrl(map)
+      $longLink.attr('href', options.getUrl(map));
-      );
     }
 
     function select() {

app/assets/stylesheets/common.css.scss

@@ -624,6 +624,15 @@ a.donate {
   }
 }
 
+.share-ui {
+  .share-link {
+    li {
+      display: inline-block;
+      width: 50%;
+    }
+  }
+}
+
 .site-index .leaflet-top,
 .site-export .leaflet-top {
   top: $lineheight/2 !important;

config/locales/en.yml

@@ -2042,7 +2042,8 @@ en:
       title: "Share"
       cancel: "Cancel"
       link: "Link"
-      short_url: "Short URL"
+      short_link: "Short Link"
+      long_link: "Long Link"
     key:
       title: "Map Key"
       tooltip: "Map Key"