-
Notifications
You must be signed in to change notification settings - Fork 99
Closed
Milestone
Description
Description
Basically all Chart extension functions should be lower camel case (Chart.someFunctionName
). The API is kind of inconsistent in that regard:
see e.g.Chart.Show
or Chart.Combine
vs. Chart.withTitle
or Chart.withX_AxisStyle
This is a change that is absolutely not backwards compatible but i think it should be done before finally releasing 2.0.0
However, i would keep the core Chart API the same to make the those look more like constructors (meaning we keep Chart.Point
, Chart.Line
, etc.)