From 5f409125c60cf183527dda52380a89b892f036c0 Mon Sep 17 00:00:00 2001
From: Alexey <sudilovskiy@yandex-team.ru>
Date: Mon, 11 Nov 2024 14:32:00 +0100
Subject: [PATCH 1/3] fix(Popover): use visible title for popup a11y label

---
 src/components/Popover/Popover.tsx | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/components/Popover/Popover.tsx b/src/components/Popover/Popover.tsx
index 8461e5e836..92f25d0da6 100644
--- a/src/components/Popover/Popover.tsx
+++ b/src/components/Popover/Popover.tsx
@@ -4,6 +4,7 @@ import React from 'react';
 
 import {Xmark} from '@gravity-ui/icons';
 
+import {useUniqId} from '../../hooks/useUniqId'
 import {Button} from '../Button';
 import {Icon} from '../Icon';
 import type {PopupPlacement} from '../Popup';
@@ -115,6 +116,8 @@ export const Popover = React.forwardRef<PopoverInstanceProps, PopoverProps & QAP
 
     const hasTitle = Boolean(title);
 
+    const popoverTitleId = `popover-${tooltipId ?? ''}-title-${useUniqId()}`;
+
     const tooltip = (
         <Popup
             id={tooltipId}
@@ -144,9 +147,10 @@ export const Popover = React.forwardRef<PopoverInstanceProps, PopoverProps & QAP
             restoreFocus={true}
             restoreFocusRef={restoreFocusRef || controlRef}
             modifiers={modifiers}
+            aria-labelledby={title ? popoverTitleId : undefined}
         >
             <React.Fragment>
-                {title && <h3 className={cnPopover('tooltip-title')}>{title}</h3>}
+                {title && <h3 id={popoverTitleId} className={cnPopover('tooltip-title')}>{title}</h3>}
                 <Content
                     secondary={hasTitle ? theme !== 'announcement' : false}
                     content={content}

From f4b312a21ffc02a0c454498181cb670689979020 Mon Sep 17 00:00:00 2001
From: Alexey <sudilovskiy@yandex-team.ru>
Date: Mon, 11 Nov 2024 14:38:10 +0100
Subject: [PATCH 2/3] chore: lint

---
 src/components/Popover/Popover.tsx | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/components/Popover/Popover.tsx b/src/components/Popover/Popover.tsx
index 92f25d0da6..437b6a931a 100644
--- a/src/components/Popover/Popover.tsx
+++ b/src/components/Popover/Popover.tsx
@@ -4,7 +4,7 @@ import React from 'react';
 
 import {Xmark} from '@gravity-ui/icons';
 
-import {useUniqId} from '../../hooks/useUniqId'
+import {useUniqId} from '../../hooks/useUniqId';
 import {Button} from '../Button';
 import {Icon} from '../Icon';
 import type {PopupPlacement} from '../Popup';
@@ -150,7 +150,9 @@ export const Popover = React.forwardRef<PopoverInstanceProps, PopoverProps & QAP
             aria-labelledby={title ? popoverTitleId : undefined}
         >
             <React.Fragment>
-                {title && <h3 id={popoverTitleId} className={cnPopover('tooltip-title')}>{title}</h3>}
+                {title && (
+                    <h3 id={popoverTitleId} className={cnPopover('tooltip-title')}>{title}</h3>
+                )}
                 <Content
                     secondary={hasTitle ? theme !== 'announcement' : false}
                     content={content}

From 512c3b0a15edc404dc184c57b4ccb4b11109e87a Mon Sep 17 00:00:00 2001
From: Alexey <sudilovskiy@yandex-team.ru>
Date: Mon, 11 Nov 2024 14:41:33 +0100
Subject: [PATCH 3/3] chore: lint

---
 src/components/Popover/Popover.tsx | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/components/Popover/Popover.tsx b/src/components/Popover/Popover.tsx
index 437b6a931a..d6d7087be9 100644
--- a/src/components/Popover/Popover.tsx
+++ b/src/components/Popover/Popover.tsx
@@ -151,7 +151,9 @@ export const Popover = React.forwardRef<PopoverInstanceProps, PopoverProps & QAP
         >
             <React.Fragment>
                 {title && (
-                    <h3 id={popoverTitleId} className={cnPopover('tooltip-title')}>{title}</h3>
+                    <h3 id={popoverTitleId} className={cnPopover('tooltip-title')}>
+                        {title}
+                    </h3>
                 )}
                 <Content
                     secondary={hasTitle ? theme !== 'announcement' : false}