@centreon/ui 24.9.4 → 24.9.6

package/src/Panel/index.tsx

@@ -1,18 +1,16 @@
+/* eslint-disable react/no-unused-prop-types */
+// There is an issue about using forwardRef: https://github.com/yannickcr/eslint-plugin-react/issues/2760
+
+import * as React from 'react';
+
 import { isEmpty, isNil } from 'ramda';
 import { makeStyles } from 'tss-react/mui';
 
+import { Paper, Slide, Divider, AppBar, Tabs } from '@mui/material';
 import IconClose from '@mui/icons-material/Clear';
-import { AppBar, Divider, Paper, Slide, Tabs } from '@mui/material';
 
-import { IconButton } from '..';
+import IconButton from '../Button/Icon';
 
-import {
-  ReactElement,
-  RefObject,
-  forwardRef,
-  useCallback,
-  useEffect
-} from 'react';
 import { minTabHeight } from './Tab';
 
 interface StylesProps extends Pick<Props, 'headerBackgroundColor' | 'width'> {
@@ -35,7 +33,7 @@ const useStyles = makeStyles<StylesProps>()(
       height: '100%'
     },
     container: {
-      backgroundColor: theme.palette.background.panel,
+      backgroundColor: theme.palette.background.default,
       borderTopRightRadius: 0,
       display: 'grid',
       gridTemplate: 'auto auto 1fr / 1fr',
@@ -87,21 +85,20 @@ export interface Tab {
 }
 
 export interface Props {
-  className?: string;
-  header?: ReactElement;
+  header: React.ReactElement;
   headerBackgroundColor?: string;
   labelClose?: string;
   minWidth?: number;
   onClose?: () => void;
   onResize?: (newWidth: number) => void;
   onTabSelect?: (event, id: number) => void;
-  selectedTab: ReactElement;
+  selectedTab: React.ReactElement;
   selectedTabId?: number;
   tabs?: Array<JSX.Element>;
   width?: number;
 }
 
-const Panel = forwardRef(
+const Panel = React.forwardRef(
   (
     {
       header,
@@ -114,12 +111,11 @@ const Panel = forwardRef(
       width = 550,
       minWidth = 550,
       headerBackgroundColor,
-      onResize,
-      className
+      onResize
     }: Props,
     ref
   ): JSX.Element => {
-    const { classes, cx } = useStyles({
+    const { classes } = useStyles({
       hasTabs: !isEmpty(tabs),
       headerBackgroundColor,
       width
@@ -135,7 +131,7 @@ const Panel = forwardRef(
       }
     };
 
-    useEffect(() => {
+    React.useEffect(() => {
       window.addEventListener('resize', resizeWindow);
 
       return (): void => {
@@ -153,7 +149,7 @@ const Panel = forwardRef(
       document.removeEventListener('mousemove', moveMouse, true);
     };
 
-    const moveMouse = useCallback((e) => {
+    const moveMouse = React.useCallback((e) => {
       e.preventDefault();
 
       const maxWidth = getMaxWidth();
@@ -183,9 +179,9 @@ const Panel = forwardRef(
           exit: 50
         }}
       >
-        <Paper className={cx(classes.container, className)} elevation={2}>
+        <Paper className={classes.container} elevation={2}>
           {!isNil(onResize) && (
-            <div className={classes.dragger} onMouseDown={resize} />
+            <div className={classes.dragger} role="none" onMouseDown={resize} />
           )}
           {header && (
             <>
@@ -226,11 +222,9 @@ const Panel = forwardRef(
             </AppBar>
             <div
               className={classes.contentContainer}
-              ref={ref as RefObject<HTMLDivElement>}
+              ref={ref as React.RefObject<HTMLDivElement>}
             >
-              <div className={classes.content} id="panel-content">
-                {selectedTab}
-              </div>
+              <div className={classes.content}>{selectedTab}</div>
             </div>
           </div>
         </Paper>

package/src/PopoverMenu/index.tsx

@@ -1,4 +1,4 @@
-import { type Dispatch, type SetStateAction, useEffect, useState } from 'react';
+import { useState, useEffect } from 'react';
 
 import { makeStyles } from 'tss-react/mui';
 
@@ -6,38 +6,28 @@ import {
   ClickAwayListener,
   Paper,
   Popper,
-  type PopperPlacementType
+  PopperPlacementType,
+  useTheme
 } from '@mui/material';
-import type { PopperProps } from '@mui/material/Popper';
 
 import { IconButton } from '..';
 
-const useStyles = makeStyles()((theme) => ({
-  popover: {
-    zIndex: theme.zIndex.tooltip
-  },
+const useStyles = makeStyles()(() => ({
   popoverIconButton: {
     padding: 0,
     width: '100%'
   }
 }));
 
-interface PopoverData {
-  anchorEl: HTMLElement | undefined;
-  setAnchorEl: Dispatch<SetStateAction<HTMLElement | undefined>>;
-}
-
 interface Props {
   canOpen?: boolean;
   children: (props?) => JSX.Element;
   className?: string;
   dataTestId?: string;
-  getPopoverData?: (data: PopoverData) => void;
   icon: JSX.Element;
   onClose?: () => void;
   onOpen?: () => void;
   popperPlacement?: PopperPlacementType;
-  popperProps?: Partial<PopperProps>;
   title?: string;
 }
 
@@ -50,20 +40,20 @@ const PopoverMenu = ({
   onClose,
   canOpen = true,
   className,
-  dataTestId,
-  getPopoverData,
-  popperProps
+  dataTestId
 }: Props): JSX.Element => {
+  const theme = useTheme();
   const { classes, cx } = useStyles();
   const [anchorEl, setAnchorEl] = useState<HTMLElement | undefined>();
+
   const isOpen = Boolean(anchorEl);
 
   const close = (reason?): void => {
     const isClosedByInputClick = reason?.type === 'mousedown';
+
     if (isClosedByInputClick) {
       return;
     }
-
     onClose?.();
     setAnchorEl(undefined);
   };
@@ -85,10 +75,6 @@ const PopoverMenu = ({
     }
   }, [canOpen]);
 
-  useEffect(() => {
-    getPopoverData?.({ anchorEl, setAnchorEl });
-  }, [anchorEl]);
-
   return (
     <>
       <IconButton
@@ -106,12 +92,11 @@ const PopoverMenu = ({
           <Popper
             open
             anchorEl={anchorEl}
-            className={classes.popover}
             nonce={undefined}
             placement={popperPlacement}
+            style={{ zIndex: theme.zIndex.tooltip }}
             onResize={(): undefined => undefined}
             onResizeCapture={(): undefined => undefined}
-            {...popperProps}
           >
             <Paper>{children({ close })}</Paper>
           </Popper>

package/src/RichTextEditor/ContentEditable.tsx

@@ -1,195 +1,149 @@
-import { useCallback, useEffect, useLayoutEffect, useState } from 'react';
-
-import { useLexicalComposerContext } from '@lexical/react/LexicalComposerContext';
-import { isEmpty, isNil } from 'ramda';
-import { useTranslation } from 'react-i18next';
-import { makeStyles } from 'tss-react/mui';
-
-import { Typography } from '@mui/material';
-
-interface StyleProps {
-  editable: boolean;
-  error?: string;
-  minInputHeight: number;
-}
-
-const useStyles = makeStyles<StyleProps>()(
-  (theme, { minInputHeight, editable, error }) => ({
-    container: {
-      backgroundColor: theme.palette.background.paper,
-      border: error
-        ? `1px solid ${theme.palette.error.main}`
-        : '1px solid transparent',
-      borderRadius: theme.shape.borderRadius,
-      padding: theme.spacing(0.5, 1)
-    },
-    emptyInput: {
-      marginTop: '-22px'
-    },
-    input: {
-      minHeight: editable ? minInputHeight : 'min-content',
-      outline: '0px solid transparent'
-    },
-    inputFocused: {
-      border: error
-        ? `1px solid ${theme.palette.error.main}`
-        : `1px solid ${theme.palette.primary.main}`
-    },
-    notEditable: {
-      backgroundColor: 'transparent'
-    },
-    placeholder: {
-      color: theme.palette.grey[500],
-      pointerEvents: 'none'
-    },
-    root: {
-      '& p,h1,h2,h3,h4,h5,h6,span': {
-        margin: 0
-      }
-    }
-  })
-);
-
-interface Props {
-  className?: string;
-  disabled?: boolean;
-  editable: boolean;
-  editorState?: string;
-  error?: string;
-  hasInitialTextContent?: boolean;
-  initialEditorState?: string;
-  initialize?: (editor) => void;
-  inputClassname?: string;
-  minInputHeight: number;
-  namespace: string;
-  onBlur?: (e: string) => void;
-  placeholder: string;
-  resetEditorToInitialStateCondition?: () => boolean;
-}
-
-const defaultState =
-  '{"root":{"children":[{"children":[],"direction":null,"format":"","indent":0,"type":"paragraph","version":1}],"direction":null,"format":"","indent":0,"type":"root","version":1}}';
-
-const ContentEditable = ({
-  minInputHeight,
-  inputClassname,
-  placeholder,
-  hasInitialTextContent,
-  editable,
-  editorState,
-  namespace,
-  resetEditorToInitialStateCondition,
-  initialEditorState,
-  error,
-  onBlur,
-  className,
-  disabled,
-  initialize
-}: Props): JSX.Element => {
-  const { classes, cx } = useStyles({ editable, error, minInputHeight });
-  const { t } = useTranslation();
-
-  const [editor] = useLexicalComposerContext();
-  const [isFocused, setFocused] = useState(false);
-  const [root, setRoot] = useState('');
-
-  const ref = useCallback(
-    (rootElement: null | HTMLElement) => {
-      editor.setRootElement(rootElement);
-    },
-    [editor]
-  );
-
-  useLayoutEffect(() => {
-    if (!editable) {
-      const newEditorState = editor.parseEditorState(
-        editorState || defaultState
-      );
-
-      editor.setEditorState(newEditorState);
-    }
-  }, [editor, editorState]);
-
-  useEffect(() => {
-    editor.registerTextContentListener((currentRoot) => {
-      setRoot(currentRoot);
-    });
-
-    if (!hasInitialTextContent) {
-      return;
-    }
-
-    setRoot(' ');
-  }, [editor]);
-
-  useEffect(() => {
-    const shouldResetEditorToInitialState =
-      resetEditorToInitialStateCondition?.();
-
-    if (!shouldResetEditorToInitialState) {
-      return;
-    }
-
-    const newEditorState = editor.parseEditorState(
-      initialEditorState || defaultState
-    );
-
-    editor.setEditorState(newEditorState);
-  }, [editorState]);
-
-  const isTextEmpty =
-    isEmpty(root) &&
-    !editor.getEditorState().toJSON().root.children?.[0]?.children?.length;
-
-  const handleBlur = (event: React.FocusEvent<HTMLInputElement>): void => {
-    setFocused(false);
-    onBlur?.(event);
-  };
-
-  const isEditable = editor.isEditable();
-
-  useEffect(() => {
-    if (isNil(disabled)) {
-      return;
-    }
-
-    editor.setEditable(!disabled);
-  }, [disabled]);
-
-  useEffect(() => {
-    initialize?.(editor);
-  }, []);
-
-  return (
-    <div
-      className={cx(
-        classes.root,
-        editable && classes.container,
-        !isEditable && !disabled && classes.notEditable,
-        className,
-        isFocused && classes.inputFocused
-      )}
-    >
-      {editable && isTextEmpty && (
-        <Typography className={classes.placeholder}>
-          {t(placeholder)}
-        </Typography>
-      )}
-      <div
-        aria-label={namespace}
-        className={cx(
-          editable && isTextEmpty && classes.emptyInput,
-          classes.input,
-          inputClassname
-        )}
-        contentEditable={isEditable}
-        data-testid={namespace}
-        ref={ref}
-        onBlur={handleBlur}
-        onFocus={(): void => setFocused(true)}
-      />
-    </div>
-  );
-};
-
-export default ContentEditable;
+import { useCallback, useEffect, useLayoutEffect, useState } from 'react';
+
+import { useLexicalComposerContext } from '@lexical/react/LexicalComposerContext';
+import { makeStyles } from 'tss-react/mui';
+import { isEmpty, isNil } from 'ramda';
+import { useTranslation } from 'react-i18next';
+
+import { Typography } from '@mui/material';
+
+interface StyleProps {
+  editable: boolean;
+  minInputHeight: number;
+}
+
+const useStyles = makeStyles<StyleProps>()(
+  (theme, { minInputHeight, editable }) => ({
+    container: {
+      '& p': {
+        margin: 0
+      },
+      backgroundColor: theme.palette.background.paper,
+      border: '1px solid transparent',
+      borderRadius: theme.shape.borderRadius,
+      padding: theme.spacing(0.5, 1)
+    },
+    emptyInput: {
+      marginTop: '-22px'
+    },
+    input: {
+      minHeight: editable ? minInputHeight : 'min-content',
+      outline: '0px solid transparent'
+    },
+    inputFocused: {
+      border: `1px solid ${theme.palette.primary.main}`
+    },
+    placeholder: {
+      color: theme.palette.grey[500],
+      pointerEvents: 'none'
+    }
+  })
+);
+
+interface Props {
+  editable: boolean;
+  editorState?: string;
+  hasInitialTextContent?: boolean;
+  initialEditorState?: string;
+  inputClassname?: string;
+  minInputHeight: number;
+  namespace: string;
+  placeholder: string;
+  resetEditorToInitialStateCondition?: () => boolean;
+}
+
+const ContentEditable = ({
+  minInputHeight,
+  inputClassname,
+  placeholder,
+  hasInitialTextContent,
+  editable,
+  editorState,
+  namespace,
+  resetEditorToInitialStateCondition,
+  initialEditorState
+}: Props): JSX.Element => {
+  const { classes, cx } = useStyles({ editable, minInputHeight });
+  const { t } = useTranslation();
+
+  const [editor] = useLexicalComposerContext();
+  const [isEditable, setEditable] = useState(false);
+  const [isFocused, setFocused] = useState(false);
+  const [root, setRoot] = useState('');
+
+  const ref = useCallback(
+    (rootElement: null | HTMLElement) => {
+      editor.setRootElement(rootElement);
+    },
+    [editor]
+  );
+
+  useLayoutEffect(() => {
+    setEditable(editor.isEditable());
+
+    if (editorState && !editable) {
+      const newEditorState = editor.parseEditorState(editorState);
+
+      editor.setEditorState(newEditorState);
+    }
+
+    return editor.registerEditableListener((currentIsEditable) => {
+      setEditable(currentIsEditable);
+    });
+  }, [editor, editorState]);
+
+  useEffect(() => {
+    editor.registerTextContentListener((currentRoot) => {
+      setRoot(currentRoot);
+    });
+
+    if (!hasInitialTextContent) {
+      return;
+    }
+
+    setRoot(' ');
+  }, [editor]);
+
+  useEffect(() => {
+    const shouldResetEditorToInitialState =
+      resetEditorToInitialStateCondition?.();
+
+    if (!shouldResetEditorToInitialState || isNil(initialEditorState)) {
+      return;
+    }
+
+    const newEditorState = editor.parseEditorState(initialEditorState);
+
+    editor.setEditorState(newEditorState);
+  }, [editorState]);
+
+  const isTextEmpty = isEmpty(root);
+
+  return (
+    <div
+      className={cx(classes.container, isFocused && classes.inputFocused)}
+      id={namespace}
+    >
+      {editable && isTextEmpty && (
+        <Typography className={classes.placeholder}>
+          {t(placeholder)}
+        </Typography>
+      )}
+      <div
+        aria-label={namespace}
+        className={cx(
+          isTextEmpty && classes.emptyInput,
+          classes.input,
+          inputClassname
+        )}
+        contentEditable={isEditable}
+        data-testid={namespace}
+        ref={ref}
+        onBlur={(): void => setFocused(false)}
+        onFocus={(): void => setFocused(true)}
+      />
+    </div>
+  );
+};
+
+export default ContentEditable;