Skip to content

feat(ui): update colors to semantic tokens in home and settings folders#16877

Merged
purnimagarg1 merged 11 commits into
masterfrom
pg--cat-1661-update-tokens-home-settings
Apr 13, 2026
Merged

feat(ui): update colors to semantic tokens in home and settings folders#16877
purnimagarg1 merged 11 commits into
masterfrom
pg--cat-1661-update-tokens-home-settings

Conversation

@purnimagarg1
Copy link
Copy Markdown
Collaborator

@purnimagarg1 purnimagarg1 commented Apr 1, 2026

Linear ticket:
https://linear.app/acryl-data/issue/CAT-1661/update-colors-to-semantic-tokens-in-settings-and-home-folders

Description:

Updates colors to semantic tokens in the following folders:

  • homeV3
  • homeV2
  • settingsV2

@github-actions github-actions Bot added the product PR or Issue related to the DataHub UI/UX label Apr 1, 2026
@purnimagarg1 purnimagarg1 changed the title feat(ui): update colors to semantic tokens in homeV3 folder feat(ui): update colors to semantic tokens in home and settings folders Apr 1, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 1, 2026

Bundle Report

Changes will increase total bundle size by 1.99kB (0.01%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 22.74MB 1.99kB (0.01%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js 1.99kB 12.48MB 0.02%

Files in assets/index-*.js:

  • ./src/app/homeV2/content/tabs/discovery/sections/insight/cards/PopularGlossaryTerms.tsx → Total Size: 3.72kB

  • ./src/app/homeV2/content/tabs/discovery/sections/Section.tsx → Total Size: 1.44kB

  • ./src/app/homeV2/introduce/IntroduceYourselfLeftSidebar.tsx → Total Size: 1.49kB

  • ./src/app/homeV2/layout/navBarRedesign/NavBarMenuItemGroup.tsx → Total Size: 955 bytes

  • ./src/app/homeV2/action/announcement/Announcement.tsx → Total Size: 1.41kB

  • ./src/app/homeV2/content/tabs/discovery/sections/dataProducts/DataProductCard.tsx → Total Size: 2.43kB

  • ./src/app/homeV2/layout/CustomNavLink.tsx → Total Size: 2.18kB

  • ./src/app/homeV2/action/announcement/Announcements.tsx → Total Size: 3.42kB

  • ./src/app/homeV2/content/tabs/discovery/sections/platform/PlatformCard.tsx → Total Size: 2.43kB

  • ./src/app/homeV2/content/tabs/discovery/sections/domains/DomainCard.tsx → Total Size: 2.7kB

  • ./src/app/homeV2/introduce/IntroduceYourself.tsx → Total Size: 607 bytes

  • ./src/app/homeV2/content/recent/EntityCardList.tsx → Total Size: 1.71kB

  • ./src/app/homeV2/content/recent/EntityCard.tsx → Total Size: 2.44kB

  • ./src/app/homeV2/layout/CenterContent.tsx → Total Size: 1.65kB

  • ./src/app/homeV2/content/tabs/discovery/sections/onboarding/OnboardingCards.tsx → Total Size: 2.05kB

  • ./src/app/homeV2/introduce/IntroduceYourselfMainContent.tsx → Total Size: 12.53kB

  • ./src/app/homeV2/layout/NavLinks.tsx → Total Size: 348 bytes

  • ./src/app/homeV2/layout/NavLinksMenu.tsx → Total Size: 7.88kB

  • ./src/app/homeV2/content/recent/RecentActions.tsx → Total Size: 1.26kB

  • ./src/app/homeV2/content/tabs/announcements/AnnouncementCard.tsx → Total Size: 1.51kB

  • ./src/app/homeV2/layout/LeftSidebar.tsx → Total Size: 4.0kB

  • ./src/app/homeV2/content/tabs/discovery/sections/insight/shared/InsightCard.tsx → Total Size: 1.32kB

  • ./src/app/homeV2/content/tabs/discovery/sections/insight/cards/SearchListInsightCard.tsx → Total Size: 3.25kB

  • ./src/app/homeV2/content/tabs/discovery/sections/insight/shared/InsightCardSkeleton.tsx → Total Size: 2.94kB

  • ./src/app/homeV2/layout/navBarRedesign/NavBarMenuItem.tsx → Total Size: 3.98kB

  • ./src/app/homeV2/layout/RightSidebar.tsx → Total Size: 1.47kB

  • ./src/app/homeV2/content/HorizontalListSkeletons.tsx → Total Size: 1.41kB

  • ./src/app/homeV2/layout/navBarRedesign/NavBarToggler.tsx → Total Size: 967 bytes

  • ./src/app/homeV2/layout/navBarRedesign/NavSidebar.tsx → Total Size: 13.31kB

  • ./src/app/homeV2/layout/navBarRedesign/NavBarMenuItemDropdown.tsx → Total Size: 3.43kB

@alwaysmeticulous
Copy link
Copy Markdown

alwaysmeticulous Bot commented Apr 1, 2026

✅ Meticulous spotted visual differences in 460 of 1446 screens tested, but all differences have already been approved: view differences detected.

Meticulous evaluated ~9 hours of user flows against your PR.

Last updated for commit 8bd57ea Merge branch 'master' into pg--cat-1661-update-tokens-home-settings. This comment will update as new commits are pushed.

@purnimagarg1 purnimagarg1 marked this pull request as ready for review April 6, 2026 11:14
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 6, 2026

Linear: CAT-1733

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@maggiehays maggiehays added the needs-review Label for PRs that need review from a maintainer. label Apr 6, 2026
Copy link
Copy Markdown
Contributor

@v-tarasevich-blitz-brain v-tarasevich-blitz-brain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me
Nothing blocking, just a few suggestions/questions


const Card = styled.div`
border: 1px solid ${ANTD_GRAY[4]};
border: 1px solid ${(props) => props.theme.colors.bgSurface};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use border* colors


:hover {
border: 1.5px solid ${SEARCH_COLORS.LINK_BLUE};
border: 1.5px solid ${(props) => props.theme.colors.hyperlinks};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

borderBrand

const SubHeader = styled.div`
font-size: 14px;
color: ${ANTD_GRAY[5]};
color: ${(props) => props.theme.colors.border};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's try with textTertiary


> .slick-dots li button {
background-color: #d9d9d9;
background-color: ${(props) => props.theme.colors.border};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we have some more suitable button* token?


const Card = styled.div`
border: 1px solid ${ANTD_GRAY[4]};
border: 1px solid ${(props) => props.theme.colors.bgHover};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we use border?


:hover {
${(props) => props.showHover && 'background-color: #f5f7fa;'}
${(props) => props.showHover && `background-color: ${props.theme.colors.bgSurface};`}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bgHover

@@ -58,7 +59,7 @@ export default function LinkModule(props: ModuleProps) {
ellipsis={{
tooltip: {
color: 'white',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps we can update this white too


justify-content: center;
background: ${colors.gray[1600]};
background: ${(props) => props.theme.colors.bgSurfaceNewNav};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we use another tokens not related to new nav?

margin: 5px;
border: 1px solid ${colors.gray[100]};
border-radius: 12px;
position: relative;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it safe to remove?

border-radius: 12px;
background-color: ${colors.white};
background-color: ${(props) => props.theme.colors.bg};
${(props) => props.theme.id === 'themeV2Dark' && 'opacity: 0;'}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what do you think if we change it to theme.id !== themeV2Light? I think this background supported only in light theme

@purnimagarg1 purnimagarg1 merged commit 791577c into master Apr 13, 2026
44 checks passed
@purnimagarg1 purnimagarg1 deleted the pg--cat-1661-update-tokens-home-settings branch April 13, 2026 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-review Label for PRs that need review from a maintainer. product PR or Issue related to the DataHub UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants