From 4b5c956dbe77cefebe2c0bf90e3e2b212fe6f9e9 Mon Sep 17 00:00:00 2001
From: Cristhian Zanforlin Lousa <cristhian.lousa@gmail.com>
Date: Mon, 2 Dec 2024 13:17:39 -0300
Subject: [PATCH] feat: add NvidiaIcon component to support dark mode (#4994)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* ✨ (Nvidia): Add support for dark mode in Nvidia icon by using useDarkStore hook to determine fill color based on dark mode state

* ♻️ (index.tsx): remove unnecessary console.log statement from CustomInputPopover component
---
 .../components/inputComponent/components/popover/index.tsx    | 2 --
 src/frontend/src/icons/Nvidia/index.tsx                       | 4 +++-
 src/frontend/src/icons/Nvidia/nvidia.jsx                      | 2 ++
 .../tests/extended/integrations/youtube-transcripts.spec.ts   | 2 --
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/frontend/src/components/core/parameterRenderComponent/components/inputComponent/components/popover/index.tsx b/src/frontend/src/components/core/parameterRenderComponent/components/inputComponent/components/popover/index.tsx
index f13a46127..8147ce84c 100644
--- a/src/frontend/src/components/core/parameterRenderComponent/components/inputComponent/components/popover/index.tsx
+++ b/src/frontend/src/components/core/parameterRenderComponent/components/inputComponent/components/popover/index.tsx
@@ -62,8 +62,6 @@ const CustomInputPopover = ({
     }
   };
 
-  console.log(placeholder, value, id);
-
   return (
     <Popover modal open={showOptions} onOpenChange={setShowOptions}>
       <PopoverAnchor>
diff --git a/src/frontend/src/icons/Nvidia/index.tsx b/src/frontend/src/icons/Nvidia/index.tsx
index 830b62b9b..3562b3a93 100644
--- a/src/frontend/src/icons/Nvidia/index.tsx
+++ b/src/frontend/src/icons/Nvidia/index.tsx
@@ -1,3 +1,4 @@
+import { useDarkStore } from "@/stores/darkStore";
 import React, { forwardRef } from "react";
 import NvidiaSVG from "./nvidia";
 
@@ -5,5 +6,6 @@ export const NvidiaIcon = forwardRef<
   SVGSVGElement,
   React.PropsWithChildren<{}>
 >((props, ref) => {
-  return <NvidiaSVG ref={ref} {...props} />;
+  const isdark = useDarkStore((state) => state.dark);
+  return <NvidiaSVG ref={ref} isdark={isdark} {...props} />;
 });
diff --git a/src/frontend/src/icons/Nvidia/nvidia.jsx b/src/frontend/src/icons/Nvidia/nvidia.jsx
index f572e6128..456f23309 100644
--- a/src/frontend/src/icons/Nvidia/nvidia.jsx
+++ b/src/frontend/src/icons/Nvidia/nvidia.jsx
@@ -16,6 +16,7 @@ const NvidiaSVG = (props) => (
     </title>
     <path
       id="path17"
+      fill={props.isdark ? "#fff" : "#000"}
       d="M384.195,282.109c0,3.771-2.769,6.302-6.047,6.302v-0.023c-3.371,0.023-6.089-2.508-6.089-6.278
 	c0-3.769,2.718-6.293,6.089-6.293C381.427,275.816,384.195,278.34,384.195,282.109z M386.648,282.109c0-5.175-4.02-8.179-8.5-8.179
 	c-4.511,0-8.531,3.004-8.531,8.179c0,5.172,4.021,8.188,8.531,8.188C382.629,290.297,386.648,287.281,386.648,282.109
@@ -24,6 +25,7 @@ const NvidiaSVG = (props) => (
     />
     <path
       id="path19"
+      fill={props.isdark ? "#fff" : "#000"}
       d="M329.406,237.027l10.598,28.993H318.48L329.406,237.027z M318.056,225.738l-24.423,61.88h17.246l3.863-10.934
 	h28.903l3.656,10.934h18.722l-24.605-61.888L318.056,225.738z M269.023,287.641h17.497v-61.922l-17.5-0.004L269.023,287.641z
 	 M147.556,225.715l-14.598,49.078l-13.984-49.074l-18.879-0.004l19.972,61.926h25.207l20.133-61.926H147.556z M218.281,239.199h7.52
diff --git a/src/frontend/tests/extended/integrations/youtube-transcripts.spec.ts b/src/frontend/tests/extended/integrations/youtube-transcripts.spec.ts
index 5d7dc8e57..f09648c98 100644
--- a/src/frontend/tests/extended/integrations/youtube-transcripts.spec.ts
+++ b/src/frontend/tests/extended/integrations/youtube-transcripts.spec.ts
@@ -56,8 +56,6 @@ test(
       .getByTestId("textarea_str_url")
       .fill("https://www.youtube.com/watch?v=VqhCQZaH4Vs");
 
-    await page.getByTestId("textarea_str_language").fill("en");
-
     await page.getByTestId("button_run_youtube transcripts").click();
 
     await page.waitForSelector("text=built successfully", { timeout: 30000 });