fix: treat ison-size as scale

geostyler · Dec 5, 2023 · ebc9434 · ebc9434
1 parent e1c20bf
commit ebc9434
Show file tree

Hide file tree

Showing 4 changed files with 37 additions and 5 deletions.
diff --git a/data/mapbox/icon_simpleicon.ts b/data/mapbox/icon_simpleicon.ts
@@ -16,7 +16,8 @@ const iconSimpleIcon: MbStyle = {
       source: 'testsource',
       'source-layer': 'foo',
       layout: {
-        'icon-image': 'poi'
+        'icon-image': 'poi',
+        'icon-size': 2
       }
     }
   ]

diff --git a/data/mapbox_metadata/icon_simpleicon.ts b/data/mapbox_metadata/icon_simpleicon.ts
@@ -16,7 +16,8 @@ const iconSimpleIcon: MbStyle = {
       'source-layer': 'foo',
       type: 'symbol',
       layout: {
-        'icon-image': 'poi'
+        'icon-image': 'poi',
+        'icon-size': 2
       }
     }
   ],

diff --git a/data/styles/icon_simpleicon.ts b/data/styles/icon_simpleicon.ts
@@ -10,6 +10,10 @@ const iconSimpleIcon: Style = {
         source: 'https://testurl.com/sprites/mysprite.png',
         position: [0, 0],
         size: [12, 12]
+      },
+      size: {
+        name: 'mul',
+        args: [2, 12]
       }
     }]
   }],
@@ -28,6 +32,7 @@ const iconSimpleIcon: Style = {
       },
       sprite: {
         poi: {
+          'icon-size': 2,
           position: [
             0,
             0,

diff --git a/src/MapboxStyleParser.ts b/src/MapboxStyleParser.ts
@@ -93,6 +93,7 @@ export type MapboxRef = {
   };
   sprite?: {
     [key: string]: {
+      'icon-size'?: number;
       position: [number, number];
       size: [number, number];
     };
@@ -423,13 +424,32 @@ export class MapboxStyleParser implements StyleParser<Omit<MbStyle, 'sources'>>
       pitchAlignment: layout?.['icon-pitch-alignment'],
       rotate: mb2gsExpression<number>(layout?.['icon-rotate']),
       rotationAlignment: layout?.['icon-rotation-alignment'],
-      size: mb2gsExpression<number>(layout?.['icon-size']),
       textFit: layout?.['icon-text-fit'], // TODO: handle enum values
       // TODO: handle array values
       textFitPadding: layout?.['icon-text-fit-padding'] as IconSymbolizer['textFitPadding'],
       visibility: layout?.visibility && layout?.visibility !== 'none'
     };
 
+    // mabpox icon-size scales the image and does not define its size
+    if (layout?.['icon-size'] && image) {
+      const scale = mb2gsExpression<number>(layout['icon-size']) as number;
+      const widthHeightAverage = ((image.size[0] as number) + (image.size[1] as number)) / 2;
+      symbolizer.size = {
+        name: 'mul',
+        args: [
+          scale,
+          widthHeightAverage
+        ]
+      };
+
+      // Add icon-size to metadata
+      set(
+        this.gsMetadata['mapbox:ref'],
+        `sprite.${layout?.['icon-image']}.icon-size`,
+        layout['icon-size']
+      );
+    };
+
     if (MapboxStyleUtil.symbolizerAllUndefined(symbolizer)) {
       return undefined;
     }
@@ -1510,18 +1530,23 @@ export class MapboxStyleParser implements StyleParser<Omit<MbStyle, 'sources'>>
       visibility
     } = symbolizer;
 
+    const iconImage = image ? this.handleSprite(image) : undefined;
+    const iconSize = iconImage
+      ? this.gsMetadata['mapbox:ref'].sprite?.[iconImage]['icon-size']
+      : gs2mbExpression<number>(size);
+
     const layout: SymbolLayout = {
       'symbol-avoid-edges': avoidEdges as SymbolLayout['symbol-avoid-edges'],
       'icon-allow-overlap': gs2mbExpression<boolean>(allowOverlap),
       'icon-optional': optional as SymbolLayout['icon-optional'],
       'icon-rotation-alignment': gs2mbExpression<SymbolLayout['icon-rotation-alignment']>
       (rotationAlignment) as SymbolLayout['icon-rotation-alignment'],
-      'icon-size': gs2mbExpression<number>(size),
+      'icon-size': iconSize,
       'icon-text-fit': gs2mbExpression<SymbolLayout['icon-text-fit']>(textFit) as SymbolLayout['icon-text-fit'],
       // TODO: handle array values
       'icon-text-fit-padding': textFitPadding as SymbolLayout['icon-text-fit-padding'],
       // TODO: check sprite handling
-      'icon-image': image ? this.handleSprite(image) : undefined,
+      'icon-image': iconImage,
       'icon-rotate': gs2mbExpression<number>(rotate),
       'icon-padding': gs2mbExpression<number>(padding),
       'icon-keep-upright': keepUpright as SymbolLayout['icon-keep-upright'],