[openstreetmap-website] Relation membership should be on top of detail pages (#794)

danstowell notifications at github.com
Sat Sep 27 23:40:22 UTC 2014

It's always helpful to link to an example. Here's a way that's part of a relation (NOT a multipolygon in this case): http://www.openstreetmap.org/way/150897569 and here's a node that's part of a relation: http://www.openstreetmap.org/node/1813224670

I like the argument made in the main suggestion. It resonates with my experience - very often we see a way with dozens of anonymous nodes, and those anonymous nodes are rarely the info we want to get to first.

Note that **nodes** don't contain other objects, so there is no fix needed for the node page.

For **relations**, I can't decide if I agree that parent relations should be shown before children. Maybe...?

I'm not convinced by @mboeringa's added suggestion. I'm not saying it's a bad suggestion, but there's a risk that users would get confused about the tagging if the system shows them info from a _different_ object than the one they chose, in a minority of cases. That kind of inconsistency can be a usability problem that leads to mistakes.

So all in all, it's really the core-of-the-core suggestion that I'd advocate, and that is achieved by the following very simple tweak...

diff --git a/app/views/browse/_way.html.erb b/app/views/browse/_way.html.erb
index aff405d..8ac6a30 100644
--- a/app/views/browse/_way.html.erb
+++ b/app/views/browse/_way.html.erb
@@ -10,6 +10,13 @@
   <div class='browse-section browse-way'>
     <%= render :partial => "common_details", :object => way %>
+    <% unless way.containing_relation_members.empty? %>
+      <h4><%= t'browse.part_of' %></h4>
+      <ul>
+        <%= render :partial => "containing_relation", :collection => way.containing_relation_members.uniq %>
+      </ul>
+    <% end %>
     <% unless way.way_nodes.empty? %>
       <h4><%= t'browse.way.nodes' %></h4>
@@ -24,12 +31,5 @@
         <% end %>
     <% end %>
-    <% unless way.containing_relation_members.empty? %>
-      <h4><%= t'browse.part_of' %></h4>
-      <ul>
-        <%= render :partial => "containing_relation", :collection => way.containing_relation_members.uniq %>
-      </ul>
-    <% end %>
 <% end %>


Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20140927/fc536cc2/attachment.html>

More information about the rails-dev mailing list