From: Tom Hughes <tom@compton.nu>
Date: Mon, 29 Jul 2013 21:49:59 +0000 (+0100)
Subject: Remove geolink class from add note button while adding notes
X-Git-Tag: live~4837
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/393651e8bde09adb38e3d3d377f9b5f6627ea429?hp=c93308dd5c41a921305399980cfc8f1ed890daf7;ds=sidebyside

Remove geolink class from add note button while adding notes

This was lost in the recent UI work but is important to stop changes
in map location causing the button to be reenabled.
---

diff --git a/app/assets/javascripts/index/notes.js.erb b/app/assets/javascripts/index/notes.js.erb
index d0f6cb388..012538e1d 100644
--- a/app/assets/javascripts/index/notes.js.erb
+++ b/app/assets/javascripts/index/notes.js.erb
@@ -191,7 +191,7 @@ function initializeNotes(map) {
       notes[feature.properties.id] = updateMarker(marker, feature);
       newNote = null;
 
-      addNoteButton.removeClass("disabled");
+      addNoteButton.removeClass("disabled").addClass("geolink");
     }
   }
 
@@ -226,7 +226,7 @@ function initializeNotes(map) {
 
     if (addNoteButton.hasClass("disabled")) return;
 
-    addNoteButton.addClass("disabled");
+    addNoteButton.removeClass("geolink").addClass("disabled");
 
     map.addLayer(noteLayer);
 
@@ -263,7 +263,7 @@ function initializeNotes(map) {
     newNote.addTo(noteLayer).bindPopup(popupContent[0], popupOptions()).openPopup();
 
     newNote.on("remove", function (e) {
-      addNoteButton.removeClass("disabled");
+      addNoteButton.removeClass("disabled").addClass("geolink");
     }).on("dragstart", function (e) {
       $(newNote).stopTime("removenote");
     }).on("dragend", function (e) {