Open-EO / openeo-web-editor

An interactive and easy to use web-based editor for the OpenEO API.
https://editor.openeo.org
Apache License 2.0
12 stars 17 forks source link

Delay request for location #264

Closed soxofaan closed 1 year ago

soxofaan commented 2 years ago

The editor currently asks access to current location on first view (before log in): Screenshot from 2022-06-08 12-19-14

This feels a bit intrusive without obvious gain (location is only actually used when opening a map without obvious starting point)

Is it possible to delay the permission request to a later point, when the location is actually necessary?

m-mohr commented 2 years ago

I could delay it until after login, but it is annoying if I delay it until the map is actually shown, because:

  1. Then the map loads twice, you have a refit in the location and that may disturb you when working with the map already (location is not always available immediately).
  2. You get this request box everytime you open a new map, which is annoying (except you checked the box to remember the decision). So a one time request at the beginning might be better.
soxofaan commented 2 years ago

hmm, if that popup can not be delayed or only shown when necessary, then maybe this is a case where the cure (having to show the permission popup to everybody) is worse than the disease (not having an obvious map center for some edge cases)?

m-mohr commented 2 years ago

So for now (v0.11.2) it is only requested after the authentication screen and you can disable it for custom deployments via the requestUserLocation config option. Leaving this open for the future to potentially solve this better through a reactive connection with the mapping library or so.

m-mohr commented 1 year ago

Different idea: Add a button to grab the location instead of using it by default.

m-mohr commented 1 year ago

Implemented, available on editor.openeo.org, soon also .cloud