Sticking with the Tremor switch as the NextUI switch has the same per…

…formance bug as Daisy.
buchananwill · Mar 29, 2024 · 64c5a97 · 64c5a97
1 parent 49dcf4e
commit 64c5a97
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 8 deletions.
diff --git a/app/generic/components/tooltips/tool-tips-toggle.tsx b/app/generic/components/tooltips/tool-tips-toggle.tsx
@@ -1,5 +1,5 @@
 'use client';
-import { Switch, Text } from '@tremor/react';
+import { Switch } from '@tremor/react';
 import React from 'react';
 import { useTooltipsContext } from './tooltips-context-provider';
 
@@ -8,13 +8,13 @@ const ToolTipsToggle = () => {
 
   return (
     <label className="flex items-center">
-      <Text className={'text-gray-200'}>Tooltips:</Text>
+      <p className={'text-gray-400'}>Tooltips:</p>
       <Switch
         // type="checkbox"
         name="tool-tips"
         // className="toggle toggle-xs ml-1 toggle-success"
         checked={showTooltips}
-        onChange={() => setShowTooltips(!showTooltips)}
+        onChange={() => setShowTooltips((prev) => !prev)}
       ></Switch>
     </label>
   );

diff --git a/app/generic/components/tooltips/tooltips-context-provider.tsx b/app/generic/components/tooltips/tooltips-context-provider.tsx
@@ -15,9 +15,7 @@ export default function TooltipsContextProvider({
 
   const contextValue: TooltipsState = {
     showTooltips: showTooltips,
-    setShowTooltips: (value: boolean) => {
-      setShowTooltips(value);
-    }
+    setShowTooltips: setShowTooltips
   };
 
   return (

diff --git a/app/generic/components/tooltips/tooltips-context.tsx b/app/generic/components/tooltips/tooltips-context.tsx
@@ -1,8 +1,8 @@
-import { createContext } from 'react';
+import { createContext, Dispatch, SetStateAction } from 'react';
 
 export interface TooltipsState {
   showTooltips: boolean;
-  setShowTooltips: (value: boolean) => void;
+  setShowTooltips: Dispatch<SetStateAction<boolean>>;
 }
 
 export default createContext<TooltipsState>({