]> git.openstreetmap.org Git - rails.git/commitdiff
Merge tooltip fixes/tests
authorAnton Khorev <tony29@yandex.ru>
Mon, 12 Sep 2022 22:52:23 +0000 (01:52 +0300)
committerAnton Khorev <tony29@yandex.ru>
Mon, 12 Sep 2022 22:52:23 +0000 (01:52 +0300)
1  2 
app/assets/javascripts/leaflet.layers.js
test/system/site_test.rb

index faca07e3c18d0297b686160e2fd139a5a64c6605,2633d103794d6e6af8396545b49f7568be3900ae..d762696a18e556b4ffa5fb2548733ae9bd512251
@@@ -171,8 -145,7 +145,7 @@@ L.OSM.layers = function (options) 
            }
  
            $(item).attr("class", disabled ? "disabled" : "");
-           // item.attr("data-bs-original-title", disabled ? // has additional bug when zooming out from enabled state
-           item.attr("data-original-title", disabled ?
 -          item.attr("data-bs-original-title", disabled ?
++          item.attr("data-bs-original-title", disabled ? // has additional bug when zooming out from enabled state
              I18n.t("javascripts.site.map_" + name + "_zoom_in_tooltip") : "");
          });
        };
index 6dd8550d1795ba1132bc70d9198ae4277b9f86e6,a08f7f344cd55672a935b85fc7b5af00e53f4f93..2809feb93bc3a2f16d4566109ce4d7c713dbfcb9
@@@ -38,17 -38,34 +38,28 @@@ class SiteTest < ApplicationSystemTestC
      tooltip.assert_text "not available"
    end
  
 -  test "tooltip shows for query button when zoomed out" do
 -    visit "/#map=10/0/0"
+   test "tooltip shows for query button when zoomed in" do
+     visit "/#map=14/0/0"
+     assert_no_selector ".tooltip"
+     button = find ".control-query .control-button"
+     button.hover
+     tooltip = find ".tooltip"
+     tooltip.assert_text "Query features"
+     tooltip.assert_no_text "Zoom in"
+   end
-       visit "/#map=1/1/1"
 +  [
 +    "#edit_tab",
 +    ".control-note .control-button",
 +    ".control-query .control-button"
 +  ].each do |selector|
 +    test "tooltips on low zoom levels for disabled control '#{selector}'" do
++      visit "/#map=10/0/0"
  
 -    assert_no_selector ".tooltip"
 -    button = find ".control-query .control-button"
 -    button.hover
 -    tooltip = find ".tooltip"
 -    tooltip.assert_text "Zoom in to query features"
 -  end
 -
 -  test "tooltip shows for edit button when zoomed out" do
 -    visit "/#map=11/0/0"
 -
 -    assert_no_selector ".tooltip"
 -    button = find "#edit_tab"
 -    button.hover
 -    tooltip = find ".tooltip"
 -    tooltip.assert_text "Zoom in to edit the map"
 +      assert_no_selector ".tooltip"
 +      find(selector).hover
 +      assert_selector ".tooltip", :text => "Zoom in"
 +    end
    end
  end