Measure Tool Weirdness - Githubissues

asascience / ott-coops

Project Tracking for OTT Co-Ops

1 stars 0 forks source link

Measure Tool Weirdness #419

Closed jamesdoyle21 closed 2 years ago

jamesdoyle21 commented 2 years ago

Few things I noticed while testing the measure tool:

It took me a long time to figure out how to create a line with multiple vertices (and not sure it was the way we intended)
Opening the GIS Tool allowed me to create a second line to measure which did allow for multiple vertices
Can the Esc key exit the tool like it does for the bounding box tool?
When zoomed out, the tooltip does not land on the drawn line
Double clicking shows nothing, then user needs to deselect and reselect tool to measure again
Capitalize the D in "Measure distance" tooltip

jamesdoyle21 commented 2 years ago

https://user-images.githubusercontent.com/82955395/161612241-b33958ea-642e-495e-95a3-d4075715e269.mp4

jamesdoyle21 commented 2 years ago

Steps to reproduce the bug allowing for a second (and third) line to be drawn:

Draw line with measure tool
Open a drawer from any tool
Draw a second line (which allows for multiple vertices)

kvmartin001 commented 2 years ago

Yeah when Measure is activated, I have a limiter on the drawing functionality that says "only allow one line with two vertices". It looks like the limiter is breaking sometimes.

The drawing session is getting re-initialized when you open and close another tool, and when that happens, the limiter doesn't get applied so you get the normal line drawing tool.

The popup is supposed to appear at the midpoint of the most recently active feature, so that would be the last one you drew or the last one you selected. (the limiter is supposed to make multiple features impossible)

No idea why the tooltip coordinates are wrong though.

kvmartin001 commented 2 years ago

@jamesdoyle21 the tooltip position is actually "correct" because it's following the geodesic line between the points.

But that's still problematic, so I'm looking for a way around this.

Our map (Mercator)

Google Earth 3D

jamesdoyle21 commented 2 years ago

Are we intentionally not allowing for a line with multiple vertices? Is it problematic? I can see it being useful when a user is mapping out a route

mpiannucci commented 2 years ago

For now i said to simplify it to start because there was a lot of weirdness with multiple vertices

jamesdoyle21 commented 2 years ago

Ok, got it. I'll close it and we can think about adding functionality in the future as needed