Use jquery interface for enabling/disabling tooltips

diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js
index 8b8beb5b4a5164ca10a2c26652ef5662458b0024..24d50dcce623d204dfb12569d2d3ffb4696ae9c3 100644 (file)
--- a/app/assets/javascripts/application.js
+++ b/app/assets/javascripts/application.js
@@ -56,20 +56,15 @@ window.updateLinks = function (loc, zoom, layers, object) {
     link.href = href;
   });
 
+  // Disable the button group and also the buttons to avoid
+  // inconsistent behaviour when zooming
   var editDisabled = zoom < 13;
-  var editTab = $("#edit_tab");
-  editTab
-    // Disable the button group and also the buttons to avoid
-    // inconsistent behaviour when zooming
+  $("#edit_tab")
+    .tooltip({ placement: "bottom" })
+    .tooltip(editDisabled ? "enable" : "disable")
     .toggleClass("disabled", editDisabled)
     .find("a")
     .toggleClass("disabled", editDisabled);
-  var editTooltip = bootstrap.Tooltip.getOrCreateInstance(editTab[0], { placement: "bottom" });
-  if (editDisabled) {
-    editTooltip.enable();
-  } else {
-    editTooltip.disable();
-  }
 };
 
 window.maximiseMap = function () {

diff --git a/app/assets/javascripts/leaflet.layers.js b/app/assets/javascripts/leaflet.layers.js
index f372b86cfbe5819886b78726d9b87f9651fa1e5b..020876c470c71f134ab5e2543e29a73be8c7b7b8 100644 (file)
--- a/app/assets/javascripts/leaflet.layers.js
+++ b/app/assets/javascripts/leaflet.layers.js
@@ -100,11 +100,10 @@ L.OSM.layers = function (options) {
         var item = $("<li>")
           .appendTo(overlays);
 
-        var tooltip;
         if (name === "notes" || name === "data") {
-          item.attr("title", I18n.t("javascripts.site.map_" + name + "_zoom_in_tooltip"));
-          tooltip = new bootstrap.Tooltip(item[0]);
-          tooltip.disable();
+          item
+            .attr("title", I18n.t("javascripts.site.map_" + name + "_zoom_in_tooltip"))
+            .tooltip("disable");
         }
 
         var label = $("<label>")
@@ -148,14 +147,9 @@ L.OSM.layers = function (options) {
               .trigger("change");
           }
 
-          $(item).attr("class", disabled ? "disabled" : "");
-          if (tooltip) {
-            if (disabled) {
-              tooltip.enable();
-            } else {
-              tooltip.disable();
-            }
-          }
+          $(item)
+            .attr("class", disabled ? "disabled" : "")
+            .tooltip(disabled ? "enable" : "disable");
         });
       };
 

diff --git a/config/eslint.json b/config/eslint.json
index b54c8855f47806cc420b2a7259354786942cd70c..3b878d48a8cb813955fb399243ed01af1e892fef 100644 (file)
--- a/config/eslint.json
+++ b/config/eslint.json
@@ -7,7 +7,6 @@
     "eslint:recommended"
   ],
   "globals": {
-    "bootstrap": "readonly",
     "Cookies": "readonly",
     "I18n": "readonly",
     "L": "readonly",

diff --git a/test/system/site_test.rb b/test/system/site_test.rb
index 0ed755a0953d066b0381da4f95a70ab2b77e9b9d..5528bb79bb6a11ebb51a0104cc94b78dd9213e1e 100644 (file)
--- a/test/system/site_test.rb
+++ b/test/system/site_test.rb
@@ -70,7 +70,7 @@ class SiteTest < ApplicationSystemTestCase
       assert_no_selector ".tooltip", :text => "Zoom in"
       find("h1").hover # un-hover original element
 
-      visit "/#map=10/0/0"
+      visit "#map=10/0/0"
       find(selector).hover
       assert_selector ".tooltip", :text => "Zoom in"
     end