mjourdan left a comment (openstreetmap/openstreetmap-website#3123)

Thanks for the changes! It's great to see markers are in the input form, the 
reverse direction button is well placed, and the mode is finally separate from 
the routing engines.

Did a quick informal user review with a panel of 1 person, and what came up 
immediately:
- there is no clue of what the dropdown is for, what OSRM, GraphHopper, and 
Valhallah are
- pressing the go button during a calculation seemed to break the route that 
was beeing processed (could not reproduce today though)

The issues I see too:
- The close button is a bit too close the reverse directions button and the 
input form
- the matching locations are not suggested while typing, which likely results 
in routing beeing calculated for non-expected locations (this might deserve an 
issue on its own)
 
Ideally we could improve things further by:
- adding a short message to advertise that routing engines are used by the 
mapper community to ensure the points they add to the map are well connected.
- triggerring the calculation as soon as two locations are selected, a location 
is changed, the transport mode is changed, or the routing engine is changed
- removing the go button

I could update mockups to reflect those suggestions if that sounds sensible.

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

Message ID: 
<openstreetmap/openstreetmap-website/issues/3123/2726356...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to