{icon &&
}
diff --git a/src/Card/tests/CardFooter.test.jsx b/src/Card/tests/CardFooter.test.jsx
index 0d158cc6ac..a04ff1c7df 100644
--- a/src/Card/tests/CardFooter.test.jsx
+++ b/src/Card/tests/CardFooter.test.jsx
@@ -14,13 +14,15 @@ const actions = (
);
// eslint-disable-next-line react/prop-types
-const CardFooterWrapper = ({ orientation = 'vertical', isLoading, ...props }) => (
-
-
- {actions}
-
-
-);
+function CardFooterWrapper({ orientation = 'vertical', isLoading, ...props }) {
+ return (
+
+
+ {actions}
+
+
+ );
+}
describe('
', () => {
it('renders vertical orientation without footer text', () => {
diff --git a/src/Card/tests/CardHeader.test.jsx b/src/Card/tests/CardHeader.test.jsx
index eb4afc2eec..8f28a490c4 100644
--- a/src/Card/tests/CardHeader.test.jsx
+++ b/src/Card/tests/CardHeader.test.jsx
@@ -6,11 +6,13 @@ import CardHeader from '../CardHeader';
import CardContext from '../CardContext';
// eslint-disable-next-line react/prop-types
-const CardHeaderWrapper = ({ isLoading }) => (
-
-
-
-);
+function CardHeaderWrapper({ isLoading }) {
+ return (
+
+
+
+ );
+}
describe('
', () => {
it('renders with title prop', () => {
diff --git a/src/Card/tests/CardImageCap.test.jsx b/src/Card/tests/CardImageCap.test.jsx
index a3ce6f26b6..894e49f58f 100644
--- a/src/Card/tests/CardImageCap.test.jsx
+++ b/src/Card/tests/CardImageCap.test.jsx
@@ -5,8 +5,9 @@ import CardImageCap from '../CardImageCap';
import CardContext from '../CardContext';
// eslint-disable-next-line react/prop-types
-const CardImageCapWrapper = ({ orientation = 'vertical', isLoading, ...props }) => (
-
);
+function CardImageCapWrapper({ orientation = 'vertical', isLoading, ...props }) {
+ return
;
+}
describe('
', () => {
it('renders with scr prop and srcAlt', () => {
diff --git a/src/Card/tests/CardSection.test.jsx b/src/Card/tests/CardSection.test.jsx
index 18ba52a847..641b44de2f 100644
--- a/src/Card/tests/CardSection.test.jsx
+++ b/src/Card/tests/CardSection.test.jsx
@@ -6,11 +6,13 @@ import CardSection from '../CardSection';
import CardContext from '../CardContext';
// eslint-disable-next-line react/prop-types
-const CardSectionWrapper = ({ isLoading }) => (
-
-
-
-);
+function CardSectionWrapper({ isLoading }) {
+ return (
+
+
+
+ );
+}
describe('
', () => {
it('renders with default props', () => {
diff --git a/src/Chip/Chip.test.jsx b/src/Chip/Chip.test.jsx
index 3733988345..cdd6f13bfe 100644
--- a/src/Chip/Chip.test.jsx
+++ b/src/Chip/Chip.test.jsx
@@ -2,15 +2,16 @@ import React from 'react';
import { mount } from 'enzyme';
import renderer from 'react-test-renderer';
-import Chip from './index';
-
import { Close } from '../../icons';
+import Chip from './index';
-const TestChip = (props) => (
-
- Test
-
-);
+function TestChip(props) {
+ return (
+
+ Test
+
+ );
+}
describe('
', () => {
describe('snapshots', () => {
diff --git a/src/Collapsible/Collapsible.test.jsx b/src/Collapsible/Collapsible.test.jsx
index 6e86e3710d..37e1e52604 100644
--- a/src/Collapsible/Collapsible.test.jsx
+++ b/src/Collapsible/Collapsible.test.jsx
@@ -91,10 +91,10 @@ describe('
', () => {
});
it('correct behavior with unmountOnExit', () => {
let i = 0;
- const Comp = () => {
+ function Comp() {
i += 1;
return
Hello world
;
- };
+ }
const component = mount((
diff --git a/src/Collapsible/CollapsibleAdvanced.jsx b/src/Collapsible/CollapsibleAdvanced.jsx
index 60858d3ecd..8e11cf3a1b 100644
--- a/src/Collapsible/CollapsibleAdvanced.jsx
+++ b/src/Collapsible/CollapsibleAdvanced.jsx
@@ -34,7 +34,7 @@ class CollapsibleAdvanced extends React.Component {
if (this.props.onToggle) {
this.props.onToggle(true);
}
- }
+ };
close = () => {
this.setState({ isOpen: false });
@@ -45,7 +45,7 @@ class CollapsibleAdvanced extends React.Component {
if (this.props.onToggle) {
this.props.onToggle(false);
}
- }
+ };
toggle = () => {
if (this.state.isOpen) {
@@ -53,7 +53,7 @@ class CollapsibleAdvanced extends React.Component {
} else {
this.open();
}
- }
+ };
render() {
const {
diff --git a/src/Collapsible/CollapsibleVisible.jsx b/src/Collapsible/CollapsibleVisible.jsx
index 83d1b888ef..482287e1ef 100644
--- a/src/Collapsible/CollapsibleVisible.jsx
+++ b/src/Collapsible/CollapsibleVisible.jsx
@@ -12,6 +12,7 @@ function CollapsibleVisible({
const isVisible = (isOpen && visibleWhenOpen) || (!isOpen && visibleWhenClosed);
if (isVisible) {
+ // eslint-disable-next-line react/jsx-no-useless-fragment
return <>{children};
}
return null;
diff --git a/src/Collapsible/index.jsx b/src/Collapsible/index.jsx
index b12833a210..5b4fc5c7a3 100644
--- a/src/Collapsible/index.jsx
+++ b/src/Collapsible/index.jsx
@@ -2,12 +2,12 @@ import React from 'react';
import classNames from 'classnames';
import PropTypes from 'prop-types';
+import { ExpandLess, ExpandMore } from '../../icons';
import CollapsibleAdvanced, { CollapsibleContext } from './CollapsibleAdvanced';
import CollapsibleBody from './CollapsibleBody';
import CollapsibleTrigger from './CollapsibleTrigger';
import CollapsibleVisible from './CollapsibleVisible';
import Icon from '../Icon';
-import { ExpandLess, ExpandMore } from '../../icons';
const styleIcons = {
basic: {
diff --git a/src/Container/index.jsx b/src/Container/index.jsx
index c51dbccf99..a00b3c6e21 100644
--- a/src/Container/index.jsx
+++ b/src/Container/index.jsx
@@ -11,17 +11,19 @@ const SIZE_CLASS_NAMES = {
xl: 'container-mw-xl',
};
-const Container = ({ size, children, ...props }) => (
-
- {children}
-
-);
+function Container({ size, children, ...props }) {
+ return (
+
+ {children}
+
+ );
+}
Container.propTypes = {
...RBContainer.propTypes,
diff --git a/src/DataTable/ActionDisplay.jsx b/src/DataTable/ActionDisplay.jsx
index 48d1156645..a059cb093c 100644
--- a/src/DataTable/ActionDisplay.jsx
+++ b/src/DataTable/ActionDisplay.jsx
@@ -4,7 +4,7 @@ import TableActions from './TableActions';
import DataTableContext from './DataTableContext';
// handles display logic for actions
-const ActionDisplay = () => {
+function ActionDisplay() {
const { bulkActions, tableActions, selectedFlatRows } = useContext(DataTableContext);
const noBulkActions = typeof bulkActions !== 'function' && bulkActions?.length < 1;
const noTableActions = typeof tableActions !== 'function' && tableActions?.length < 1;
@@ -19,6 +19,6 @@ const ActionDisplay = () => {
return ;
}
return ;
-};
+}
export default ActionDisplay;
diff --git a/src/DataTable/BulkActions.jsx b/src/DataTable/BulkActions.jsx
index d6344101e2..9d2a4b60bb 100644
--- a/src/DataTable/BulkActions.jsx
+++ b/src/DataTable/BulkActions.jsx
@@ -5,7 +5,7 @@ import classNames from 'classnames';
import DataTableContext from './DataTableContext';
import Actions from './CollapsibleButtonGroup';
-const BulkActions = ({ className }) => {
+function BulkActions({ className }) {
const tableInstance = useContext(DataTableContext);
const {
bulkActions,
@@ -31,7 +31,7 @@ const BulkActions = ({ className }) => {
actions={actions}
/>
);
-};
+}
BulkActions.defaultProps = {
className: null,
diff --git a/src/DataTable/CardView.jsx b/src/DataTable/CardView.jsx
index caa95feb88..d5dcb7be6c 100644
--- a/src/DataTable/CardView.jsx
+++ b/src/DataTable/CardView.jsx
@@ -4,9 +4,9 @@ import classNames from 'classnames';
import { useRows } from './hooks';
import { CardGrid } from '..';
-const CardView = ({
+function CardView({
columnSizes, CardComponent, className,
-}) => {
+}) {
const {
getTableProps, prepareRow, displayRows,
} = useRows();
@@ -30,7 +30,7 @@ const CardView = ({
{renderCards()}
);
-};
+}
CardView.defaultProps = {
columnSizes: {
diff --git a/src/DataTable/CollapsibleButtonGroup.jsx b/src/DataTable/CollapsibleButtonGroup.jsx
index f266969e56..fced75f8ea 100644
--- a/src/DataTable/CollapsibleButtonGroup.jsx
+++ b/src/DataTable/CollapsibleButtonGroup.jsx
@@ -17,10 +17,10 @@ import {
export const ACTION_OVERFLOW_BUTTON_TEXT = 'More actions';
export const SMALL_SCREEN_ACTION_OVERFLOW_BUTTON_TEXT = 'Actions';
-const CollapsibleButtonGroup = ({
+function CollapsibleButtonGroup({
className,
actions,
-}) => {
+}) {
const [isOverflowMenuOpen, openOverflowMenu, closeOverflowMenu] = useToggle(false);
const [overflowMenuTarget, setOverflowMenuTarget] = useState(null);
const {
@@ -92,7 +92,7 @@ const CollapsibleButtonGroup = ({