From c49723056acec489765acb54bae3889ac07f25af Mon Sep 17 00:00:00 2001
From: MCMXC <16797721+mcmxcdev@users.noreply.github.com>
Date: Thu, 18 Apr 2024 08:18:36 -0600
Subject: refactor: bring down eslint warnings to zero (#1714)

- install `@eslint-react/eslint-plugin` dependency
- configure `@eslint-react/eslint-plugin` in eslint config
- modernize `lint` command in `package.json`
- disable or fix various reported lint issues
- fix `div` being nested in `p` for settings
- replace deprecated `event.keyCode` with `event.key`
- update isEscKeyPress method and unit tests which used deprecated `event.keyCode`
- allow `eslint` v8 as peer dependency for `@eslint-react/eslint-plugin`
---
 src/components/ui/Tabs/Tabs.tsx | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to 'src/components/ui/Tabs')

diff --git a/src/components/ui/Tabs/Tabs.tsx b/src/components/ui/Tabs/Tabs.tsx
index a5ec148aa..d1e397122 100644
--- a/src/components/ui/Tabs/Tabs.tsx
+++ b/src/components/ui/Tabs/Tabs.tsx
@@ -35,6 +35,7 @@ class Tab extends Component<IProps, IState> {
 
   render(): ReactElement {
     const { children: childElements } = this.props;
+    // eslint-disable-next-line @eslint-react/no-children-to-array
     const children = Children.toArray(childElements); // removes all null values
 
     if (children.length === 1) {
@@ -44,9 +45,10 @@ class Tab extends Component<IProps, IState> {
     return (
       <div className="content-tabs">
         <div className="content-tabs__tabs">
+          {/* eslint-disable-next-line @eslint-react/no-children-map */}
           {Children.map(children, (child, i) => (
             <button
-              // eslint-disable-next-line react/no-array-index-key
+              // eslint-disable-next-line react/no-array-index-key, @eslint-react/no-array-index-key
               key={i}
               className={classnames({
                 'content-tabs__item': true,
@@ -60,9 +62,10 @@ class Tab extends Component<IProps, IState> {
           ))}
         </div>
         <div className="content-tabs__content">
+          {/* eslint-disable-next-line @eslint-react/no-children-map */}
           {Children.map(children, (child, i) => (
             <div
-              // eslint-disable-next-line react/no-array-index-key
+              // eslint-disable-next-line react/no-array-index-key, @eslint-react/no-array-index-key
               key={i}
               className={classnames({
                 'content-tabs__item': true,
-- 
cgit v1.2.3-70-g09d2