-
Notifications
You must be signed in to change notification settings - Fork 3.5k
feat(ui): update colors to semantic tokens in home and settings folders #16877
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
Changes from all commits
2979f26
d8170d3
9ac9c54
f58f662
6c31453
41554dd
6fbd00f
f628db3
26985dc
b6bb67f
8bd57ea
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,17 +6,16 @@ import styled from 'styled-components'; | |
|
|
||
| import analytics, { EventType, HomePageModule } from '@app/analytics'; | ||
| import { useUserContext } from '@app/context/useUserContext'; | ||
| import { ANTD_GRAY } from '@app/entity/shared/constants'; | ||
| import { Announcement } from '@app/homeV2/action/announcement/Announcement'; | ||
| import { useGetUnseenAnnouncements } from '@app/homeV2/action/announcement/useGetUnseenAnnouncements'; | ||
| import AnnouncementsSkeleton from '@app/homeV2/content/tabs/announcements/AnnouncementsSkeleton'; | ||
| import { useUpdateLastViewedAnnouncementTime } from '@app/homeV2/shared/updateLastViewedAnnouncementTime'; | ||
| import OnboardingContext from '@app/onboarding/OnboardingContext'; | ||
|
|
||
| const Card = styled.div` | ||
| border: 1px solid ${ANTD_GRAY[4]}; | ||
| border: 1px solid ${(props) => props.theme.colors.bgSurface}; | ||
| border-radius: 11px; | ||
| background-color: #ffffff; | ||
| background-color: ${(props) => props.theme.colors.bg}; | ||
| overflow: hidden; | ||
| padding: 16px 20px 8px 20px; | ||
| width: 380px; | ||
|
|
@@ -31,20 +30,20 @@ const Header = styled.div` | |
| const Title = styled.div` | ||
| font-weight: 600; | ||
| font-size: 14px; | ||
| color: #434863; | ||
| color: ${(props) => props.theme.colors.text}; | ||
| display: flex; | ||
| align-items: center; | ||
| justify-content: start; | ||
| `; | ||
|
|
||
| const Icon = styled(NotificationOutlined)` | ||
| margin-right: 8px; | ||
| color: #3cb47a; | ||
| color: ${(props) => props.theme.colors.textSuccess}; | ||
| font-size: 16px; | ||
| `; | ||
|
|
||
| const StyledCloseOutlined = styled(CloseOutlined)` | ||
| color: ${ANTD_GRAY[8]}; | ||
| color: ${(props) => props.theme.colors.textSecondary}; | ||
| font-size: 12px; | ||
| `; | ||
|
|
||
|
|
@@ -55,11 +54,11 @@ const StyledCarousel = styled(Carousel)` | |
| overflow: hidden; | ||
|
|
||
| > .slick-dots li button { | ||
| background-color: #d9d9d9; | ||
| background-color: ${(props) => props.theme.colors.border}; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. do we have some more suitable |
||
| } | ||
|
|
||
| > .slick-dots li.slick-active button { | ||
| background-color: #5c3fd1; | ||
| background-color: ${(props) => props.theme.colors.buttonFillBrand}; | ||
| } | ||
| `; | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,9 +2,7 @@ import React from 'react'; | |
| import { Link } from 'react-router-dom'; | ||
| import styled from 'styled-components'; | ||
|
|
||
| import { ANTD_GRAY } from '@app/entity/shared/constants'; | ||
| import { GenericEntityProperties } from '@app/entity/shared/types'; | ||
| import { SEARCH_COLORS } from '@app/entityV2/shared/constants'; | ||
| import { getDisplayedEntityType } from '@app/entityV2/shared/containers/profile/header/utils'; | ||
| import { HoverEntityTooltip } from '@app/recommendations/renderer/component/HoverEntityTooltip'; | ||
| import PlatformIcon from '@app/sharedV2/icons/PlatformIcon'; | ||
|
|
@@ -16,21 +14,21 @@ const Container = styled.div` | |
| display: flex; | ||
| flex-direction: row; | ||
| padding: 10px 12px 10px 12px; | ||
| background-color: #ffffff; | ||
| background-color: ${(props) => props.theme.colors.bg}; | ||
| border-radius: 10px; | ||
| min-width: 200px; | ||
| max-width: 260px; | ||
| border: 1.5px solid #0000001a; | ||
| border: 1.5px solid ${(props) => props.theme.colors.border}; | ||
| gap: 12px; | ||
|
|
||
| :hover { | ||
| border: 1.5px solid ${SEARCH_COLORS.LINK_BLUE}; | ||
| border: 1.5px solid ${(props) => props.theme.colors.hyperlinks}; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. borderBrand |
||
| } | ||
| `; | ||
|
|
||
| const Name = styled.div` | ||
| font-size: 14px; | ||
| color: #565657; | ||
| color: ${(props) => props.theme.colors.textSecondary}; | ||
| margin-bottom: 4px; | ||
| font-weight: 500; | ||
| overflow: hidden; | ||
|
|
@@ -46,15 +44,15 @@ const Context = styled.div` | |
|
|
||
| const SubHeader = styled.div` | ||
| font-size: 14px; | ||
| color: ${ANTD_GRAY[5]}; | ||
| color: ${(props) => props.theme.colors.border}; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. let's try with |
||
| margin-top: 4px; | ||
| overflow: hidden; | ||
| text-overflow: ellipsis; | ||
| `; | ||
|
|
||
| const Type = styled.div` | ||
| font-size: 14px; | ||
| color: ${ANTD_GRAY[7]}; | ||
| color: ${(props) => props.theme.colors.textTertiary}; | ||
| text-overflow: ellipsis; | ||
| overflow: hidden; | ||
| white-space: nowrap; | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,15 +2,14 @@ import { Editor } from '@components'; | |
| import React from 'react'; | ||
| import styled from 'styled-components/macro'; | ||
|
|
||
| import { ANTD_GRAY } from '@app/entity/shared/constants'; | ||
| import { toRelativeTimeString } from '@app/shared/time/timeUtils'; | ||
|
|
||
| import { Post } from '@types'; | ||
|
|
||
| const Card = styled.div` | ||
| border: 1px solid ${ANTD_GRAY[4]}; | ||
| border: 1px solid ${(props) => props.theme.colors.bgHover}; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. can we use |
||
| border-radius: 8px; | ||
| background-color: #ffffff; | ||
| background-color: ${(props) => props.theme.colors.bg}; | ||
| overflow: auto; | ||
| `; | ||
|
|
||
|
|
@@ -34,7 +33,7 @@ const Title = styled.div` | |
|
|
||
| const Time = styled.div` | ||
| font-size: 12px; | ||
| color: ${ANTD_GRAY[7]}; | ||
| color: ${(props) => props.theme.colors.textTertiary}; | ||
| `; | ||
|
|
||
| type Props = { | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,24 +2,23 @@ import React from 'react'; | |
| import { Link } from 'react-router-dom'; | ||
| import styled from 'styled-components/macro'; | ||
|
|
||
| import { ANTD_GRAY } from '@app/entity/shared/constants'; | ||
| import { SEARCH_COLORS } from '@app/entityV2/shared/constants'; | ||
| import { DomainColoredIcon } from '@app/entityV2/shared/links/DomainColoredIcon'; | ||
| import { HoverEntityTooltip } from '@app/recommendations/renderer/component/HoverEntityTooltip'; | ||
| import { useEntityRegistry } from '@app/useEntityRegistry'; | ||
| import { colors } from '@src/alchemy-components'; | ||
| import { useShowNavBarRedesign } from '@src/app/useShowNavBarRedesign'; | ||
|
|
||
| import { DataProduct, Domain, EntityType } from '@types'; | ||
|
|
||
| const Card = styled(Link)<{ $isShowNavBarRedesign?: boolean }>` | ||
| border-radius: ${(props) => (props.$isShowNavBarRedesign ? '8px' : '10px')}; | ||
| background-color: #ffffff; | ||
| background-color: ${(props) => props.theme.colors.bg}; | ||
| padding: 10px 16px; | ||
| border: ${(props) => (props.$isShowNavBarRedesign ? `1px solid ${colors.gray[100]}` : '2px solid transparent')}; | ||
| border: ${(props) => | ||
| props.$isShowNavBarRedesign ? `1px solid ${props.theme.colors.border}` : '2px solid transparent'}; | ||
|
|
||
| :hover { | ||
| border: ${(props) => (props.$isShowNavBarRedesign ? '1px' : '2px')} solid ${SEARCH_COLORS.LINK_BLUE}; | ||
| border: ${(props) => (props.$isShowNavBarRedesign ? '1px' : '2px')} solid | ||
| ${(props) => props.theme.colors.hyperlinks}; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. borderHover |
||
| } | ||
|
|
||
| display: flex; | ||
|
|
@@ -36,7 +35,7 @@ const Text = styled.div` | |
|
|
||
| const Name = styled.div` | ||
| font-size: 16px; | ||
| color: ${ANTD_GRAY[9]}; | ||
| color: ${(props) => props.theme.colors.text}; | ||
| overflow: hidden; | ||
| text-overflow: ellipsis; | ||
| white-space: nowrap; | ||
|
|
@@ -53,7 +52,7 @@ const Section = styled.div` | |
| const SectionName = styled.div` | ||
| font-size: 14px; | ||
| margin-left: 10px; | ||
| color: ${ANTD_GRAY[7]}; | ||
| color: ${(props) => props.theme.colors.textTertiary}; | ||
| overflow: hidden; | ||
| text-overflow: ellipsis; | ||
| `; | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -3,25 +3,24 @@ import React from 'react'; | |
| import { Link } from 'react-router-dom'; | ||
| import styled from 'styled-components/macro'; | ||
|
|
||
| import { ANTD_GRAY } from '@app/entity/shared/constants'; | ||
| import { SEARCH_COLORS } from '@app/entityV2/shared/constants'; | ||
| import { DomainColoredIcon } from '@app/entityV2/shared/links/DomainColoredIcon'; | ||
| import { HoverEntityTooltip } from '@app/recommendations/renderer/component/HoverEntityTooltip'; | ||
| import { formatNumber } from '@app/shared/formatNumber'; | ||
| import { useEntityRegistry } from '@app/useEntityRegistry'; | ||
| import { colors } from '@src/alchemy-components'; | ||
| import { useShowNavBarRedesign } from '@src/app/useShowNavBarRedesign'; | ||
|
|
||
| import { Domain, EntityType } from '@types'; | ||
|
|
||
| const Card = styled(Link)<{ $isShowNavBarRedesign?: boolean }>` | ||
| border-radius: ${(props) => (props.$isShowNavBarRedesign ? '8px' : '10px')}; | ||
| background-color: #ffffff; | ||
| background-color: ${(props) => props.theme.colors.bg}; | ||
| padding: 16px; | ||
| border: ${(props) => (props.$isShowNavBarRedesign ? `1px solid ${colors.gray[100]}` : '2px solid transparent')}; | ||
| border: ${(props) => | ||
| props.$isShowNavBarRedesign ? `1px solid ${props.theme.colors.border}` : '2px solid transparent'}; | ||
|
|
||
| :hover { | ||
| border: ${(props) => (props.$isShowNavBarRedesign ? '1px' : '2px')} solid ${SEARCH_COLORS.LINK_BLUE}; | ||
| border: ${(props) => (props.$isShowNavBarRedesign ? '1px' : '2px')} solid | ||
| ${(props) => props.theme.colors.hyperlinks}; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. borderHover |
||
| } | ||
|
|
||
| display: flex; | ||
|
|
@@ -39,7 +38,7 @@ const Text = styled.div` | |
|
|
||
| const Name = styled.div` | ||
| font-size: 16px; | ||
| color: ${ANTD_GRAY[9]}; | ||
| color: ${(props) => props.theme.colors.text}; | ||
| overflow: hidden; | ||
| text-overflow: ellipsis; | ||
| white-space: nowrap; | ||
|
|
@@ -54,7 +53,7 @@ const Counts = styled.div` | |
|
|
||
| const Count = styled.div` | ||
| font-size: 14px; | ||
| color: ${ANTD_GRAY[7]}; | ||
| color: ${(props) => props.theme.colors.textTertiary}; | ||
| overflow: hidden; | ||
| text-overflow: ellipsis; | ||
| `; | ||
|
|
||
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.
use border* colors