Skip to content

Commit 6c42a37

Browse files
Merge pull request #2345 from dxc-technology/jialecl/remove-default-icon
Remove default icon in baseMenu in a certain scenario
2 parents dc43c65 + 27ad641 commit 6c42a37

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

packages/lib/src/base-menu/ItemAction.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ const ItemAction = memo(
114114
<Icon>{collapseIcon}</Icon>
115115
</Control>
116116
)}
117-
{(icon || hasPopOver) && (
117+
{(((icon || hasPopOver) && !isHorizontal) || (isHorizontal && icon)) && (
118118
<TooltipWrapper condition={hasPopOver} label={label}>
119119
<Icon>
120120
{typeof icon === "string" ? <DxcIcon icon={icon} /> : icon ? icon : <DxcIcon icon="topic" />}

0 commit comments

Comments
 (0)