fix: adjust colors for disabled menu item (#619)

ariser · web-flow · commit 0f2c25ee1780 · 2025-06-20T14:03:44.000+02:00
diff --git a/src/components/GenericMenu.tsx b/src/components/GenericMenu.tsx
@@ -105,8 +105,8 @@ export const GenericMenuItem = styled.div`
       font: ${theme.click.genericMenu.item.typography.label.active};
     }
     &[data-disabled] {
-      background:${theme.click.genericMenu.item.color.background.disabled};
-      color:${theme.click.genericMenu.item.color.text.disabled};
+      // TODO use theme.click.genericMenu.item.color.text.disabled after it's fixed https://github.com/ClickHouse/click-ui/issues/621
+      color:${theme.global.color.text.disabled};
       font: ${theme.click.genericMenu.item.typography.label.disabled};
       pointer-events: none;
     }

Original file line number	Diff line number	Diff line change
@@ -105,8 +105,8 @@ export const GenericMenuItem = styled.div`
`105`	`105`	`font: ${theme.click.genericMenu.item.typography.label.active};`
`106`	`106`	`}`
`107`	`107`	`&[data-disabled] {`
`108`		`- background:${theme.click.genericMenu.item.color.background.disabled};`
`109`		`- color:${theme.click.genericMenu.item.color.text.disabled};`
	`108`	`+ // TODO use theme.click.genericMenu.item.color.text.disabled after it's fixed https://github.com/ClickHouse/click-ui/issues/621`
	`109`	`+ color:${theme.global.color.text.disabled};`
`110`	`110`	`font: ${theme.click.genericMenu.item.typography.label.disabled};`
`111`	`111`	`pointer-events: none;`
`112`	`112`	`}`