Merge pull request #62 from ahrefs/fix/type-defs

Fix tooltip cursor type
ahrefs · Dec 6, 2024 · 36bf79a · 36bf79a
2 parents 62dfc3f + 4c11a57
commit 36bf79a
Show file tree

Hide file tree

Showing 27 changed files with 285 additions and 403 deletions.
diff --git a/CHANGES.md b/CHANGES.md
@@ -1,3 +1,7 @@
+## 5.0.2
+
+- change Tooltip.cursor Obj type to open object type
+
 ## 5.0.1
 
 - update `recharts` to `2.14.1`

diff --git a/example/Example.re b/example/Example.re
@@ -2,9 +2,9 @@ open Recharts;
 
 [@react.component]
 let make = (~data) =>
-  <ResponsiveContainer height=(Px(200.)) width=(Px(300.))>
+  <ResponsiveContainer height={Px(200.)} width={Px(300.)}>
     <BarChart
-      barCategoryGap=(Px(1.))
+      barCategoryGap={Px(1.)}
       margin={"top": 0, "right": 0, "bottom": 0, "left": 0}
       data>
       <Bar name="Some bar" dataKey="pv" fill="#2078b4" stackId="a" />

diff --git a/src/Area.re b/src/Area.re
@@ -8,15 +8,14 @@ external make:
     ~activeDot: 'activeDot=?,
     ~animationBegin: int=?,
     ~animationDuration: int=?,
-    ~animationEasing:
-      [@mel.string] [
-        | `ease
-        | [@mel.as "ease-in"] `easeIn
-        | [@mel.as "ease-out"] `easeOut
-        | [@mel.as "ease-in-out"] `easeInOut
-        | `linear
-      ]
-        =?,
+    ~animationEasing: [@mel.string] [
+                        | `ease
+                        | [@mel.as "ease-in"] `easeIn
+                        | [@mel.as "ease-out"] `easeOut
+                        | [@mel.as "ease-in-out"] `easeInOut
+                        | `linear
+                      ]
+                        =?,
     ~baseLine: 'baseLine=?,
     ~connectNulls: bool=?,
     ~hide: bool=?,
@@ -33,8 +32,7 @@ external make:
     ~name: string=?,
     ~onClick: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseDown: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
+    ~onMouseEnter: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseLeave: (Js.t({..}), React.Event.Mouse.t) => unit=?,
     ~onMouseMove: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseOut: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,

diff --git a/src/AreaChart.re b/src/AreaChart.re
@@ -14,8 +14,7 @@ external make:
     ~onClick: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseUp: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseDown: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
+    ~onMouseEnter: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseLeave: (Js.t({..}), React.Event.Mouse.t) => unit=?,
     ~onMouseMove: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~stackOffset: stackOffset=?,

