fix(plasma-core): Popover refactoring

salute-developers · Sep 26, 2023 · 2366fa8 · 2366fa8
1 parent 003f4e9
commit 2366fa8
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/packages/plasma-b2c/src/components/Popover/Popover.stories.tsx b/packages/plasma-b2c/src/components/Popover/Popover.stories.tsx
@@ -76,7 +76,7 @@ Live.args = {
     trigger: 'click',
     closeOnOverlayClick: true,
     closeOnEsc: true,
-    isFocusTrapped: false,
+    isFocusTrapped: true,
     skidding: 0,
     distance: 6,
 };

diff --git a/packages/plasma-core/api/plasma-core.api.md b/packages/plasma-core/api/plasma-core.api.md
@@ -818,10 +818,10 @@ export interface PinProps {
 // @public
 export const Popover: React_2.NamedExoticComponent<PopoverProps & React_2.RefAttributes<HTMLDivElement>>;
 
-// Warning: (ae-forgotten-export) The symbol "PopoverBasicPlacement" needs to be exported by the entry point index.d.ts
+// Warning: (ae-forgotten-export) The symbol "PopoverPlacementBasic" needs to be exported by the entry point index.d.ts
 //
 // @public (undocumented)
-export type PopoverPlacement = PopoverBasicPlacement | 'auto';
+export type PopoverPlacement = PopoverPlacementBasic | 'auto';
 
 // @public (undocumented)
 export interface PopoverProps extends HTMLAttributes<HTMLDivElement> {
@@ -833,7 +833,7 @@ export interface PopoverProps extends HTMLAttributes<HTMLDivElement> {
     isOpen?: boolean;
     offset?: [number, number];
     onToggle?: (isOpen: boolean, event: SyntheticEvent | Event) => void;
-    placement?: PopoverPlacement | Array<PopoverBasicPlacement>;
+    placement?: PopoverPlacement | Array<PopoverPlacementBasic>;
     target?: ReactNode;
     trigger: 'hover' | 'click';
 }

diff --git a/packages/plasma-core/src/components/Popover/Popover.tsx b/packages/plasma-core/src/components/Popover/Popover.tsx
@@ -8,8 +8,8 @@ import { useFocusTrap, useForkRef } from '../../hooks';
 
 const ESCAPE_KEYCODE = 27;
 
-export type PopoverBasicPlacement = 'top' | 'bottom' | 'right' | 'left';
-export type PopoverPlacement = PopoverBasicPlacement | 'auto';
+export type PopoverPlacementBasic = 'top' | 'bottom' | 'right' | 'left';
+export type PopoverPlacement = PopoverPlacementBasic | 'auto';
 
 export interface PopoverProps extends HTMLAttributes<HTMLDivElement> {
     /**
@@ -23,7 +23,7 @@ export interface PopoverProps extends HTMLAttributes<HTMLDivElement> {
     /**
      * Расположение всплывающего окна. По умолчанию "auto".
      */
-    placement?: PopoverPlacement | Array<PopoverBasicPlacement>;
+    placement?: PopoverPlacement | Array<PopoverPlacementBasic>;
     /**
      * Отступ окна относительно элемента, у которого оно вызвано.
      */
@@ -41,7 +41,7 @@ export interface PopoverProps extends HTMLAttributes<HTMLDivElement> {
      */
     children?: ReactNode;
     /**
-     * Блокировать ли фокус на всплывающем окне.
+     * Блокировать ли фокус на всплывающем окне(по умолчанию true).
      */
     isFocusTrapped?: boolean;
     /**
@@ -109,7 +109,7 @@ export const Popover = memo<PopoverProps & RefAttributes<HTMLDivElement>>(
                 arrow,
                 placement = 'auto',
                 offset = [0, 0],
-                isFocusTrapped = false,
+                isFocusTrapped = true,
                 closeOnOverlayClick = true,
                 closeOnEsc = true,
                 onToggle,

diff --git a/packages/plasma-web/src/components/Popover/Popover.stories.tsx b/packages/plasma-web/src/components/Popover/Popover.stories.tsx
@@ -76,7 +76,7 @@ Live.args = {
     trigger: 'click',
     closeOnOverlayClick: true,
     closeOnEsc: true,
-    isFocusTrapped: false,
+    isFocusTrapped: true,
     skidding: 0,
     distance: 6,
 };