Skip to content

Add reverse route button - #462

Merged
karussell merged 10 commits into
graphhopper:masterfrom
ratrun:reverse_route
Aug 22, 2026
Merged

Add reverse route button#462
karussell merged 10 commits into
graphhopper:masterfrom
ratrun:reverse_route

Conversation

@ratrun

@ratrun ratrun commented Aug 21, 2026

Copy link
Copy Markdown
Contributor

I added the possibility to reverse the route right to the settings button:
grafik
The change was created with the help of AI.

@karussell

Copy link
Copy Markdown
Member

Thanks!

I do not like this position for a feature like this as it is already very crowded at this position.

What about the context menu of the map? (we could remove "center map" while we are at it).

@ratrun

ratrun commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

What about the context menu of the map? (we could remove "center map" while we are at it).

I fear that nobody will expect this functionality there. It does not seem to be the right position. I checked how Google and komoot do this. The Google approach does not really work for us because we support via points. Komoot uses a similar position as compared to my PR. But they have a dropdown list for the available profiles, which seems a better concept to me.

The problem is that I don't find another logical position.

@ratrun

ratrun commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

Maybe here where I put the "x"?

grafik

@karussell

Copy link
Copy Markdown
Member

Aren't there other route-modifying functions that could be equally added like "create round trip"?

IMO it is rather easy to add new functions and distract the average user and we should say 99% of the time "no" to new buttons :)

@ratrun

ratrun commented Aug 22, 2026

Copy link
Copy Markdown
Contributor Author

I just implemented your suggestion and also removed the center map functionality. Please check.

@karussell

Copy link
Copy Markdown
Member

Thanks, looks good. Two nitpicks: the formatting seems to be off (just do npm run format)

And can you also update the translation spreadsheet directly?

@ratrun

ratrun commented Aug 22, 2026

Copy link
Copy Markdown
Contributor Author

I ran npm run format and updated the spreadsheet.

@karussell
karussell merged commit f0b5bb2 into graphhopper:master Aug 22, 2026
@karussell

Copy link
Copy Markdown
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants