-
Notifications
You must be signed in to change notification settings - Fork 31
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feat/dark theme #562
Feat/dark theme #562
Conversation
/build |
/build |
/build |
/build |
local_modules/styles/mixins.less
Outdated
} | ||
|
||
.dropdown-checkbox-item { | ||
padding: 6px 8px 6px 8px; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
padding: 6px 8px;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
поправил
Было бы круто если в сторибуке тоже можно было бы тему переключать. |
Ещё предлагаю завести тикет на добавление скриншотных тестов для тёмной темы |
@@ -0,0 +1,29 @@ | |||
import { useEffect, useState } from "react"; | |||
|
|||
export const useIsBrowserPrefersDarkTheme = () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Этот файл - полный дубликат useBrowserThemeDetector
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
удолил
/build |
Build and push Docker images with tag: 2024-12-09.bf1a9dd |
PR Summary
replaced contact icons svg with react components, to change color with prop
changing theme in dependance of feature flag value
there are 3 possible value available
provided specific color variables for each theme
updated screenshots due to contacticons diffs
provided all custom components with corresponding colors