Add Defs to recharts components

[Darksinian]

No description provided.

[greptile-apps]

Greptile Summary

This PR adds "Defs" (the SVG <defs> element) to the _valid_children list of all remaining recharts chart types — BarChart, LineChart, ComposedChart, PieChart, RadarChart, RadialBarChart, ScatterChart, and FunnelChart — bringing them in line with AreaChart, which already had this entry. This allows users to embed SVG definitions (e.g., linearGradient, filter) inside any chart for gradient fills or other effects.

Confidence Score: 5/5

Safe to merge — additive, no-risk change that aligns all chart types with existing AreaChart behavior.

The change is purely additive, consistent with the established pattern in AreaChart, and correctly uses the class name "Defs" that the _valid_children validation resolves against type(child).name. No logic, rendering, or data-integrity concerns.

No files require special attention.

Important Files Changed

Filename	Overview
packages/reflex-components-recharts/src/reflex_components_recharts/charts.py	Adds "Defs" to the _valid_children list of 8 chart classes (BarChart, LineChart, ComposedChart, PieChart, RadarChart, RadialBarChart, ScatterChart, FunnelChart), consistent with AreaChart which already had it; Treemap has no _valid_children at all and needs no change.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User creates chart] --> B{Chart type}
    B --> C[AreaChart already had Defs]
    B --> D[BarChart - Defs added]
    B --> E[LineChart - Defs added]
    B --> F[ComposedChart - Defs added]
    B --> G[PieChart - Defs added]
    B --> H[RadarChart - Defs added]
    B --> I[RadialBarChart - Defs added]
    B --> J[ScatterChart - Defs added]
    B --> K[FunnelChart - Defs added]
    B --> L[Treemap - no _valid_children, accepts any child]
    C & D & E & F & G & H & I & J & K --> M[rx.el.defs / Defs SVG element]
    M --> N[linearGradient, filter, clipPath, etc.]

Loading

_{Reviews (1): Last reviewed commit: "Add Defs to recharts components" | Re-trigger Greptile}

[codspeed-hq]

Merging this PR will not alter performance

✅ 9 untouched benchmarks

---

_{Comparing mo/allow-defs-in-recharts (f72149a) with main (ecccd05)}