From ea3f09b6a96b8a6bdf5c12905aa080aff1e8ee55 Mon Sep 17 00:00:00 2001
From: kseniyakuzina <kseniyakuzina@yandex-team.ru>
Date: Sat, 3 Feb 2024 17:34:36 +0300
Subject: [PATCH] fix(MenuItem): fix non interactive MenuItem style

---
 src/components/Menu/Menu.scss                 | 12 +++-
 src/components/Menu/MenuItem.tsx              |  6 +-
 .../Menu/__stories__/Menu.stories.tsx         | 68 ++++++++++++-------
 3 files changed, 58 insertions(+), 28 deletions(-)

diff --git a/src/components/Menu/Menu.scss b/src/components/Menu/Menu.scss
index ccab5a0a71..1b70eb18f7 100644
--- a/src/components/Menu/Menu.scss
+++ b/src/components/Menu/Menu.scss
@@ -26,7 +26,6 @@ $block: '.#{variables.$ns}menu';
     &__item {
         display: flex;
         align-items: center;
-        cursor: pointer;
         color: var(--g-color-text-primary);
         text-decoration: none;
         outline: none;
@@ -47,8 +46,15 @@ $block: '.#{variables.$ns}menu';
             min-width: 0;
         }
 
-        &:hover,
-        &:focus,
+        &_interactive {
+            cursor: pointer;
+
+            &:hover,
+            &:focus {
+                background-color: var(--g-color-base-simple-hover);
+            }
+        }
+
         &_selected {
             background-color: var(--g-color-base-simple-hover);
         }
diff --git a/src/components/Menu/MenuItem.tsx b/src/components/Menu/MenuItem.tsx
index 0e1a10a494..a3336fad69 100644
--- a/src/components/Menu/MenuItem.tsx
+++ b/src/components/Menu/MenuItem.tsx
@@ -70,7 +70,11 @@ export const MenuItem = React.forwardRef<HTMLElement, MenuItemProps>(function Me
         onClickCapture: disabled ? undefined : handleClickCapture,
         style,
         tabIndex: disabled ? -1 : 0,
-        className: b('item', {disabled, active, selected, theme}, className),
+        className: b(
+            'item',
+            {disabled, active, selected, theme, interactive: Boolean(onClick)},
+            className,
+        ),
         'data-qa': qa,
     };
     const content = [
diff --git a/src/components/Menu/__stories__/Menu.stories.tsx b/src/components/Menu/__stories__/Menu.stories.tsx
index 8fea81f3df..3de91fa9f3 100644
--- a/src/components/Menu/__stories__/Menu.stories.tsx
+++ b/src/components/Menu/__stories__/Menu.stories.tsx
@@ -1,6 +1,7 @@
 import React from 'react';
 
 import {CircleExclamationFill, Gear} from '@gravity-ui/icons';
+import {action} from '@storybook/addon-actions';
 import type {Meta, StoryFn} from '@storybook/react';
 
 import {Icon} from '../../Icon';
@@ -12,24 +13,36 @@ export default {
     component: Menu,
 } as Meta<typeof Menu>;
 
+const createItemClickHandler = (text: string) => () => {
+    action(`Click ${text} item`);
+    alert(`${text} item clicked`);
+};
+
 export const Default: StoryFn<MenuProps> = (args) => (
     <Menu {...args}>
-        <Menu.Item>First</Menu.Item>
-        <Menu.Item>Second</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('First')}>First</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('Second')}>Second</Menu.Item>
     </Menu>
 );
 
 export const ItemActive: StoryFn<MenuProps> = (args) => (
     <Menu {...args}>
-        <Menu.Item active>First</Menu.Item>
-        <Menu.Item>Second</Menu.Item>
-        <Menu.Item>Third</Menu.Item>
+        <Menu.Item active onClick={createItemClickHandler('First')}>
+            First
+        </Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('Second')}>Second</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('Third')}>Third</Menu.Item>
     </Menu>
 );
 
 export const ItemIcon: StoryFn<MenuProps> = (args) => (
     <Menu {...args}>
-        <Menu.Item iconStart={<Icon data={Gear} size={16} />}>Settings</Menu.Item>
+        <Menu.Item
+            iconStart={<Icon data={Gear} size={16} />}
+            onClick={createItemClickHandler('Settings')}
+        >
+            Settings
+        </Menu.Item>
     </Menu>
 );
 
