<p>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): <a href="http://www.openstreetmap.org/way/150897569">http://www.openstreetmap.org/way/150897569</a> and here's a node that's part of a relation: <a href="http://www.openstreetmap.org/node/1813224670">http://www.openstreetmap.org/node/1813224670</a></p>
<p>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.</p>
<p>Note that <strong>nodes</strong> don't contain other objects, so there is no fix needed for the node page.</p>
<p>For <strong>relations</strong>, I can't decide if I agree that parent relations should be shown before children. Maybe...?</p>
<p>I'm not convinced by <a href="https://github.com/mboeringa" class="user-mention">@mboeringa</a>'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 <em>different</em> object than the one they chose, in a minority of cases. That kind of inconsistency can be a usability problem that leads to mistakes.</p>
<p>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...</p>
<pre><code>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>
<ul>
@@ -24,12 +31,5 @@
<% end %>
</ul>
<% 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 %>
</div>
<% end %>
</code></pre>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/openstreetmap/openstreetmap-website/issues/794#issuecomment-57069428">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/1419053__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyNzQ4MDQyMiwiZGF0YSI6eyJpZCI6MzgyMDQxMDh9fQ==--59b9897fdbc32703be3978dc3cc365c2fa7ac07e.gif" width="1" /></p>
<script type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","description":"View this Issue on GitHub","action":{"@type":"ViewAction","url":"https://github.com/openstreetmap/openstreetmap-website/issues/794#issuecomment-57069428","name":"View Issue"}}</script>