Merge pull request #37 from membean/fix/default-props-warning

Removed default props as it is no more supported

Author: shruti-14

src/lib/components/DefinitionTooltip/DefinitionTooltip.js

@@ -30,24 +30,24 @@ import positionTooltip from "../../helpers/positionTooltip.js";
     url [String] - A url to link to. Will take precedence over onClick.
 */
 
-const DefinitionTooltip = props => {
+const DefinitionTooltip = (props) => {
   const {
     classes,
     definition,
     explanation,
     id,
     link,
     onClick,
-    position,
+    position = "top",
     term,
-    url
+    url,
   } = props;
 
   const tooltipElementId = `${id}-desc`;
   const tooltipRef = useRef(null);
   const triggerClass = classnames("dfn-tooltip-trigger", classes);
 
-  const handleMouseEnter = event => {
+  const handleMouseEnter = (event) => {
     const currentTooltip = tooltipRef.current;
     positionTooltip(currentTooltip, position);
   };
@@ -68,7 +68,7 @@ const DefinitionTooltip = props => {
         aria-labelledby={tooltipElementId}
         className={triggerClass}
         dangerouslySetInnerHTML={{
-          __html: term
+          __html: term,
         }}
         href={url}
         onMouseEnter={handleMouseEnter}
@@ -82,7 +82,7 @@ const DefinitionTooltip = props => {
         aria-labelledby={tooltipElementId}
         className={triggerClass}
         dangerouslySetInnerHTML={{
-          __html: term
+          __html: term,
         }}
         onClick={onClick}
         onMouseEnter={handleMouseEnter}
@@ -100,7 +100,7 @@ const DefinitionTooltip = props => {
       >
         <span
           dangerouslySetInnerHTML={{
-            __html: term
+            __html: term,
           }}
         />
       </Link>
@@ -118,7 +118,7 @@ const DefinitionTooltip = props => {
       >
         <p
           dangerouslySetInnerHTML={{
-            __html: `<dfn>${term}:</dfn> ${definition}`
+            __html: `<dfn>${term}:</dfn> ${definition}`,
           }}
           id={`${id}-definition`}
         />
@@ -134,10 +134,6 @@ const DefinitionTooltip = props => {
   );
 };
 
-DefinitionTooltip.defaultProps = {
-  position: "top"
-};
-
 DefinitionTooltip.propTypes = {
   classes: PropTypes.string,
   definition: PropTypes.string.isRequired,
@@ -147,7 +143,7 @@ DefinitionTooltip.propTypes = {
   onClick: PropTypes.func,
   position: PropTypes.oneOf(["bottom", "left", "right", "top"]),
   term: PropTypes.string.isRequired,
-  url: PropTypes.string
+  url: PropTypes.string,
 };
 
 export default DefinitionTooltip;

src/lib/components/TextInput/TextInput.js

@@ -59,7 +59,7 @@ import classnames from "classnames";
 const TextInput = React.forwardRef((props, ref) => {
   const {
     autoComplete,
-    autoFocus,
+    autoFocus = false,
     classes,
     disabled,
     feedbackContext,
@@ -79,22 +79,22 @@ const TextInput = React.forwardRef((props, ref) => {
     placeholder,
     polite,
     readOnly,
-    required,
+    required = false,
     step,
     type,
-    value
+    value,
   } = props;
 
   const controlClasses = classnames("text", "control", classes, {
     disabled: disabled,
     inline: inline,
-    invalid: isValid !== undefined && !isValid
+    invalid: isValid !== undefined && !isValid,
   });
   const feedbackClasses = classnames(
     "control-feedback",
     `${feedbackContext || "error"}`,
     {
-      "visually-hidden": !feedbackText
+      "visually-hidden": !feedbackText,
     }
   );
   const feedbackId = `${id}-feedback`;
@@ -191,14 +191,9 @@ TextInput.propTypes = {
     "text",
     "time",
     "url",
-    "week"
+    "week",
   ]),
-  value: PropTypes.string
-};
-
-TextInput.defaultProps = {
-  autoFocus: false,
-  required: false
+  value: PropTypes.string,
 };
 
 export default TextInput;

src/lib/components/Tooltip/Tooltip.js

@@ -38,11 +38,11 @@ const Tooltip = (props) => {
     id,
     link,
     onClick,
-    position,
+    position = "top",
     text,
     tooltipBody,
     url,
-    buttonType
+    buttonType = "button",
   } = props;
 
   const tooltipElementId = `${id}-desc`;
@@ -119,11 +119,6 @@ const Tooltip = (props) => {
   );
 };
 
-Tooltip.defaultProps = {
-  position: "top",
-  buttonType: "button"
-};
-
 Tooltip.propTypes = {
   classes: PropTypes.string,
   tooltipBody: PropTypes.string.isRequired,