mapper icon indicating copy to clipboard operation
mapper copied to clipboard

Fluent drawing enhancement

Open ollesmaps opened this issue 7 years ago • 4 comments

Steps to reproduce

At this moment it is not possible to draw continuously. If you want to draw a line (or area) outside screen, you have to 1) finish line 2) pan map 3) begin a new line 4) merge lines.

Expected behaviour

It would be much better if you could let line be unfinished 1) pan map 2) continue line.

Note

This issue was already mentioned by @mlerjen at #534.

Configuration

Android App 0.7.0

ollesmaps avatar Oct 10 '17 15:10 ollesmaps

Sidenote: If you zoom out/ zoom in - you can continue drawing an unfinished line.

ollesmaps avatar Oct 10 '17 18:10 ollesmaps

As much as I understood from this issue and from #1175, #1195, we are talking about using the "Pan" tool transiently here.

@lpechacek This might be implemented by activating the PanTool as an override tool to the current tool, instead of as a replacement. I investigated a little bit, and found that we may need a little bit more API and state handling than we have at the moment. I can share the proof of concept.

dg0yt avatar Dec 23 '18 17:12 dg0yt

As much as I understood from this issue and from #1175, #1195, we are talking about using the "Pan" tool transiently here.

Ack.

@lpechacek This might be implemented by activating the PanTool as an override tool to the current tool, instead of as a replacement.

I noticed the concept of override tool in the sources but haven't studied it yet.

I investigated a little bit, and found that we may need a little bit more API and state handling than we have at the moment. I can share the proof of concept.

That was my conclusion too. Draw path tool state needs to be restored after Pan tool operation is finished. A PoC will surely give a welcome clue. I also won't feel offended if you take over the issue. :)

lpechacek avatar Dec 23 '18 21:12 lpechacek

Relates to #1851 Pan map window automatically when editing to the edge.

ollesmaps avatar Jan 18 '21 11:01 ollesmaps