fix: correct toggle button type (#2651)

callstack · Apr 9, 2021 · 3593cd5 · 3593cd5
1 parent cdfb9ab
commit 3593cd5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/components/ToggleButton/ToggleButton.tsx b/src/components/ToggleButton/ToggleButton.tsx
@@ -102,7 +102,7 @@ const ToggleButton = ({
 
   return (
     <ToggleButtonGroupContext.Consumer>
-      {(context: { value: string; onValueChange: Function } | null) => {
+      {(context: { value: string | null; onValueChange: Function } | null) => {
         let backgroundColor;
 
         const checked: boolean | null =

diff --git a/src/components/ToggleButton/ToggleButtonGroup.tsx b/src/components/ToggleButton/ToggleButtonGroup.tsx
@@ -4,20 +4,20 @@ type Props = {
   /**
    * Function to execute on selection change.
    */
-  onValueChange: (value: string) => void;
+  onValueChange: (value: string) => void | null;
   /**
    * Value of the currently selected toggle button.
    */
-  value: string;
+  value: string | null;
   /**
    * React elements containing toggle buttons.
    */
   children: React.ReactNode;
 };
 
 type ToggleButtonContextType = {
-  value: string;
-  onValueChange: (item: string) => void;
+  value: string | null;
+  onValueChange: (item: string) => void | null;
 };
 
 export const ToggleButtonGroupContext = React.createContext<