Client part of TestY.

[danielsheh02]

The client part of the test management system has been implemented. Authorization and basic functionality of working with projects, suites, test cases, test plans and setting up user data in the profile are implemented.

[bogdanaks]

я не увидел где используется это состояние, кроме как его редактирование

[danielsheh02]

Речь о состоянии hover? Hover хранит html элемент, который выделен цветом (как в браузерном поиске), при переключении на следующий, через hover обращаемся к этому элементу и снимаем выделение.

[bogdanaks]

да, про hover, понял, ну это прям совсем не гуд, от такого нужно постараться уйти
или через useRef изменять состояние выделения или по православному через обычный css, если это возможно

[bogdanaks]

прямое изменение св-в html нод в реакте, это плохо, плохо влияет на производительность и это хак в обход самого реакта
для этого лучше использовать useRef, вешать ref на элемент и обращаться к нему через ref.current

[bogdanaks]

обернуть в useMemo, это улучшит производительность

[bogdanaks]

507 строк :)
3 компонента в одном файле, желательно разбить на несколько

[bogdanaks]

Есть вариант сохранить видимую вложенность подобных условий
Заменяем эту строку на if (!projectId) return
И весь ниже описанный код сдвинется влево, это сугубо индивидуально, но легко позволяет не разводить лапшу

[bogdanaks]

зачем тут slice(0) ?
итерация итак начнется с 0 индекса

[bogdanaks]

не нужно экономить с названием функций, что такое bfs сходу не понять

[bogdanaks]

useParams хук в react-router-dom

[bogdanaks]

хук useNavigate - react-router-dom

[bogdanaks]

хук useNavigate - react-router-dom

[bogdanaks]

Можете взять за основу этот eslint конфиг

{
  "env": {
    "browser": true,
    "es2021": true
  },
  "extends": [
    "eslint:recommended",
    "plugin:react/recommended",
    "plugin:@typescript-eslint/recommended"
  ],
  "overrides": [],
  "parser": "@typescript-eslint/parser",
  "parserOptions": {
    "ecmaVersion": "latest",
    "sourceType": "module"
  },
  "plugins": [
    "react",
    "react-hooks",
    "@typescript-eslint",
    "simple-import-sort"
  ],
  "rules": {
    "indent": [
      "error",
      2
    ],
    "quotes": [
      "error",
      "double"
    ],
    "semi": [
      "error",
      "never"
    ],
    "simple-import-sort/imports": [
      "error",
      {
        "groups": [
          [
            "^react",
            "^@?\\w"
          ],
          [
            "^(app)(/.*|$)"
          ],
          [
            "^(entities)(/.*|$)"
          ],
          [
            "^(features)(/.*|$)"
          ],
          [
            "^(pages)(/.*|$)"
          ],
          [
            "^(shared)(/.*|$)"
          ],
          [
            "^(widgets)(/.*|$)"
          ],
          [
            "^\\u0000"
          ],
          [
            "^\\.\\.(?!/?$)",
            "^\\.\\./?$"
          ],
          [
            "^\\./(?=.*/)(?!/?$)",
            "^\\.(?!/?$)",
            "^\\./?$"
          ],
          [
            "^.+\\.?(css)$"
          ]
        ]
      }
    ],
    "simple-import-sort/exports": "error",
    "react/react-in-jsx-scope": "off",
    "object-curly-spacing": [
      "error",
      "always"
    ],
    "react/prop-types": 1,
    "react/no-children-prop": 1,
    "no-multi-spaces": [
      "error"
    ],
    "keyword-spacing": [
      "error",
      {
        "before": true
      }
    ],
    "comma-spacing": [
      "error",
      {
        "before": false,
        "after": true
      }
    ],
    "space-infix-ops": [
      "error",
      {
        "int32Hint": false
      }
    ],
    "comma-dangle": [
      "error",
      {
        "arrays": "always-multiline",
        "objects": "always-multiline",
        "imports": "always-multiline",
        "exports": "always-multiline",
        "functions": "never"
      }
    ],
    "react-hooks/rules-of-hooks": "error",
    "no-multiple-empty-lines": [
      "error",
      {
        "max": 1,
        "maxEOF": 0
      }
    ],
    "eol-last": [
      "error",
      "always"
    ],
    "@typescript-eslint/no-unnecessary-type-constraint": "off"
  }
}