Okay, I'm a bit blocked from doing testing with large changesets that break the map, since I have a limited extract of the database around.<div><br></div><div>I get that the other information is important, but it's important to advanced users, and people who don't immediately grasp what a 'node' is are blindsided by technical information - and presenting a map with the thing highlighted makes it clearer. Hopefully we can please both camps.</div>
<div><br></div><div>I've experimented with more literate approaches to representing this information, and have that implemented in the changeset view - <a href="http://dl.dropbox.com/u/68059/Screenshots/k.png">http://dl.dropbox.com/u/68059/Screenshots/k.png</a> . Doing the same for node views - which share code with node history views - is a bit more difficult, and the whole endeavor is not easy to reconcile with translatable strings. So I think this is an incremental improvement that can have further pushes.</div>
<div><br></div><div>That's also the main reason why the 'current version' text is how it is - this page shares a partial with the history view. This might be resolvable - I'll take a look and anyone else who wants to hack on this, you can grab the changeset_view branch and jfdi.</div>
<div><br></div><div>Tom<br><br><div class="gmail_quote">On Mon, Feb 27, 2012 at 4:25 AM, Andy Allan <span dir="ltr"><<a href="mailto:gravitystorm@gmail.com">gravitystorm@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 27 February 2012 00:42, Tom Hughes <<a href="mailto:tom@compton.nu">tom@compton.nu</a>> wrote:<br>
> On 26/02/12 20:11, Tom MacWright wrote:<br>
><br>
>> This refactors changeset views, node and way pages. It needs to be fully<br>
>> tested with ways - so far my database import isn't accepting them, so I<br>
>> don't have any test data. For these pages, it moves and enlarges the map,<br>
>> changes the styling of the menu, introduces some more clickable&  visible<br>
>> navigation.<br>
>><br>
>><br>
>> ![changeset](<a href="http://dl.dropbox.com/u/68059/Screenshots/g.png" target="_blank">http://dl.dropbox.com/u/68059/Screenshots/g.png</a>)<br>
>><br>
>> You can merge this Pull Request by running:<br>
>><br>
>>   git pull <a href="https://github.com/tmcw/openstreetmap-website" target="_blank">https://github.com/tmcw/openstreetmap-website</a> changeset_view<br>
><br>
><br>
> I've added this to <a href="http://tomh.apis.dev.openstreetmap.org/" target="_blank">http://tomh.apis.dev.openstreetmap.org/</a> and I'd like to<br>
> know what people think.<br>
<br>
</div>* The version number could do with alignment<br>
* Actually, I wonder if "Version 4 (current version) (view history)"<br>
or similar would be better as the subtitle - I'm not sure why the<br>
fixed text "Current version (view history)" would be the second-level<br>
title of the page<br>
* Edited at / edited by could easily be combined into "edited by tcmw<br>
at whenever (nicely formatted)"<br>
* If there's some way of making the coordinates available, but not as<br>
the (fairly prominent) final thing, that would be good. I know<br>
sometimes they are needed but the 6 decimal places make the page a<br>
little ugly.<br>
<br>
Basically, the list of attributes shown is so clearly just a<br>
one-by-one list of attributes on the model, and I think it can be<br>
improved!<br>
<br>
As for the bigger map, I can only echo Sarah's point. The<br>
mis-behaviour of the map when loading large relations (i.e. starting<br>
at the full world, and only zooming to the right place after the<br>
massive geometry has been loaded and parsed) might need to be fixed<br>
first before increasing the map prominence.<br>
<br>
Cheers,<br>
Andy<br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
rails-dev mailing list<br>
<a href="mailto:rails-dev@openstreetmap.org">rails-dev@openstreetmap.org</a><br>
<a href="http://lists.openstreetmap.org/listinfo/rails-dev" target="_blank">http://lists.openstreetmap.org/listinfo/rails-dev</a><br>
</div></div></blockquote></div><br></div>