search

mapbox / search-playground

Geocoding Debugging Tool
https://mapbox.com/search-playground
14 stars 4 forks source link

Bounding Box Tool Doesn't Allow Drawing #16

Open yuletide opened 6 years ago

yuletide commented 6 years ago

Proximity toggle and bbox toggle should default to off and then fill with current map center or extent when turned on, and when the icon is clicked (as in our internal version). Much easier than drawing a box or clicking a point, just use the current map view. Also the drawing tool is very buggy.

Context

gif

ingalls commented 6 years ago

cc/ @mapbox/search

boblannon commented 6 years ago

also seeing this. i agree that it'd be fine to just populate using the extent. that's what we used to do, iirc

karenell commented 6 years ago

It works, but the interaction doesn't match the intstructions. Rather than a click and drag, it's:

  1. single click (down->up)
  2. move the mouse (not clicked) and an orange box appears
  3. click again to set the opposite corner of the bbox.

We could change the instructions to be "click once to start drawing your box, click again to set your box size", or do UI magic to make the click-drag isolated to this action immediately after you click the bbox icon.

budda commented 6 years ago

Without the instructions from @karenell i didn't find the BBox filter to be very clear at all how to draw a bounding rectangle. I just kept panning the map around.

ingalls commented 6 years ago

@Fa7C0n can you spend a few minutes today improving the instructions based on the above feedback?