openstreetmap / openstreetmap-website

The Rails application that powers OpenStreetMap
https://www.openstreetmap.org/
GNU General Public License v2.0
2.17k stars 910 forks source link

Always show changeset element page links below headings #5213

Closed AntonKhorev closed 2 weeks ago

AntonKhorev commented 2 weeks ago

What I tried to achieve previously was to save space. If the page links fit to the right of the heading, let them be there.

But this works fine only with full sidebar reloads. With turbo frame reloads it's more noticeable when the links become wider, don't fit and move below the heading. Here I'm making them always be below.

Before:

image image

After:

image image

kcne commented 2 weeks ago

In my opinion this looks more coherent anyway. Thank you.

tomhughes commented 2 weeks ago

Looks good to me, thanks.