fix lint errors

strapi · May 17, 2023 · c0e6eef · c0e6eef
1 parent ce18e78
commit c0e6eef
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 7 deletions.
diff --git a/packages/core/admin/admin/src/content-manager/components/ComponentIcon/ComponentIcon.js b/packages/core/admin/admin/src/content-manager/components/ComponentIcon/ComponentIcon.js
@@ -29,7 +29,7 @@ export function ComponentIcon({ showBackground = true, size = 'M', icon }) {
       size={size}
       showBackground={showBackground}
     >
-      <Icon as={Icons[icon] || Icons['Cube']} />
+      <Icon as={Icons[icon] || Icons.Cube} />
     </Wrapper>
   );
 }

diff --git a/...s/core/admin/admin/src/content-manager/components/DynamicZone/components/ComponentCard.js b/...s/core/admin/admin/src/content-manager/components/DynamicZone/components/ComponentCard.js
@@ -62,6 +62,7 @@ export default function ComponentCard({ children, onClick, icon }) {
 
 ComponentCard.defaultProps = {
   onClick() {},
+  icon: 'Cube',
 };
 
 ComponentCard.propTypes = {

diff --git a/...re/content-type-builder/admin/src/components/ComponentCard/ComponentIcon/ComponentIcon.js b/...re/content-type-builder/admin/src/components/ComponentCard/ComponentIcon/ComponentIcon.js
@@ -23,7 +23,7 @@ export function ComponentIcon({ isActive, icon }) {
       background={isActive ? 'primary200' : 'neutral200'}
       justifyContent="center"
     >
-      <Icon as={Icons[icon] || Icons['Cube']} />
+      <Icon as={Icons[icon] || Icons.Cube} />
     </Wrapper>
   );
 }

diff --git a/packages/core/content-type-builder/admin/src/components/IconPicker/index.js b/packages/core/content-type-builder/admin/src/components/IconPicker/index.js
@@ -15,6 +15,7 @@ import {
 import * as Icons from '@strapi/icons';
 import { useIntl } from 'react-intl';
 import styled from 'styled-components';
+import PropTypes from 'prop-types';
 import { getTrad } from '../../utils';
 
 const EXCLUDE_ICONS = [
@@ -83,6 +84,14 @@ const IconPick = ({ iconKey, name, onChange, isSelected, ariaLabel }) => {
   );
 };
 
+IconPick.propTypes = {
+  iconKey: PropTypes.string.isRequired,
+  name: PropTypes.string.isRequired,
+  onChange: PropTypes.func.isRequired,
+  isSelected: PropTypes.bool.isRequired,
+  ariaLabel: PropTypes.string.isRequired,
+};
+
 const IconPicker = ({ intlLabel, name, onChange, value }) => {
   const { formatMessage } = useIntl();
   const [showSearch, setShowSearch] = useState(false);
@@ -91,9 +100,6 @@ const IconPicker = ({ intlLabel, name, onChange, value }) => {
   const [icons, setIcons] = useState(allIcons);
   const searchIconRef = useRef(null);
 
-  const SearchIcon = Icons['Search'];
-  const TrashIcon = Icons['Trash'];
-
   const toggleSearch = () => {
     setShowSearch(!showSearch);
   };
@@ -154,7 +160,7 @@ const IconPicker = ({ intlLabel, name, onChange, value }) => {
                 id: getTrad('IconPicker.search.button.label'),
                 defaultMessage: 'Search icon button',
               })}
-              icon={<SearchIcon />}
+              icon={<Icons.Search />}
               noBorder
             />
           )}
@@ -165,7 +171,7 @@ const IconPicker = ({ intlLabel, name, onChange, value }) => {
                 id: getTrad('IconPicker.remove.label'),
                 defaultMessage: 'Remove the selected icon',
               })}
-              icon={<TrashIcon />}
+              icon={<Icons.Trash />}
               noBorder
             />
           )}
@@ -214,4 +220,15 @@ const IconPicker = ({ intlLabel, name, onChange, value }) => {
   );
 };
 
+IconPicker.defaultProps = {
+  value: '',
+};
+
+IconPicker.propTypes = {
+  intlLabel: PropTypes.object.isRequired,
+  name: PropTypes.string.isRequired,
+  onChange: PropTypes.func.isRequired,
+  value: PropTypes.string,
+};
+
 export default IconPicker;