MushroomObserver / mushroom-observer

A website for sharing observations of mushrooms.
https://mushroomobserver.org
MIT License
77 stars 25 forks source link

Longitude lost focus #2365

Open JoeCohen opened 1 week ago

JoeCohen commented 1 week ago

Focus jumps from Longitude to Localities each time you change anything in Longitude. Example:

nimmolo commented 1 week ago

Yeah, i've experienced this. It's a little mind boggling to fix. What may need to happen is making something like the "keep button" more persistently available, maybe a "lock" icon on the Locality input.

What's the actual use case, though? (Not saying there isn't one, just that i couldn't think of one...) Why would people be trying to tweak the lat/lng numbers instead of clicking on the map? If people have a lat/lng they're trying to enter manually, it will stop interfering if they "Clear location", and maybe the help text should say this. But the easiest way to adjust a point is by clicking a different place on the map.

It is tricky differentiating between someone who wants to move the point of the obs on the map, and thereby potentially change the encompassing Locality (which we are doing everything to facilitate), and someone who is futzing with their lat/lng. I'll try to think of something. Maybe we can differentiate between a lat/lng input change originating in the map interface, and one originating in the text input. It's not obvious how, though.

JoeCohen commented 1 week ago

@nimmolo: I noticed it because I needed to both add a lat/lng and correct the Location for one of my Observations. I forget the exact sequence of events an keyboard presses. It was a struggle to get things exactly right. But if it doesn't come up that much, it's a lower priority. We all have bigger fish to fry.

nimmolo commented 1 week ago

Ok. I'm sure others will encounter it too, but for the record the easier path when you're doing that is to hit the "Clear location" button, add the lat/lngs, and then pick a locality.

As long as one lat/lng field is blank, it will leave you alone until you fill both. You have to be quick with the second one, though, because as soon as it thinks it has a viable lat/lng pair of any number of digits, it starts looking for a Locality after a one-second delay.

We could just try to increase the Location-search delay, huh? What do you think

JoeCohen commented 1 week ago

I don't like longer delay. Maybe add something to the instructions.