diff --git a/src/Bar.re b/src/Bar.re
@@ -6,15 +6,14 @@ external make:
   (
     ~animationBegin: int=?,
     ~animationDuration: int=?,
-    ~animationEasing:
-      [@mel.string] [
-        | `ease
-        | [@mel.as "ease-in"] `easeIn
-        | [@mel.as "ease-out"] `easeOut
-        | [@mel.as "ease-in-out"] `easeInOut
-        | `linear
-      ]
-        =?,
+    ~animationEasing: [@mel.string] [
+                        | `ease
+                        | [@mel.as "ease-in"] `easeIn
+                        | [@mel.as "ease-out"] `easeOut
+                        | [@mel.as "ease-in-out"] `easeInOut
+                        | `linear
+                      ]
+                        =?,
     ~background: 'background=?,
     ~barSize: int=?,
     ~className: string=?,
@@ -30,22 +29,37 @@ external make:
     ~maxBarSize: int=?,
     ~minPointSize: int=?,
     ~name: string=?,
-    ~onClick:
-      (Js.t({.. "payload": 'dataItem}), int, React.Event.Mouse.t) => unit=?,
-    ~onMouseDown:
-      (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.t({.. "payload": 'dataItem}), int, React.Event.Mouse.t) => unit=?,
-    ~onMouseLeave:
-      (Js.t({.. "payload": 'dataItem}), int, React.Event.Mouse.t) => unit=?,
-    ~onMouseMove:
-      (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) => unit=?,
-    ~onMouseOut:
-      (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) => unit=?,
-    ~onMouseOver:
-      (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) => unit=?,
-    ~onMouseUp:
-      (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) => unit=?,
+    ~onClick: (Js.t({.. "payload": 'dataItem}), int, React.Event.Mouse.t) =>
+              unit
+                =?,
+    ~onMouseDown: (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) =>
+                  unit
+                    =?,
+    ~onMouseEnter: (
+                     Js.t({.. "payload": 'dataItem}),
+                     int,
+                     React.Event.Mouse.t
+                   ) =>
+                   unit
+                     =?,
+    ~onMouseLeave: (
+                     Js.t({.. "payload": 'dataItem}),
+                     int,
+                     React.Event.Mouse.t
+                   ) =>
+                   unit
+                     =?,
+    ~onMouseMove: (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) =>
+                  unit
+                    =?,
+    ~onMouseOut: (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) =>
+                 unit
+                   =?,
+    ~onMouseOver: (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) =>
+                  unit
+                    =?,
+    ~onMouseUp: (Js.t({.. "payload": 'dataItem}), React.Event.Mouse.t) => unit
+                  =?,
     ~radius: array(int)=?,
     ~shape: 'shape=?,
     ~stackId: string=?,

diff --git a/src/BarChart.re b/src/BarChart.re
@@ -17,8 +17,7 @@ external make:
     ~onClick: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseUp: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseDown: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
+    ~onMouseEnter: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseLeave: (Js.t({..}), React.Event.Mouse.t) => unit=?,
     ~onMouseMove: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~reverseStackOrder: bool=?,

diff --git a/src/CartesianAxis.re b/src/CartesianAxis.re
@@ -24,4 +24,5 @@ external make:
   React.element =
   "CartesianAxis";
 
-let makeProps = (~interval=?) => makeProps(~interval=?interval->AxisInterval.encodeOpt);
+let makeProps = (~interval=?) =>
+  makeProps(~interval=?interval->AxisInterval.encodeOpt);
diff --git a/src/ComposedChart.re b/src/ComposedChart.re
@@ -16,8 +16,7 @@ external make:
     ~onClick: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseUp: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseDown: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
+    ~onMouseEnter: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseLeave: (Js.t({..}), React.Event.Mouse.t) => unit=?,
     ~onMouseMove: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~reverseStackOrder: bool=?,

diff --git a/src/Dot.re b/src/Dot.re
@@ -9,8 +9,7 @@ external make:
     ~fill: string,
     ~onClick: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseDown: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
+    ~onMouseEnter: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseLeave: (Js.t({..}), React.Event.Mouse.t) => unit=?,
     ~onMouseMove: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseOut: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,

diff --git a/src/Legend.re b/src/Legend.re
@@ -4,51 +4,37 @@ open Utils;
 [@mel.module "recharts"] [@react.component]
 external make:
   (
-    ~align:
-      [
-        | `left
-        | `center
-        | `right
-      ]
-        =?,
+    ~align: [ | `left | `center | `right]=?,
     ~chartHeight: int=?,
     ~chartWidth: int=?,
     ~content: 'content=?,
     ~className: string=?,
     ~height: int=?,
     ~iconSize: int=?,
-    ~iconType:
-      [
-        | `line
-        | `square
-        | `rect
-        | `circle
-        | `cross
-        | `diamond
-        | `star
-        | `triangle
-        | `wye
-      ]
-        =?,
+    ~iconType: [
+                 | `line
+                 | `square
+                 | `rect
+                 | `circle
+                 | `cross
+                 | `diamond
+                 | `star
+                 | `triangle
+                 | `wye
+               ]
+                 =?,
     ~layout: layout=?,
     ~margin: margin=?,
     ~onClick: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseDown: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
+    ~onMouseEnter: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseLeave: (Js.t({..}), React.Event.Mouse.t) => unit=?,
     ~onMouseMove: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseOut: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseOver: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseUp: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~payload: array(Js.t({..}))=?,
-    ~verticalAlign:
-      [
-        | `top
-        | `middle
-        | `bottom
-      ]
-        =?,
+    ~verticalAlign: [ | `top | `middle | `bottom]=?,
     ~width: int=?,
     ~wrapperStyle: Js.t({..})=?
   ) =>

diff --git a/src/Line.re b/src/Line.re
@@ -8,15 +8,14 @@ external make:
     ~activeDot: 'activeDot=?,
     ~animationBegin: int=?,
     ~animationDuration: int=?,
-    ~animationEasing:
-      [@mel.string] [
-        | `ease
-        | [@mel.as "ease-in"] `easeIn
-        | [@mel.as "ease-out"] `easeOut
-        | [@mel.as "ease-in-out"] `easeInOut
-        | `linear
-      ]
-        =?,
+    ~animationEasing: [@mel.string] [
+                        | `ease
+                        | [@mel.as "ease-in"] `easeIn
+                        | [@mel.as "ease-out"] `easeOut
+                        | [@mel.as "ease-in-out"] `easeInOut
+                        | `linear
+                      ]
+                        =?,
     ~className: string=?,
     ~connectNulls: bool=?,
     ~hide: bool=?,
@@ -31,8 +30,7 @@ external make:
     ~name: string=?,
     ~onClick: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseDown: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
+    ~onMouseEnter: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseLeave: (Js.t({..}), React.Event.Mouse.t) => unit=?,
     ~onMouseMove: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseOut: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,

diff --git a/src/LineChart.re b/src/LineChart.re
@@ -13,8 +13,7 @@ external make:
     ~onClick: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseUp: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseDown: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
-    ~onMouseEnter:
-      (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
+    ~onMouseEnter: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~onMouseLeave: (Js.t({..}), React.Event.Mouse.t) => unit=?,
     ~onMouseMove: (Js.Nullable.t(Js.t({..})), React.Event.Mouse.t) => unit=?,
     ~syncId: string=?,