deevroman left a comment (openstreetmap/openstreetmap-website#5924)

> Use scrollIntoView() to bring the selected list-group-item into view in the 
> sidebar so you can always see what you're hovering over in the map.

This is a tricky function. I use it in my script for osm.org to navigate 
between objects in a changesets, and sometimes it works unpredictably and moves 
the scroll to the wrong place (or rather, sometimes it doesn't move the scroll 
at all). Perhaps this is a problem with the current layout, but in any case, I 
would be careful in expectations of it.

> Sure it might jump around if you move your mouse all over the map but it 
> makes intuitive sense.

What is the point of doing this, then? Imagine, you are scrolling through 
changesets, and finally an interesting cluster of edits appears in the area you 
need, you move the cursor to them and the sidebar starts jumping. Then it turns 
out that you need to load more changesets, and you move the cursor to the 
sidebar again, enjoying its jumping.

Also, this is not the only solution. You can scroll with a delay. You can 
scroll by alt + click by bbox. You can display by hover title with meta 
information. You can show a popup. PR welcome :)

> This should be reverted.

Now, viewing changesets on osm.org is a clean slate, so almost any change 
doesn't make worse.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5924#issuecomment-2816719146
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/5924/c2816719...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to