search

Zverik / every_door

A dedicated app for collecting thousands of POI for OpenStreetMap
https://every-door.app
ISC License
404 stars 34 forks source link

[Bug] Undo button sometimes removes next to last line #758

Open pkoby opened 2 months ago

pkoby commented 2 months ago

For some reason, sometimes after adding a line in Draw Mode, a hit of the Undo button removes the line drawn before the last line. I cannot reproduce it consistently, so I am unable to pin it to zoom level, switching backgrounds, or how quickly I hit undo.

Like my other issue, this could be allieved by a redo option, but I think this is actually a bug.

EDIT: I think it might be due to caching lag/RAM issues. I have a Moto X4 with 3GB of RAM that is often insufficient. I reproduced the issue by zooming out, thus loading more map tiles, then drawing long lines and quickly undoing. It took half a second for the lines to settle after drawing, during which process an undo action would affect earlier lines.

EDIT 2: Okay, what really does it is loading new day for an area, then drawing lines. It seems to lag until I close and reopen the app.

Zverik commented 22 hours ago

The lag is weird: it does indeed take a few milliseconds to save a line, but to undo in that time... One must have really good reflexes!