fix(components): disable preserveDrawingBuffer by default for better performance (#3326)

preserveDrawingBuffer was hardcoded to true for the base map, causing a
known WebGL performance penalty on every frame. This option is only needed
for image export (canvas.toDataURL()), so we now only enable it when
isExport is true. The plot-container already sets isExport: true when
rendering for export, ensuring image export continues to work correctly.

Users who need preserveDrawingBuffer for other purposes can still override
it via bottomMapContainerProps.

Addresses #3237

Signed-off-by: pierreeurope <pierre.europe@pm.me>

Author: pierreeurope

src/components/src/map-container.tsx

@@ -1112,7 +1112,7 @@ export default function MapContainerFactory(
       const internalViewState = this.context?.getInternalViewState(index);
       const mapProps = {
         ...internalViewState,
-        preserveDrawingBuffer: true,
+        preserveDrawingBuffer: this.props.isExport ?? false,
         mapboxAccessToken: currentStyle?.accessToken || mapboxApiAccessToken,
         // baseApiUrl: mapboxApiUrl,
         mapLib: baseMapLibraryConfig.getMapLib(),