From a4d64d8e05c6c66ee50444c377f0e4132369ca62 Mon Sep 17 00:00:00 2001 From: Caio Cabral Date: Mon, 2 Dec 2024 19:14:26 -0500 Subject: [PATCH] fix: fixing button color text on dark mode --- Client/src/Pages/Account/index.jsx | 2 +- Client/src/Utils/Theme/constants.js | 4 +++- Client/src/Utils/Theme/lightTheme.js | 1 + 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/Client/src/Pages/Account/index.jsx b/Client/src/Pages/Account/index.jsx index 835c1a711..e3a019f98 100644 --- a/Client/src/Pages/Account/index.jsx +++ b/Client/src/Pages/Account/index.jsx @@ -1,4 +1,4 @@ -import React, { useState } from "react"; +import { useState } from "react"; import PropTypes from "prop-types"; import { useNavigate } from "react-router"; import { useSelector } from "react-redux"; diff --git a/Client/src/Utils/Theme/constants.js b/Client/src/Utils/Theme/constants.js index 6f71c5d1b..1b5cdc27f 100644 --- a/Client/src/Utils/Theme/constants.js +++ b/Client/src/Utils/Theme/constants.js @@ -206,12 +206,14 @@ const semanticColors = { light: { light: paletteColors.gray200, dark: paletteColors.gray800, + /* TODO this should live in a different key (border.disabled.light and .dark) */ disabled: paletteColors.gray150, }, dark: { light: paletteColors.gray200, dark: paletteColors.gray750, - disabled: paletteColors.gray800, + /* TODO this should live in a different key (border.disabled.light and .dark) */ + disabled: paletteColors.gray150, }, }, unresolved: { diff --git a/Client/src/Utils/Theme/lightTheme.js b/Client/src/Utils/Theme/lightTheme.js index 88e47282d..a4758f22d 100644 --- a/Client/src/Utils/Theme/lightTheme.js +++ b/Client/src/Utils/Theme/lightTheme.js @@ -28,6 +28,7 @@ const { } = colors; const palette = { + /* TODO check if we need the addition of a new color gray150 for this. Also, this color would probably fit for primary contrastText */ action: { disabled: border.light.disabled, },