googlemaps / googlemaps/android-maps-compose

RFC: Remove experimental MapEffect composable

Aperta
#267 4 commenti 4 reazioni 0 assegnatari Vedi su GitHub
needs more info priority: p3 type: feature request
Lingua principale
Kotlin
Stelle
1.3k
Fork
181
Merge medio
2g 23h
PR unite (30g)
18

Descrizione

If you would like to recommend against this proposal, please comment below or react with 👎 .
-------------

MapEffect was added in #140 to expose the underlying GoogleMap Maps SDK object to enable extension and usages such as clustering with the utility library.

With Compose-native clustering being added by #258, we're hoping that MapEffect is no longer needed. Providing direct access to the map is potentially buggy as noted in [this comment](https://github.com/googlemaps/android-maps-compose/pull/140#pullrequestreview-1003063252).

It seems like MapEffect would still be necessary for other Android Maps Utils such as GeoJSON layer (#29) and KML layer (#129).

**Please comment below if you have other use cases that would require exposing the map object.**

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia esaminando l'introduzione di MapEffect in #140, il lavoro sul clustering nativo di Compose in #258 e i casi d'uso rimanenti di GeoJSON e KML in #29 e #129. Il lavoro sarà considerato completato quando sarà stato stabilito se questi casi d'uso necessitano ancora dell'accesso diretto alla mappa e la decisione risultante sarà stata documentata o implementata.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
kotlin
Ambito
mobile-dev
Tipo di issue
Refactoring
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
20/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.