rememberCameraPositionState() should accept `inputs` parameter

I noticed that in version 2.7.0 of maps-compose the function rememberCameraPositionState does not accept an inputs parameter although it just delegates to rememberSaveable, which has vararg inputs: Any?. Is this a deliberate design decision?

I have a situation where the value actually should be recreated because latitude and longitude change after the user has selected a different location.

I’m happy to provide a pull request if this is a desired change.

[arriolac]

Adding inputs to rememberMarkerState will not recreate the Marker so in theory you can expose inputs to both remember functions. But, recreating CameraPositionState needs to set the associated GoogleMap and MarkerState needs to set the correct internal Marker. These side effects make implementing this feature not so straightforward actually… I suggest holding off on the pull request for that reason.

[svenjacobs]

Good point. In case of rememberCameraPositionState() I tried to work around it by using

val pos = remember(latitude, longitude) { LatLng(latitude, longitude) }
val cameraPositionState = rememberSaveable(pos, saver = CameraPositionState.Saver) {
    CameraPositionState(
        position = CameraPosition.fromLatLngZoom(pos, 17f),
    )
}

which works as expected but applying the same pattern to rememberMarkerState() did not work. The inner workings of Marker probably don’t take state changes into account at the moment.