diff --git a/src/icon-button.css.ts b/src/icon-button.css.ts index c7b51cc844..b29ebf367d 100644 --- a/src/icon-button.css.ts +++ b/src/icon-button.css.ts @@ -280,24 +280,24 @@ export const iconButtonTokens = styleVariants({ 'danger-transparent-default': { vars: { [background]: 'transparent', - [backgroundHover]: skinVars.colors.backgroundContainerHover, - [backgroundActive]: skinVars.colors.backgroundContainerPressed, + [backgroundHover]: skinVars.colors.buttonLinkDangerBackgroundPressed, + [backgroundActive]: skinVars.colors.buttonLinkDangerBackgroundPressed, [iconColor]: skinVars.colors.error, }, }, 'danger-transparent-brand': { vars: { [background]: skinVars.colors.buttonLinkDangerBackgroundBrand, - [backgroundHover]: skinVars.colors.backgroundContainerHover, - [backgroundActive]: skinVars.colors.backgroundContainerPressed, + [backgroundHover]: skinVars.colors.buttonLinkDangerBackgroundBrandPressed, + [backgroundActive]: skinVars.colors.buttonLinkDangerBackgroundBrandPressed, [iconColor]: skinVars.colors.error, }, }, 'danger-transparent-negative': { vars: { [background]: skinVars.colors.buttonLinkDangerBackgroundBrand, - [backgroundHover]: skinVars.colors.backgroundContainerHover, - [backgroundActive]: skinVars.colors.backgroundContainerPressed, + [backgroundHover]: skinVars.colors.buttonLinkDangerBackgroundNegativePressed, + [backgroundActive]: skinVars.colors.buttonLinkDangerBackgroundNegativePressed, [iconColor]: skinVars.colors.error, }, }, diff --git a/src/menu.css.ts b/src/menu.css.ts index c9cd9bd05e..8f3d578459 100644 --- a/src/menu.css.ts +++ b/src/menu.css.ts @@ -95,6 +95,20 @@ export const menuItemHovered = style({ }, }); +export const menuItemHoveredDestructive = style({ + transition: 'background-color 0.1s ease-in-out', + backgroundColor: skinVars.colors.buttonLinkDangerBackgroundPressed, + ':active': { + backgroundColor: skinVars.colors.buttonLinkDangerBackgroundPressed, + }, + '@media': { + [mq.touchableOnly]: { + backgroundColor: 'transparent', + transition: 'none', + }, + }, +}); + export const menuItemEnabled = style({ '@media': { [mq.touchableOnly]: { diff --git a/src/menu.tsx b/src/menu.tsx index 00a86f5865..f85ae2cc65 100644 --- a/src/menu.tsx +++ b/src/menu.tsx @@ -159,7 +159,10 @@ export const MenuItem = ({ className={classnames(styles.menuItem, { [styles.menuItemEnabled]: !disabled, [styles.menuItemDisabled]: disabled, - [styles.menuItemHovered]: !disabled && itemIndex !== null && focusedItem === itemIndex, + [styles.menuItemHovered]: + !disabled && !destructive && itemIndex !== null && focusedItem === itemIndex, + [styles.menuItemHoveredDestructive]: + !disabled && destructive && itemIndex !== null && focusedItem === itemIndex, })} onMouseMove={() => setFocusedItem(disabled ? null : itemIndex)} onMouseLeave={() => setFocusedItem(null)}