diff --git a/app/assets/javascripts/index/query.js b/app/assets/javascripts/index/query.js
index 514d6dbb2..7d02760ce 100644
--- a/app/assets/javascripts/index/query.js
+++ b/app/assets/javascripts/index/query.js
@@ -228,12 +228,14 @@ OSM.Query = function (map) {
if (results.remark) {
$("
")
+ .addClass("query-result list-group-item")
.text(I18n.t("javascripts.query.error", { server: url, error: results.remark }))
.appendTo($ul);
}
if ($ul.find("li").length === 0) {
$("")
+ .addClass("query-result list-group-item")
.text(I18n.t("javascripts.query.nothing_found"))
.appendTo($ul);
}
@@ -242,6 +244,7 @@ OSM.Query = function (map) {
$section.find(".loader").stopTime("loading").hide();
$("")
+ .addClass("query-result list-group-item")
.text(I18n.t("javascripts.query." + status, { server: url, error: error }))
.appendTo($ul);
}
diff --git a/app/assets/javascripts/index/search.js b/app/assets/javascripts/index/search.js
index ba66f4782..9ed243466 100644
--- a/app/assets/javascripts/index/search.js
+++ b/app/assets/javascripts/index/search.js
@@ -45,9 +45,9 @@ OSM.Search = function (map) {
$("#sidebar_content")
.on("click", ".search_more a", clickSearchMore)
.on("click", ".search_results_entry a.set_position", clickSearchResult)
- .on("mouseover", "p.search_results_entry:has(a.set_position)", showSearchResult)
- .on("mouseout", "p.search_results_entry:has(a.set_position)", hideSearchResult)
- .on("mousedown", "p.search_results_entry:has(a.set_position)", function () {
+ .on("mouseover", "li.search_results_entry:has(a.set_position)", showSearchResult)
+ .on("mouseout", "li.search_results_entry:has(a.set_position)", hideSearchResult)
+ .on("mousedown", "li.search_results_entry:has(a.set_position)", function () {
var moved = false;
$(this).one("click", function (e) {
if (!moved && !$(e.target).is("a")) {
diff --git a/app/assets/stylesheets/common.scss b/app/assets/stylesheets/common.scss
index abae98392..5635e3f9d 100644
--- a/app/assets/stylesheets/common.scss
+++ b/app/assets/stylesheets/common.scss
@@ -939,15 +939,8 @@ header .search_forms,
/* Rules for search sidebar */
#sidebar .search_results_entry {
- ul {
- padding: 0;
- }
-
ul li {
- border-bottom: $keyline;
cursor: pointer;
- list-style-type: none;
- &:first-child { border-top: $keyline; }
&.selected { background: $list-highlight; }
}
diff --git a/app/views/geocoder/results.html.erb b/app/views/geocoder/results.html.erb
index deec4ed39..94f19d2fe 100644
--- a/app/views/geocoder/results.html.erb
+++ b/app/views/geocoder/results.html.erb
@@ -1,9 +1,13 @@
<% if @results.empty? %>
- <%= t ".no_results" %>
+
+ -
+ <%= t ".no_results" %>
+
+
<% else %>
-
+
<% if @more_params %>
diff --git a/test/controllers/geocoder_controller_test.rb b/test/controllers/geocoder_controller_test.rb
index f8a4eed19..0459eb294 100644
--- a/test/controllers/geocoder_controller_test.rb
+++ b/test/controllers/geocoder_controller_test.rb
@@ -489,11 +489,11 @@ class GeocoderControllerTest < ActionController::TestCase
assert_select "ul.results-list", 0
else
assert_select "ul.results-list", 1 do
- assert_select "p.search_results_entry", results.count
+ assert_select "li.search_results_entry", results.count
results.each do |result|
attrs = result.collect { |k, v| "[data-#{k}='#{v}']" }.join("")
- assert_select "p.search_results_entry a.set_position#{attrs}", result[:name]
+ assert_select "li.search_results_entry a.set_position#{attrs}", result[:name]
end
end
end