openstreetmap/iD

Add a parameter to support custom {tileSize} of WMTS backgrounds

Aperta

#8688 aperta il 13 set 2021

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (1449 fork)github user discovery
help wanted

Metriche repository

Star
 (3818 stelle)
Metriche merge PR
 (Merge medio 29g 14h) (82 PR mergiate in 30 g)

Descrizione

I see the current logic is 256 by default

https://github.com/openstreetmap/iD/blob/bc0c1895f7b240a35e1beb6b9220eb15d8c36a70/modules/renderer/background_source.js#L51

Of course for some exceptions there are few hacks:

https://github.com/openstreetmap/iD/blob/1ee45ee1f03f0fe4d452012c65ac6ff7649e229f/scripts/update_imagery.js#L94-L101

Wouldn't it be better to add a parameter for custom backgrounds to define the tileSize by themselves? If iD supports this, ELI will be able to add more WMTS sources.

Currently only JOSM can handle 512x512 automatically because it reads WMTSCapabilities directly.

A WMTS service cannot be opened in iD 2.20.1: https://geodata.md.gov/imap/rest/services/Imagery/MD_SixInchImagery/MapServer/WMTS/1.0.0/WMTSCapabilities.xml

Guida contributor

Direzione di ricerca
Aggiungi un parametro tileSize configurabile alla sorgente di sfondo e propaga alla gestione WMTS. Assicurati che il tileSize personalizzato venga utilizzato nella logica di caricamento dei tile invece del default 256.
Tech stack
javascript
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70