googlemaps / googlemaps/android-maps-compose
Replace most GoogleMap() rememberUpdatedState() usages with single State object and encapsulate Subcomposition functionality
- 主要言語
- 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