googlemaps / googlemaps/android-maps-compose

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

オープン
#492 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
priority: p2 type: feature request
主要言語
Kotlin
スター
1.3k
フォーク
181
平均マージ
2日 23時間
マージ済み PR(30日)
18

説明

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.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず GoogleMap() の実装を見つけ、rememberUpdatedState() の使用箇所と subcomposition コードを洗い出します。Google Compose API のガイドラインのパターンを確認してから、subcomposition を分離し、個別の状態管理を 1 つの State オブジェクトに置き換えます。実装が動作を変更せずにカプセル化されれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
android, kotlin
領域
mobile
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。