googlemaps / googlemaps/android-maps-compose

Replace most GoogleMap() rememberUpdatedState() usages with single State object and encapsulate Subcomposition functionality

Offen
#492 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
priority: p2 type: feature request
Vorherrschende Sprache
Kotlin
Sterne
1.3k
Forks
181
Ø Merge
2 T. 23 Std.
Gemergte PRs (30 T.)
18

Beschreibung

The GoogleMap() implementation uses a gazillion individual rememberUpdatedState() calls. This is unwieldy and likely contributed to #481.

Instead, a single State object encapsulating the individual state properties can be used and passed as a parameter. This pattern is in compliance with Google Compose API guidelines.

The subcomposition should be made a separate composable function to prevent accidental access to local variables and better encapsulate subcomposition functionality.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, die Implementierung von GoogleMap() zu lokalisieren und ihre Verwendungen von rememberUpdatedState() sowie den Subcomposition-Code zu erfassen. Prüfe anschließend das Muster der Google Compose API-Richtlinie, trenne dann die Subcomposition und ersetze die individuelle Zustandsverwaltung durch ein einzelnes, kapselndes State-Objekt; abgeschlossen ist die Arbeit, wenn die Implementierung gekapselt ist, ohne das Verhalten zu ändern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
android, kotlin
Bereich
mobile
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.