Suggestion: on load, instead of showing the modal -- use device location about 1km.co.il HOT 12 CLOSED

Hi Oren, thanks for the idea!

Initially this was the behavior, but I was afraid of a UX issue this could raise; we need to build a minimal user trust before asking them for their location.
If a permission request will be shown before they saw an explanation why is it happening, it can bring up suspicious thoughts in their minds which I think we prefer to avoid.
The current method allows them to give their location out of their free will, by clicking a button that makes it clear why the permission alert shows up.

But I'm open to hear other thoughts!

from 1km.co.il.

I would if user has set location then 2. If not then 1

from 1km.co.il.

Hi Guy,

• For new users, I see your point.
• For returning users, I think that focusing on their location is better.

So a solution here might be "once the user asked to use location, remember that they did that, and in the future focus on load." A relatively simple way to implement this would be saving this decision on localStorage.

I think that removing the modal on load makes sense for everyone.

But no pressure. :) It's fine to drop this if no enthusiasm.

from 1km.co.il.

I think showing not showing the modal for returning users is a good idea.
It's a bit annoying that it pops up everytime

from 1km.co.il.

So a solution here might be "once the user asked to use location, remember that they did that, and in the future focus on load." A relatively simple way to implement this would be saving this decision on localStorage.

Maybe instead of storing the decision in localStorage, we could save the address itself. Then there won't be the problem of using the location api without user confirmation.

from 1km.co.il.

That makes sense.
Isn't the permission gets invalidated after a while and the permission alert shows up again?

from 1km.co.il.

@guytepper I don't think the permission is invalidated after a while. However, it could be revoked by the user (rare but happens).

So it seems like we want to save the user's last address/location?
I suggest to use the map's last location, as users can start with address or device location and then move the map. If we keep track of position changes (I guess when dispatching setMapPosition?), I think it'll cover all cases.

If someone else wants to take a shot at it I have no objection. :)

from 1km.co.il.

👋 Hey! I created a PR that saves the chosen user location (coordinates) on local storage - #48

from 1km.co.il.

@guytepper the permission is not invalidated once allowed.
Do you want the default location to be set to:

1. Users' current location
2. Users last selection (taken from localStorage)

?

from 1km.co.il.

@OrenShalev can you update the PR to follow @joeyede requirement (try to use users' last selected location, with fallback to users' current location, with fallback to modal) ?

from 1km.co.il.

Seems like #48 is merged and this bug is fixed, so closing. 👍

from 1km.co.il.

from 1km.co.il.