@@ -38,6 +51,7 @@ export const ItemBothIcons: StoryFn<MenuProps> = (args) => (
         <Menu.Item
             iconStart={<Icon data={Gear} size={16} />}
             iconEnd={<Icon data={CircleExclamationFill} size={16} />}
+            onClick={createItemClickHandler('Settings')}
         >
             Settings
         </Menu.Item>
@@ -46,25 +60,31 @@ export const ItemBothIcons: StoryFn<MenuProps> = (args) => (
 
 export const ItemDisabled: StoryFn<MenuProps> = (args) => (
     <Menu {...args}>
-        <Menu.Item>First</Menu.Item>
-        <Menu.Item disabled>Second (unavailable)</Menu.Item>
-        <Menu.Item>Third</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('First')}>First</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('Second')} disabled>
+            Second (unavailable)
+        </Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('Third')}>Third</Menu.Item>
     </Menu>
 );
 
 export const ItemSelected: StoryFn<MenuProps> = (args) => (
     <Menu {...args}>
-        <Menu.Item>First</Menu.Item>
-        <Menu.Item>Second</Menu.Item>
-        <Menu.Item selected>Third</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('First')}>First</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('Second')}>Second</Menu.Item>
+        <Menu.Item selected onClick={createItemClickHandler('Third')}>
+            Third
+        </Menu.Item>
     </Menu>
 );
 
 export const ItemTheme: StoryFn<MenuProps> = (args) => (
     <Menu {...args}>
-        <Menu.Item>Normal</Menu.Item>
-        <Menu.Item theme="danger">Danger</Menu.Item>
-        <Menu.Item theme="danger" disabled>
+        <Menu.Item onClick={createItemClickHandler('Normal')}>Normal</Menu.Item>
+        <Menu.Item theme="danger" onClick={createItemClickHandler('Danger')}>
+            Danger
+        </Menu.Item>
+        <Menu.Item theme="danger" disabled onClick={createItemClickHandler('Danger (disabled)')}>
             Danger (disabled)
         </Menu.Item>
     </Menu>
@@ -86,20 +106,20 @@ export const ItemLink: StoryFn<MenuProps> = (args) => (
 
 export const Group: StoryFn<MenuProps> = (args) => (
     <Menu {...args}>
-        <Menu.Item>First</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('First')}>First</Menu.Item>
         <Menu.Group label="Group One">
-            <Menu.Item>One</Menu.Item>
-            <Menu.Item>Two</Menu.Item>
+            <Menu.Item onClick={createItemClickHandler('Group One: One')}>One</Menu.Item>
+            <Menu.Item onClick={createItemClickHandler('Group One: Two')}>Two</Menu.Item>
         </Menu.Group>
         <Menu.Group label="Group Two">
-            <Menu.Item>One</Menu.Item>
-            <Menu.Item>Two</Menu.Item>
+            <Menu.Item onClick={createItemClickHandler('Group Two: One')}>One</Menu.Item>
+            <Menu.Item onClick={createItemClickHandler('Group Two: Two')}>Two</Menu.Item>
         </Menu.Group>
-        <Menu.Item>Middle</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('Middle')}>Middle</Menu.Item>
         <Menu.Group label="Group Three">
-            <Menu.Item>One</Menu.Item>
-            <Menu.Item>Two</Menu.Item>
+            <Menu.Item onClick={createItemClickHandler('Group Three: One')}>One</Menu.Item>
+            <Menu.Item onClick={createItemClickHandler('Group Three: Two')}>Two</Menu.Item>
         </Menu.Group>
-        <Menu.Item>Last</Menu.Item>
+        <Menu.Item onClick={createItemClickHandler('Last')}>Last</Menu.Item>
     </Menu>
 );