From 186ac3431757bcace7fa24a2b921e2fba28522e1 Mon Sep 17 00:00:00 2001
From: GermanVor <vorotnikov.german@yandex.ru>
Date: Thu, 23 May 2024 00:21:23 +0200
Subject: [PATCH] chore: selectId

---
 src/components/Select/Select.tsx                     |  4 +---
 .../components/SelectControl/SelectControl.tsx       |  4 +---
 src/components/Select/types.ts                       |  1 -
 src/components/TreeSelect/TreeSelect.tsx             | 12 +++++++-----
 4 files changed, 9 insertions(+), 12 deletions(-)

diff --git a/src/components/Select/Select.tsx b/src/components/Select/Select.tsx
index 60e050a0d0..29560533e8 100644
--- a/src/components/Select/Select.tsx
+++ b/src/components/Select/Select.tsx
@@ -283,8 +283,7 @@ export const Select = React.forwardRef<HTMLButtonElement, SelectProps>(function
     };
 
     const uniqId = useUniqId();
-    const controlId = id ?? uniqId;
-    const selectId = `select-${controlId}`;
+    const selectId = id ?? uniqId;
     const popupId = `select-popup-${selectId}`;
 
     const _renderList = () => {
@@ -343,7 +342,6 @@ export const Select = React.forwardRef<HTMLButtonElement, SelectProps>(function
                 renderControl={renderControl}
                 value={value}
                 popupId={popupId}
-                id={controlId}
                 selectId={selectId}
                 activeIndex={activeIndex}
                 hasCounter={multiple && hasCounter}
diff --git a/src/components/Select/components/SelectControl/SelectControl.tsx b/src/components/Select/components/SelectControl/SelectControl.tsx
index 4d68b39252..e817f23993 100644
--- a/src/components/Select/components/SelectControl/SelectControl.tsx
+++ b/src/components/Select/components/SelectControl/SelectControl.tsx
@@ -71,7 +71,6 @@ export const SelectControl = React.forwardRef<HTMLButtonElement, ControlProps>((
         renderCounter,
         hasCounter,
         title,
-        id,
     } = props;
     const showOptionsText = Boolean(selectedOptionsContent);
     const showPlaceholder = Boolean(placeholder && !showOptionsText);
@@ -165,7 +164,6 @@ export const SelectControl = React.forwardRef<HTMLButtonElement, ControlProps>((
                 popupId,
                 selectId,
                 activeIndex,
-                id,
             },
             {value},
         );
@@ -175,7 +173,7 @@ export const SelectControl = React.forwardRef<HTMLButtonElement, ControlProps>((
         <React.Fragment>
             <div className={selectControlBlock(controlMods)} role="group">
                 <button
-                    id={id}
+                    id={selectId}
                     ref={ref}
                     role="combobox"
                     aria-controls={popupId}
diff --git a/src/components/Select/types.ts b/src/components/Select/types.ts
index 0045c41ec0..9c589fe9a7 100644
--- a/src/components/Select/types.ts
+++ b/src/components/Select/types.ts
@@ -21,7 +21,6 @@ export type SelectRenderControlProps = {
     open: boolean;
     popupId: string;
     selectId: string;
-    id?: string;
     activeIndex?: number;
 };
 export type SelectRenderControlOptions = {
diff --git a/src/components/TreeSelect/TreeSelect.tsx b/src/components/TreeSelect/TreeSelect.tsx
index 9a9f0d4cfe..c947d888e2 100644
--- a/src/components/TreeSelect/TreeSelect.tsx
+++ b/src/components/TreeSelect/TreeSelect.tsx
@@ -64,8 +64,6 @@ export const TreeSelect = React.forwardRef(function TreeSelect<T>(
     ref: React.Ref<HTMLButtonElement>,
 ) {
     const mobile = useMobile();
-    const uniqId = useUniqId();
-    const treeSelectId = id ?? uniqId;
 
     const controlWrapRef = React.useRef<HTMLDivElement>(null);
     const controlRef = React.useRef<HTMLElement>(null);
@@ -175,6 +173,10 @@ export const TreeSelect = React.forwardRef(function TreeSelect<T>(
 
     const handleClose = React.useCallback(() => toggleOpen(false), [toggleOpen]);
 
+    const uniqId = useUniqId();
+    const treeSelectId = id ?? uniqId;
+    const popupId = `tree-select-popup-${treeSelectId}`;
+
     const controlProps: TreeSelectRenderControlProps = {
         open,
         toggleOpen,
@@ -197,8 +199,8 @@ export const TreeSelect = React.forwardRef(function TreeSelect<T>(
             ).join(', ')}
             view="normal"
             pin="round-round"
-            popupId={`tree-select-popup-${treeSelectId}`}
-            selectId={`tree-select-${treeSelectId}`}
+            popupId={popupId}
+            selectId={treeSelectId}
         />
     );
 
@@ -231,7 +233,7 @@ export const TreeSelect = React.forwardRef(function TreeSelect<T>(
                 handleClose={handleClose}
                 disablePortal={popupDisablePortal}
                 mobile={mobile}
-                id={`tree-select-popup-${treeSelectId}`}
+                id={popupId}
             >
                 {slotBeforeListBody}