diff --git a/.changeset/famous-olives-try.md b/.changeset/famous-olives-try.md new file mode 100644 index 0000000000..3150197203 --- /dev/null +++ b/.changeset/famous-olives-try.md @@ -0,0 +1,5 @@ +--- +'@backstage/plugin-scaffolder': patch +--- + +Fix some hard-coded white font colors in scaffolder diff --git a/plugins/scaffolder/src/components/TemplateCard/TemplateCard.tsx b/plugins/scaffolder/src/components/TemplateCard/TemplateCard.tsx index 825317ef73..d35636f507 100644 --- a/plugins/scaffolder/src/components/TemplateCard/TemplateCard.tsx +++ b/plugins/scaffolder/src/components/TemplateCard/TemplateCard.tsx @@ -107,7 +107,7 @@ const useStyles = makeStyles< top: theme.spacing(0.5), right: theme.spacing(0.5), padding: '0.25rem', - color: theme.palette.common.white, + color: ({ fontColor }) => fontColor, }, })); diff --git a/plugins/scaffolder/src/next/OngoingTask/ContextMenu.tsx b/plugins/scaffolder/src/next/OngoingTask/ContextMenu.tsx index 985dd98903..21db9f5b08 100644 --- a/plugins/scaffolder/src/next/OngoingTask/ContextMenu.tsx +++ b/plugins/scaffolder/src/next/OngoingTask/ContextMenu.tsx @@ -22,6 +22,7 @@ import { MenuItem, MenuList, Popover, + useTheme, } from '@material-ui/core'; import { useAsync } from '@react-hookz/web'; import Cancel from '@material-ui/icons/Cancel'; @@ -40,16 +41,18 @@ type ContextMenuProps = { taskId?: string; }; -const useStyles = makeStyles((theme: BackstageTheme) => ({ +const useStyles = makeStyles(() => ({ button: { - color: theme.palette.common.white, + color: ({ fontColor }) => fontColor, }, })); export const ContextMenu = (props: ContextMenuProps) => { const { cancelEnabled, logsVisible, onStartOver, onToggleLogs, taskId } = props; - const classes = useStyles(); + const { getPageTheme } = useTheme(); + const pageTheme = getPageTheme({ themeId: 'website' }); + const classes = useStyles({ fontColor: pageTheme.fontColor }); const scaffolderApi = useApi(scaffolderApiRef); const [anchorEl, setAnchorEl] = useState(); @@ -69,7 +72,6 @@ export const ContextMenu = (props: ContextMenuProps) => { setAnchorEl(event.currentTarget); }} data-testid="menu-button" - color="inherit" className={classes.button} >