Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

When using Custom URLs for tiles, the background type menu refer to it as "Aerial (by ESRI)" #506

Closed
govvin opened this issue Jan 17, 2024 · 8 comments
Labels
bug Something isn't working PR welcome Needs someone to implement it

Comments

@govvin
Copy link

govvin commented Jan 17, 2024

When a custom URL is set as a tile source, the option for selecting a background type still refers to it as "Aerial (by ESRI)". Perhaps we should have a separate entry for default aerial layer, then something else for custom layer?

Not a show-stopper, but a consistent interface for any app is always nice to have. Thanks for the awesome work on SCEE!

How to Reproduce
Add a custom tile layer URL from Settings > SCEE settings > Data Management > Tile source for satellite/aerial images.

Expected Behavior
The menu option will show "custom URL" (or something similar) instead of "Aerial (by ESRI)

Does it happen in normal StreetComplete?
I don't use it, but as far as I can tell this option is only available in SCEE?

Versions affected
v56.0

Screenshot
Background type selection options, showing "Aerial (by ESRI), even if a custom URL has been set.

@govvin govvin added the bug Something isn't working label Jan 17, 2024
@Helium314 Helium314 added the PR welcome Needs someone to implement it label Jan 17, 2024
@Helium314
Copy link
Owner

It would be nice to have, but certainly is not priority.

Considering that I'm not sure whether (and if so, when) I'll re-implement aerial imagery access after switching to mapLibre (streetcomplete#3123), I will not work on this in forseeable future.

ravenfeld added a commit to ravenfeld/SCEE that referenced this issue Jul 2, 2024
ravenfeld added a commit to ravenfeld/SCEE that referenced this issue Jul 3, 2024
ravenfeld added a commit to ravenfeld/SCEE that referenced this issue Jul 4, 2024
ravenfeld added a commit to ravenfeld/SCEE that referenced this issue Jul 7, 2024
ravenfeld added a commit to ravenfeld/SCEE that referenced this issue Jul 8, 2024
@mnalis
Copy link
Collaborator

mnalis commented Jan 21, 2025

That particular menu entry is not there, but some things (split way, insert node etc) still use that R.string.background_type_aerial_esri = Aerial (by Esri)

Perhaps just calling it Aerial without further (by Esri) qualifications would suffice for now?

At least if/until we got possibility of multiple background maps (PR #564); when those would need to be reworked anyway

@Helium314
Copy link
Owner

Calling it "Aerial" might also be incorrect, as we don't know what users do here. I remember someone was using Carto tiles.

So only "correct" thing would be using something like "Custom" if it's not the default raster background.

@mnalis
Copy link
Collaborator

mnalis commented Jan 24, 2025

Calling it "Aerial" might also be incorrect, as we don't know what users do here

That is true, but I do not feel being extra precise there is of such huge importance in SCEE. Perhaps just calling it "Raster" would suffice in current case (as default "Map" is vector-based)?

@mnalis
Copy link
Collaborator

mnalis commented Jan 24, 2025

So only "correct" thing would be using something like "Custom" if it's not the default raster background.

If there is really the need to dynamically change the name of the button depending on what background is being selected, then perhaps using (trimming the TLD) domain name as a name of the button would work? e.g. with default raster URL it would be called arcgisonline, and if I change the URL to point somewhere on dgu.hr domain the button would say dgu.

But I don't really see amount of work needed to be done being proportional to the usefulness for current version of SCEE...

It will however make more sense for the effort if #564 (implementing multiple background URLs) gets merged, as then we'd need to distinguish and switch from raster#1 to raster#2.

@andrewharvey
Copy link

andrewharvey commented Feb 26, 2025

Strange in my SCEE, under Settings Display there is no Select Background type option, instead under Data Management there is "Tile source for satellite / aerial images". No mention of ESRI. Perhaps this issue is fixed already?

@govvin
Copy link
Author

govvin commented Feb 26, 2025

Issue already fixed.

@govvin govvin closed this as completed Feb 26, 2025
@mnalis
Copy link
Collaborator

mnalis commented Feb 26, 2025

No mention of ESRI. Perhaps this issue is fixed already?

Note that in some cases (e.g. It's at the slighly different position... other answer), there is still a button named Aerial (by Esri) that allow switching map view while moving the node.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working PR welcome Needs someone to implement it
Projects
None yet
Development

No branches or pull requests

4 participants