graphhopper / graphhopper/graphhopper-maps

Changing query point field adds extra history entry

Open
#281 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
168
Forks
63
Avg merge
1d 1h
Merged PRs (30d)
2

Description

When we calculate a route, then clear a field, and then enter another location/point to calculate a new route two history entries are created. One for the empty field and another for the new one. The first one is caused by the `InvalidatePoint` action. This means that when we click 'back' after the change the route disappears as one field is empty and we need to click 'back' another time to go back to the previous route.

To fix this we need to deal with the fact that this user interaction (changing the input field) triggers two state updates and we create a history entry for every state update. One possibility would be creating history entries explicitly instead of using our current indirect approach.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.