From 04a6c41be94cea2175e49aa290bf204e69c41a5c Mon Sep 17 00:00:00 2001
From: Tom Hughes <tom@compton.nu>
Date: Mon, 15 Nov 2010 00:44:17 +0000
Subject: [PATCH] Show a popup listing available editors when hovering over the
 edit tab

---
 app/controllers/site_controller.rb |  2 +-
 app/helpers/application_helper.rb  |  4 +-
 app/views/layouts/site.html.erb    | 13 ++++++
 app/views/site/index.html.erb      |  2 +
 config/locales/en.yml              |  1 +
 public/javascripts/menu.js         | 62 +++++++++++++++++++++++++++++
 public/javascripts/site.js         | 64 ++++++++++++++++++++++++------
 public/stylesheets/common.css      | 21 ++++++++++
 8 files changed, 154 insertions(+), 15 deletions(-)
 create mode 100644 public/javascripts/menu.js

diff --git a/app/controllers/site_controller.rb b/app/controllers/site_controller.rb
index e6d09f18d..5e77b6691 100644
--- a/app/controllers/site_controller.rb
+++ b/app/controllers/site_controller.rb
@@ -32,7 +32,7 @@ class SiteController < ApplicationController
   end
 
   def edit
-    editor = @user.preferred_editor || DEFAULT_EDITOR
+    editor = params[:editor] || @user.preferred_editor || DEFAULT_EDITOR
 
     if editor == "josm"
       render :action => :index
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index ca26cfd48..c24e7ff5b 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -120,7 +120,9 @@ module ApplicationHelper
   end
 
   def preferred_editor
-    if @user and @user.preferred_editor
+    if params[:editor]
+      params[:editor]
+    elsif @user and @user.preferred_editor
       @user.preferred_editor
     else
       DEFAULT_EDITOR
diff --git a/app/views/layouts/site.html.erb b/app/views/layouts/site.html.erb
index b514fef99..f10b08949 100644
--- a/app/views/layouts/site.html.erb
+++ b/app/views/layouts/site.html.erb
@@ -5,6 +5,7 @@
     <%= javascript_strings %>
     <%= javascript_include_tag 'prototype' %>
     <%= javascript_include_tag 'site' %>
+    <%= javascript_include_tag 'menu' %>
     <!--[if lt IE 7]><%= javascript_include_tag 'pngfix' %><![endif]--> <!-- thanks, microsoft! -->
     <%= stylesheet_link_tag 'common' %>
     <!--[if IE]><%= stylesheet_link_tag 'large', :media => "screen" %><![endif]--> <!-- IE is totally broken with CSS media queries -->
@@ -73,6 +74,18 @@
       </ul>
     </div>
 
+    <div id="editmenu">
+      <ul>
+        <% Editors::ALL_EDITORS.each do |editor| %>
+          <li><%= link_to t('layouts.edit_with', :editor => t('user.account.editor.' + editor)), {:controller => 'site', :action => 'edit', :editor => editor}, {:id => editor + 'anchor'} %></li>
+        <% end %>
+      </ul>
+    </div>
+
+    <script type="text/javascript">
+      createMenu("editanchor", "editmenu", 1000);
+    </script>
+
     <div id="left">
 
       <div id="logo">
diff --git a/app/views/site/index.html.erb b/app/views/site/index.html.erb
index f9158463a..bc7ca3394 100644
--- a/app/views/site/index.html.erb
+++ b/app/views/site/index.html.erb
@@ -279,6 +279,8 @@ end
   }
 
   function installEditHandler() {
+    $("josmanchor").onclick =josmEditHandler;
+
     <% if preferred_editor == "josm" %>
       $("editanchor").onclick =josmEditHandler;
 
diff --git a/config/locales/en.yml b/config/locales/en.yml
index cfaa07bf1..2b2694a11 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -912,6 +912,7 @@ en:
     gps_traces_tooltip: Manage GPS traces
     user_diaries: User Diaries
     user_diaries_tooltip: View user diaries
+    edit_with: Edit with {{editor}}
     tag_line: The Free Wiki World Map
     intro_1: "OpenStreetMap is a free editable map of the whole world. It is made by people like you."
     intro_2: "OpenStreetMap allows you to view, edit and use geographical data in a collaborative way from anywhere on Earth."
diff --git a/public/javascripts/menu.js b/public/javascripts/menu.js
new file mode 100644
index 000000000..7a0cbf419
--- /dev/null
+++ b/public/javascripts/menu.js
@@ -0,0 +1,62 @@
+/*
+ * Open a menu.
+ */
+function openMenu(anchor, menu) {
+  menu.style.display = "block";
+
+  menu.clonePosition(anchor, {
+    setLeft: true, setTop: true, setWidth: false, setHeight: false,
+    offsetLeft: 0, offsetTop: anchor.getHeight()
+  });
+}
+
+/*
+ * Callback called when the mouse enters a menu anchor.
+ */
+function enterMenuAnchor(event, anchor, menu, delay) {
+  clearTimeout(menu.timer);
+
+  if (delay > 0) {
+    menu.timer = setTimeout(function () { openMenu(anchor, menu) }, delay);
+  } else {
+    openMenu(event, menu);
+  }
+}
+
+/*
+ * Callback called when the mouse leaves a menu anchor.
+ */
+function leaveMenuAnchor(event, anchor, menu) {
+  var to = event.relatedTarget || event.toElement;
+
+  if (to != menu && !to.descendantOf(menu)) {
+    menu.style.display = "none";
+  }
+
+  clearTimeout(menu.timer);
+}
+
+/*
+ * Callback called when the mouse leaves a menu.
+ */
+function leaveMenu(event, anchor, menu) {
+  var to = event.relatedTarget || event.toElement;
+
+  if (to != anchor && !to.descendantOf(menu)) {
+    menu.style.display = "none";
+  }
+
+  clearTimeout(menu.timer);
+}
+
+/*
+ * Setup a menu, triggered by hovering over an anchor for a given time.
+ */
+function createMenu(anchorid, menuid, delay) {
+  var anchor = $(anchorid);
+  var menu = $(menuid);
+
+  anchor.onmouseover = function (event) { enterMenuAnchor(anchor, anchor, menu, delay) };
+  anchor.onmouseout = function (event) { leaveMenuAnchor(event, anchor, menu) };
+  menu.onmouseout = function (event) { leaveMenu(event, anchor, menu) };
+}
diff --git a/public/javascripts/site.js b/public/javascripts/site.js
index 0e2c5aeb7..b11a2e012 100644
--- a/public/javascripts/site.js
+++ b/public/javascripts/site.js
@@ -1,4 +1,3 @@
-
 /*
  * Called as the user scrolls/zooms around to aniplate hrefs of the
  * view tab and various other links
@@ -69,15 +68,54 @@ function updatelinks(lon,lat,zoom,layers,minlon,minlat,maxlon,maxlat,objtype,obj
     }
   }
 
+  node = $("potlatchanchor");
+  if (node) {
+    var args = new Object();
+    args.editor = "potlatch";
+    args.lat = lat;
+    args.lon = lon;
+    args.zoom = zoom;
+    if (objtype && objid) {
+      args[objtype] = objid;
+    }
+    node.href = setArgs("/edit", args);
+  }
+
+  node = $("potlatch2anchor");
+  if (node) {
+    var args = new Object();
+    args.editor = "potlatch2";
+    args.lat = lat;
+    args.lon = lon;
+    args.zoom = zoom;
+    if (objtype && objid) {
+      args[objtype] = objid;
+    }
+    node.href = setArgs("/edit", args);
+  }
+
+  node = $("josmanchor");
+  if (node) {
+    var args = new Object();
+    args.editor = "josm";
+    args.lat = lat;
+    args.lon = lon;
+    args.zoom = zoom;
+    if (objtype && objid) {
+      args[objtype] = objid;
+    }
+    node.href = setArgs("/edit", args);
+  }
+
   node = $("historyanchor");
   if (node) {
     if (zoom >= 11) {
       var args = new Object();
       //set bbox param from 'extents' object
       if (typeof minlon == "number" &&
-	  typeof minlat == "number" &&
-	  typeof maxlon == "number" &&
-	  typeof maxlat == "number") {
+          typeof minlat == "number" &&
+          typeof maxlon == "number" &&
+          typeof maxlat == "number") {
 
         minlon = Math.round(minlon * decimals) / decimals;
         minlat = Math.round(minlat * decimals) / decimals;
@@ -225,24 +263,24 @@ function makeShortCode(lat, lon, zoom) {
     // z18 so we don't need to care for now.
     var c1 = 0, c2 = 0;
     for (var i = 31; i > 16; --i) {
-	c1 = (c1 << 1) | ((x >> i) & 1);
-	c1 = (c1 << 1) | ((y >> i) & 1);
+        c1 = (c1 << 1) | ((x >> i) & 1);
+        c1 = (c1 << 1) | ((y >> i) & 1);
     }
     for (var i = 16; i > 1; --i) {
-	c2 = (c2 << 1) | ((x >> i) & 1);
-	c2 = (c2 << 1) | ((y >> i) & 1);
+        c2 = (c2 << 1) | ((x >> i) & 1);
+        c2 = (c2 << 1) | ((y >> i) & 1);
     }
     var str = "";
     for (var i = 0; i < Math.ceil((zoom + 8) / 3.0) && i < 5; ++i) {
-	digit = (c1 >> (24 - 6 * i)) & 0x3f;
-	str += char_array.charAt(digit);
+        digit = (c1 >> (24 - 6 * i)) & 0x3f;
+        str += char_array.charAt(digit);
     }
     for (var i = 5; i < Math.ceil((zoom + 8) / 3.0); ++i) {
-	digit = (c2 >> (24 - 6 * (i - 5))) & 0x3f;
-	str += char_array.charAt(digit);
+        digit = (c2 >> (24 - 6 * (i - 5))) & 0x3f;
+        str += char_array.charAt(digit);
     }
     for (var i = 0; i < ((zoom + 8) % 3); ++i) {
-	str += "-";
+        str += "-";
     }
     return str;
 }
diff --git a/public/stylesheets/common.css b/public/stylesheets/common.css
index 42f28612e..9535caacd 100644
--- a/public/stylesheets/common.css
+++ b/public/stylesheets/common.css
@@ -344,6 +344,27 @@ hr {
   text-align: right;
 }
 
+/* Rules for edit menu */
+
+#editmenu {
+  display: none;
+  z-index: 10000;
+  position: absolute;
+  background-color: #ffffff;
+  border: 1px solid black;
+}
+
+#editmenu ul {
+  margin-top: 10px;
+  margin-bottom: 10px;
+  padding-left: 10px;
+  padding-right: 10px;
+}
+
+#editmenu li {
+  list-style-type: none;
+}
+
 /* Rules for attribution text under the main map shown on printouts */
 
 #attribution {
-- 
2.43.2