fix(swarmplot): use the borderWidth property - fixes #1813

plouc · Dec 17, 2021 · edd3990 · edd3990
1 parent 7cfbfc9
commit edd3990
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/packages/swarmplot/src/SwarmPlot.tsx b/packages/swarmplot/src/SwarmPlot.tsx
@@ -36,6 +36,7 @@ const InnerSwarmPlot = <RawDatum,>({
     colors = defaultProps.colors as OrdinalColorScaleConfig<Omit<ComputedDatum<RawDatum>, 'color'>>,
     colorBy = defaultProps.colorBy,
     borderColor = defaultProps.borderColor as InheritedColorConfig<ComputedDatum<RawDatum>>,
+    borderWidth = defaultProps.borderWidth,
     layout = defaultProps.layout,
     spacing = defaultProps.spacing,
     gap = defaultProps.gap,
@@ -141,7 +142,7 @@ const InnerSwarmPlot = <RawDatum,>({
             <Circles<RawDatum>
                 key="circles"
                 nodes={nodes}
-                borderWidth={0}
+                borderWidth={borderWidth}
                 borderColor={borderColor}
                 isInteractive={isInteractive}
                 tooltip={tooltip}

diff --git a/packages/swarmplot/tests/Swarmplot.test.tsx b/packages/swarmplot/tests/Swarmplot.test.tsx
@@ -137,5 +137,22 @@ describe('SwarmPlot', () => {
             expect(onMouseMove).not.toHaveBeenCalled()
             expect(onMouseLeave).not.toHaveBeenCalled()
         })
+
+        it('should support the borderWidth property', () => {
+            const wrapper = mount(
+                <SwarmPlot
+                    width={400}
+                    height={400}
+                    groupBy="group"
+                    groups={groups}
+                    data={sampleData}
+                    borderWidth={4}
+                />
+            )
+
+            expect(wrapper.find('circle').at(0).getDOMNode().getAttribute('stroke-width')).toEqual(
+                '4'
+            )
+        })
     })
 })