From 66d2f4f09b2d9a47258779ebb199e8741a0e1658 Mon Sep 17 00:00:00 2001
From: plouc <benitteraphael@gmail.com>
Date: Mon, 16 Nov 2020 08:45:07 +0900
Subject: [PATCH] feat(core): add proper type for useTheme hook

---
 packages/core/index.d.ts        | 87 +++++++++++++++++++++++----------
 packages/marimekko/src/hooks.ts |  1 -
 2 files changed, 61 insertions(+), 27 deletions(-)

diff --git a/packages/core/index.d.ts b/packages/core/index.d.ts
index 8a77a79db5..da68d235ef 100644
--- a/packages/core/index.d.ts
+++ b/packages/core/index.d.ts
@@ -37,65 +37,100 @@ declare module '@nivo/core' {
         colorBy?: string | GetColor<T>
     }
 
-    export type Theme = Partial<{
-        crosshair: Partial<{
-            line: Partial<{
+    export type CompleteTheme = {
+        crosshair: {
+            line: {
                 stroke: string
                 strokeWidth: number
                 strokeOpacity: number
                 strokeDasharray: string
-            }>
-        }>
+            }
+        }
         background: string
         fontFamily: string
         fontSize: number
         textColor: string
-        axis: Partial<{
-            domain: Partial<{
+        axis: {
+            domain: {
                 line: Partial<React.CSSProperties>
-            }>
-            ticks: Partial<{
+            }
+            ticks: {
                 line: Partial<React.CSSProperties>
                 text: Partial<React.CSSProperties>
-            }>
-            legend: Partial<{
+            }
+            legend: {
                 text: Partial<React.CSSProperties>
-            }>
-        }>
-        grid: Partial<{
+            }
+        }
+        grid: {
             line: Partial<React.CSSProperties>
-        }>
-        legends: Partial<{
+        }
+        legends: {
             text: Partial<React.CSSProperties>
-        }>
-        labels: Partial<{
+        }
+        labels: {
             text: Partial<React.CSSProperties>
-        }>
-        markers: Partial<{
+        }
+        markers: {
             lineColor: string
             lineStrokeWidth: number
             textColor: string
             fontSize: string | 0
             text: Partial<React.CSSProperties>
-        }>
-        dots: Partial<{
+        }
+        dots: {
             text: Partial<React.CSSProperties>
-        }>
-        tooltip: Partial<{
+        }
+        tooltip: {
             container: Partial<React.CSSProperties>
             basic: Partial<React.CSSProperties>
             chip: Partial<React.CSSProperties>
             table: Partial<React.CSSProperties>
             tableCell: Partial<React.CSSProperties>
-        }>
-        annotations: Partial<{
+        }
+        annotations: {
             text: Partial<React.CSSProperties>
             link: Partial<React.CSSProperties>
             outline: Partial<React.CSSProperties>
             symbol: Partial<React.CSSProperties>
+        }
+    }
+
+    export type Theme = Partial<{
+        crosshair: Partial<{
+            line: Partial<CompleteTheme['crosshair']['line']>
         }>
+        axis: Partial<{
+            domain: Partial<{
+                line: Partial<CompleteTheme['axis']['domain']['line']>
+            }>
+            ticks: Partial<{
+                line: Partial<CompleteTheme['axis']['ticks']['line']>
+                text: Partial<CompleteTheme['axis']['ticks']['text']>
+            }>
+            legend: Partial<{
+                text: Partial<CompleteTheme['axis']['legend']['text']>
+            }>
+        }>
+        grid: Partial<{
+            line: Partial<CompleteTheme['grid']['line']>
+        }>
+        legends: Partial<{
+            text: Partial<CompleteTheme['legends']['text']>
+        }>
+        labels: Partial<{
+            text: Partial<CompleteTheme['labels']['text']>
+        }>
+        markers: Partial<CompleteTheme['markers']>
+        dots: Partial<{
+            text: Partial<CompleteTheme['dots']['text']>
+        }>
+        tooltip: Partial<CompleteTheme['tooltip']>
+        annotations: Partial<CompleteTheme['annotations']>
     }>
 
+    export function useTheme(): CompleteTheme
+
     export type MotionProps = Partial<{
         animate: boolean
         motionDamping: number
diff --git a/packages/marimekko/src/hooks.ts b/packages/marimekko/src/hooks.ts
index 6cf20ba027..e35cc83d85 100644
--- a/packages/marimekko/src/hooks.ts
+++ b/packages/marimekko/src/hooks.ts
@@ -2,7 +2,6 @@ import { useMemo } from 'react'
 import { get } from 'lodash'
 import { stack as d3Stack, Stack, stackOffsetDiverging, Series } from 'd3-shape'
 import { ScaleLinear, scaleLinear } from 'd3-scale'
-// @ts-ignore
 import { useTheme } from '@nivo/core'
 import { InheritedColorConfig, useInheritedColor, useOrdinalColorScale } from '@nivo/colors'
 import {