Merge pull request #58 from DevExpress-Examples/fixExample_25.1.3+

Fix: eslint and ts-eslint issue in React

Author: charlescordialDevExpress

React/.eslintrc.cjs

@@ -15,8 +15,6 @@ module.exports = {
       sourceType: 'module',
     },
     rules: {
-      "space-before-function-paren": "off",              // disable base
-      "@typescript-eslint/space-before-function-paren": ["error", "never"], 
       "react/react-in-jsx-scope": "off",
       "import/no-extraneous-dependencies": ["error", {
         "devDependencies": [

React/package-lock.json

@@ -20,14 +20,14 @@
     "react-dom": "^18.2.0"
   },
   "devDependencies": {
+    "@babel/plugin-proposal-private-property-in-object": "^7.21.11",
     "@testing-library/jest-dom": "^5.16.5",
     "@testing-library/react": "^14.1.2",
     "@testing-library/user-event": "^14.4.3",
     "@types/react": "^18.2.47",
     "@types/react-dom": "^18.2.17",
     "@vitejs/plugin-react": "^4.4.1",
     "@vitest/coverage-v8": "^1.5.0",
-    "@babel/plugin-proposal-private-property-in-object": "^7.21.11",
     "eslint": "^8.35.0",
     "eslint-config-devextreme": "^1.1.4",
     "eslint-plugin-no-only-tests": "2.6.0",
@@ -39,10 +39,10 @@
     "npm-run-all": "^4.1.5",
     "stylelint": "^15.6.1",
     "stylelint-config-standard": "^33.0.0",
-    "typescript": "~5.8.2",
+    "ts-node": "10.9.2",
+    "typescript": "^5.8.3",
     "typescript-eslint": "^8.18.2",
     "vite": "^6.3.5",
-    "ts-node": "10.9.2",
     "vitest": "^1.5.0"
   }
-}
+}

React/package.json

@@ -14,11 +14,13 @@ function App(): JSX.Element {
 
   const onCellPrepared = useCallback((e: DataGridTypes.CellPreparedEvent) => {
     if (e.rowType === 'data' && e.column.dataField === 'FirstName') {
+      // eslint-disable-next-line @typescript-eslint/space-before-function-paren
       on(e.cellElement, 'mouseover', async(arg: MouseEvent) => {
         setCurrentEmployee(e.data);
         await tooltipRef.current?.instance().show(arg.target as Element);
       });
 
+      // eslint-disable-next-line @typescript-eslint/space-before-function-paren
       on(e.cellElement, 'mouseout', async() => {
         await tooltipRef.current?.instance().hide();
       });