-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Open
Labels
breaking changeUse this label to indicate a breaking change in the Recharts APIUse this label to indicate a breaking change in the Recharts APIrefactorPRs that refactor existing codePRs that refactor existing code
Description
It is not clear what benefit one gains by using specific Cartesian Chart wrappers such as the AreaChart or the LineChart, when instead all charts can be achieved using the ComposedChart.
Proposal:
- Deprecate LineChart, AreaChart, BarChart, ScatterChart
- Rename ComposedChart to CartesianChart
Advantages:
- Reduced API, thus user-facing complexity
- Reduced bug surface
Disadvantages:
- ?
Open Questions:
- The ScatterChart currently has a separate Tooltip behaviour - should the Scatterplot remain?
- Is the interface of the ComposedChart equal to the union of fields of the LineChart, BarChart and AreaChart? What is missing?
aitk, albertoblaz, snelsi, Brookke, zaaakher and 2 more
Metadata
Metadata
Assignees
Labels
breaking changeUse this label to indicate a breaking change in the Recharts APIUse this label to indicate a breaking change in the Recharts APIrefactorPRs that refactor existing codePRs that refactor existing code