[@capacitor/google-maps] updateRender method in Android does not respect width and height changes

[AE1NS]

Bug Report

Plugin(s)

@capacitor/google-maps: 4.5.0

Capacitor Version

@capacitor/cli: 4.7.0
@capacitor/core: 4.7.0
@capacitor/ios: 4.7.0
@capacitor/android: 4.7.0

Platform(s)

Android

Current Behavior

When I update the map container dimensions (width, height, x, y) and call 'updateMapBounds', the native map just get repositioned but not resized.

Expected Behavior

The native map should also update its widht and height attributes.

Reproduction

• Resize the html map wrapper, i.e. set a new height or width, with dev tools on the remote android device
• Call updateMapBounds on the plugin instance map object or alternatively call window.dispatchEvent(new CustomEvent('resize'))
• Now the map only updates it x and y values correctly but the size keeps in the initial size.

Other Technical Details

Only the x and y values are updated on the map view in the implementation:

capacitor-plugins/google-maps/android/src/main/java/com/capacitorjs/plugins/googlemaps/CapacitorGoogleMap.kt

Lines 119 to 123 in c39d84f

	CoroutineScope(Dispatchers.Main).launch {
	val mapRect = getScaledRect(delegate.bridge, updatedBounds)
	this@CapacitorGoogleMap.mapView.x = mapRect.left
	this@CapacitorGoogleMap.mapView.y = mapRect.top
	}

[AE1NS]

Replacing it with this, works:

CoroutineScope(Dispatchers.Main).launch {
        val bridge = delegate.bridge
        val layoutParams =
                FrameLayout.LayoutParams(
                        getScaledPixels(bridge, config.width),
                        getScaledPixels(bridge, config.height),
                )
        layoutParams.leftMargin = getScaledPixels(bridge, config.x)
        layoutParams.topMargin = getScaledPixels(bridge, config.y)
        [email protected] = layoutParams
}

But I dont know if it has any side effects.

[AE1NS]

Or this:

CoroutineScope(Dispatchers.Main).launch {
        val mapRect = getScaledRect(delegate.bridge, updatedBounds)
        [email protected] = mapRect.left
        [email protected] = mapRect.top
        [email protected] = getScaledPixels(delegate.bridge, config.width)
        [email protected] = getScaledPixels(delegate.bridge, config.height)
        [email protected]()
}

[AE1NS]

Closed in favor of #1594

[ionitron-bot]

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of the plugin, please create a new issue and ensure the template is fully filled out.