From a51e270899e27ea4f493673824d3a785946e795c Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Thu, 24 Jan 2019 15:14:46 -0800
Subject: [PATCH 01/29] ActionBar

---
 src/ActionBar/ActionBar.js      | 11 ++++++++---
 src/ActionBar/ActionBar.test.js | 21 ++++++++++++++++++---
 2 files changed, 26 insertions(+), 6 deletions(-)

diff --git a/src/ActionBar/ActionBar.js b/src/ActionBar/ActionBar.js
index 0be8506db..78d8c59c7 100644
--- a/src/ActionBar/ActionBar.js
+++ b/src/ActionBar/ActionBar.js
@@ -11,7 +11,7 @@ export const ActionBar = ({ mobile, width, children, className, ...props }) => {
     return (
         <React.Fragment>
             {mobile ? (
-                <div style={{ width: width ? width : '319px' }}>
+                <div style={{width}}>
                     <div {...props} className={actionBarClasses}>{children}</div>
                 </div>
             ) : (
@@ -27,6 +27,11 @@ ActionBar.propTypes = {
     width: PropTypes.string
 };
 
+ActionBar.defaultProps = {
+    mobile: false,
+    width: '319px'
+};
+
 export const ActionBarBack = ({ onClick, className, buttonProps, ...props }) => {
     const actionBarBackClasses = classnames(
         'fd-action-bar__back',
@@ -68,10 +73,10 @@ export const ActionBarHeader = ({ className, description, descriptionProps, titl
 };
 
 ActionBarHeader.propTypes = {
+    description: PropTypes.string.isRequired,
+    title: PropTypes.string.isRequired,
     className: PropTypes.string,
-    description: PropTypes.string,
     descriptionProps: PropTypes.object,
-    title: PropTypes.string,
     titleProps: PropTypes.object
 };
 
diff --git a/src/ActionBar/ActionBar.test.js b/src/ActionBar/ActionBar.test.js
index 14eccbdd4..5d8dc1599 100644
--- a/src/ActionBar/ActionBar.test.js
+++ b/src/ActionBar/ActionBar.test.js
@@ -114,7 +114,12 @@ describe('<ActionBar />', () => {
         });
 
         test('should allow props to be spread to the ActionBarHeader component', () => {
-            const element = mount(<ActionBarHeader data-sample='Sample' />);
+            const element = mount(
+                <ActionBarHeader
+                    data-sample='Sample'
+                    description=''
+                    title='' />
+            );
 
             expect(
                 element.getDOMNode().attributes['data-sample'].value
@@ -122,7 +127,12 @@ describe('<ActionBar />', () => {
         });
 
         test('should allow props to be spread to the ActionBarHeader component\'s h1 element', () => {
-            const element = mount(<ActionBarHeader titleProps={{'data-sample': 'Sample'}} />);
+            const element = mount(
+                <ActionBarHeader
+                    description=''
+                    title=''
+                    titleProps={{'data-sample': 'Sample'}} />
+            );
 
             expect(
                 element.find('h1').getDOMNode().attributes['data-sample'].value
@@ -130,7 +140,12 @@ describe('<ActionBar />', () => {
         });
 
         test('should allow props to be spread to the ActionBarHeader component\'s p element', () => {
-            const element = mount(<ActionBarHeader descriptionProps={{'data-sample': 'Sample'}} />);
+            const element = mount(
+                <ActionBarHeader
+                    description=''
+                    descriptionProps={{'data-sample': 'Sample'}}
+                    title='' />
+            );
 
             expect(
                 element.find('p').getDOMNode().attributes['data-sample'].value

From 2e76203c2823eab06e57565da062460f711a3bd9 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Thu, 24 Jan 2019 15:27:04 -0800
Subject: [PATCH 02/29] Alert

---
 src/Alert/Alert.Component.js | 2 +-
 src/Alert/Alert.js           | 6 +++++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/src/Alert/Alert.Component.js b/src/Alert/Alert.Component.js
index 885bb08b8..0d0efc512 100644
--- a/src/Alert/Alert.Component.js
+++ b/src/Alert/Alert.Component.js
@@ -157,7 +157,7 @@ export const AlertComponent = () => {
                     }
                 ]}>
                 <Alert dismissible={false} link='#'
-                    linkText='link' type=''>
+                    linkText='link'>
                     Default alert with a{' '}
                 </Alert>
             </Playground>
diff --git a/src/Alert/Alert.js b/src/Alert/Alert.js
index ed8859dfd..f5bae6cab 100644
--- a/src/Alert/Alert.js
+++ b/src/Alert/Alert.js
@@ -77,5 +77,9 @@ Alert.propTypes = {
     link: PropTypes.string,
     linkProps: PropTypes.object,
     linkText: PropTypes.string,
-    type: PropTypes.oneOf(['', 'warning', 'error', 'success', 'information'])
+    type: PropTypes.oneOf(['warning', 'error', 'success', 'information'])
+};
+
+Alert.defaultProps = {
+    dismissible: false
 };

From 8e48224c2ffe8585b399e521a7ef0de00200dbb1 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Thu, 24 Jan 2019 16:50:06 -0800
Subject: [PATCH 03/29] Button

---
 src/Button/Button.js | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/src/Button/Button.js b/src/Button/Button.js
index fba9f25e6..004c54ab5 100644
--- a/src/Button/Button.js
+++ b/src/Button/Button.js
@@ -33,25 +33,35 @@ export const Button = ({
     );
 
     return (<button {...props} className={buttonClasses}
-        disabled={disabled ? disabled : false} onClick={onClick}
-        selected={selected ? selected : false} type={typeAttr}>
+        disabled={disabled} onClick={onClick}
+        selected={selected} type={typeAttr}>
         {children}
     </button>);
 };
 
 Button.propTypes = {
+    children: PropTypes.node,
+    className: PropTypes.string,
     compact: PropTypes.bool,
     disabled: PropTypes.bool,
     dropdown: PropTypes.bool,
     glyph: PropTypes.string,
     navbar: PropTypes.bool,
-    option: PropTypes.oneOf(['', 'emphasized', 'light', 'shell']),
+    option: PropTypes.oneOf(['emphasized', 'light', 'shell']),
     selected: PropTypes.bool,
-    type: PropTypes.oneOf(['', 'standard', 'positive', 'negative', 'medium']),
+    type: PropTypes.oneOf(['standard', 'positive', 'negative', 'medium']),
     typeAttr: PropTypes.string,
     onClick: PropTypes.func
 };
 
+Button.defaultProps = {
+    compact: false,
+    disabled: false,
+    dropdown: false,
+    navbar: false,
+    selected: false
+};
+
 export const ButtonGroup = ({children, ...props}) => {
     return (
         <div

From a19970b181dbd14953ba8204652df0a5b0ac8394 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Fri, 25 Jan 2019 10:25:58 -0800
Subject: [PATCH 04/29] Calendar

---
 src/Calendar/Calendar.js | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/Calendar/Calendar.js b/src/Calendar/Calendar.js
index dbbda1a17..3386a1720 100644
--- a/src/Calendar/Calendar.js
+++ b/src/Calendar/Calendar.js
@@ -574,5 +574,12 @@ Calendar.propTypes = {
     tableBodyProps: PropTypes.object,
     tableHeaderProps: PropTypes.object,
     tableProps: PropTypes.object,
-    yearListProps: PropTypes.object
+    yearListProps: PropTypes.object,
+    onChange: PropTypes.func
+};
+
+Calendar.defaultProps = {
+    disableFutureDates: false,
+    disablePastDates: false,
+    disableWeekends: false
 };

From 91e4877926ed08cfa017eaed7be6d7ac5da59a1a Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Fri, 25 Jan 2019 12:43:25 -0800
Subject: [PATCH 05/29] DatePicker

---
 src/DatePicker/DatePicker.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/DatePicker/DatePicker.js b/src/DatePicker/DatePicker.js
index f56526bb2..04c063501 100644
--- a/src/DatePicker/DatePicker.js
+++ b/src/DatePicker/DatePicker.js
@@ -443,3 +443,8 @@ DatePicker.propTypes = {
     enableRangeSelection: PropTypes.bool,
     inputProps: PropTypes.object
 };
+
+DatePicker.defaultProps = {
+    compact: false,
+    enableRangeSelection: false
+};

From 655b005ce258c9de1a3a43946d93165e70fee0ca Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Fri, 25 Jan 2019 13:49:10 -0800
Subject: [PATCH 06/29] Dropdown

---
 src/Dropdown/Dropdown.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/Dropdown/Dropdown.js b/src/Dropdown/Dropdown.js
index dfa15a9cc..c2c8a6c75 100644
--- a/src/Dropdown/Dropdown.js
+++ b/src/Dropdown/Dropdown.js
@@ -25,3 +25,7 @@ Dropdown.propTypes = {
     className: PropTypes.string,
     standard: PropTypes.bool
 };
+
+Dropdown.defaultProps = {
+    standard: false
+};

From f1e2f19475e9122ecff42f6229caa9e84bf26aa9 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Fri, 25 Jan 2019 14:07:11 -0800
Subject: [PATCH 07/29] FormRadio

---
 src/Forms/Forms.js                         | 28 ++++++++++++++++++----
 src/Forms/__snapshots__/Forms.test.js.snap | 23 ++++++++++--------
 2 files changed, 36 insertions(+), 15 deletions(-)

diff --git a/src/Forms/Forms.js b/src/Forms/Forms.js
index a5ae16e0f..00e2ed9a2 100644
--- a/src/Forms/Forms.js
+++ b/src/Forms/Forms.js
@@ -46,6 +46,11 @@ FormItem.propTypes = {
     isInline: PropTypes.bool
 };
 
+FormItem.defaultProps = {
+    isCheck: false,
+    isInline: false
+};
+
 // ------------------------------------------------- Form Label ----------------------------------------------
 export const FormLabel = ({ required, children, className, ...props }) => {
     const formLabelClasses = classnames(
@@ -68,6 +73,10 @@ FormLabel.propTypes = {
     required: PropTypes.bool
 };
 
+FormLabel.defaultProps = {
+    required: false
+};
+
 // ------------------------------------------------- Form Message ----------------------------------------------
 export const FormMessage = ({ type, children, className, ...props }) => {
     const formMessageClasses = classnames(
@@ -88,7 +97,7 @@ export const FormMessage = ({ type, children, className, ...props }) => {
 FormMessage.propTypes = {
     children: PropTypes.node,
     className: PropTypes.string,
-    type: PropTypes.oneOf(['', 'error', 'warning', 'help'])
+    type: PropTypes.oneOf(['error', 'warning', 'help'])
 };
 
 // ------------------------------------------------- Form Input ----------------------------------------------
@@ -183,7 +192,7 @@ export const FormSelect = ({ disabled, children, className, ...props }) => {
         <select
             {...props}
             className={formSelectClasses}
-            disabled={disabled ? true : ''}>
+            disabled={disabled}>
             {children}
         </select>
     );
@@ -194,6 +203,10 @@ FormSelect.propTypes = {
     disabled: PropTypes.bool
 };
 
+FormSelect.defaultProps = {
+    disabled: false
+};
+
 // ------------------------------------------------- Form Radio ----------------------------------------------
 export class FormRadio extends Component {
     constructor(props) {
@@ -223,7 +236,7 @@ export class FormRadio extends Component {
                         <input
                             checked={this.state.selectedItem === inputItem.id}
                             className='fd-form__control'
-                            disabled={disabled ? true : ''}
+                            disabled={disabled}
                             id={inputItem.id}
                             name={inputItem.name}
                             onChange={this.handleChange}
@@ -239,7 +252,7 @@ export class FormRadio extends Component {
                     <input
                         checked={this.state.selectedItem === inputItem.id}
                         className='fd-form__control'
-                        disabled={disabled ? true : ''}
+                        disabled={disabled}
                         id={inputItem.id}
                         name={inputItem.name}
                         onChange={this.handleChange}
@@ -255,8 +268,13 @@ export class FormRadio extends Component {
     }
 }
 FormRadio.propTypes = {
+    inputs: PropTypes.array.isRequired,
     defaultChecked: PropTypes.string,
     disabled: PropTypes.bool,
-    inputs: PropTypes.array,
     isInline: PropTypes.bool
 };
+
+FormRadio.defaultProps = {
+    disabled: false,
+    isInline: false
+};
diff --git a/src/Forms/__snapshots__/Forms.test.js.snap b/src/Forms/__snapshots__/Forms.test.js.snap
index 11df68e81..ad96f07c6 100644
--- a/src/Forms/__snapshots__/Forms.test.js.snap
+++ b/src/Forms/__snapshots__/Forms.test.js.snap
@@ -8,6 +8,7 @@ exports[`<Forms /> create form item 1`] = `
     className="fd-form__item fd-form__item--inline fd-form__item--check"
   >
     <label
+      aria-required={false}
       className="fd-form__label"
       forAttr="input-1"
     >
@@ -62,7 +63,7 @@ exports[`<Forms /> create form item 1`] = `
     </span>
     <select
       className="fd-form__control"
-      disabled=""
+      disabled={false}
       id="select-1"
     >
       <option
@@ -104,6 +105,7 @@ exports[`<Forms /> create form item 2`] = `
     className="fd-form__item fd-form__item--inline fd-form__item--check"
   >
     <label
+      aria-required={false}
       className="fd-form__label"
       forAttr="input-1"
     >
@@ -136,7 +138,7 @@ exports[`<Forms /> create form item 3`] = `
       <input
         checked={false}
         className="fd-form__control"
-        disabled=""
+        disabled={false}
         id="radio-1"
         name="radio-1"
         onChange={[Function]}
@@ -156,7 +158,7 @@ exports[`<Forms /> create form item 3`] = `
       <input
         checked={true}
         className="fd-form__control"
-        disabled=""
+        disabled={false}
         id="radio-2"
         name="radio-2"
         onChange={[Function]}
@@ -176,7 +178,7 @@ exports[`<Forms /> create form item 3`] = `
       <input
         checked={false}
         className="fd-form__control"
-        disabled=""
+        disabled={false}
         id="radio-3"
         name="radio-3"
         onChange={[Function]}
@@ -210,7 +212,7 @@ exports[`<Forms /> create form item 4`] = `
       <input
         checked={false}
         className="fd-form__control"
-        disabled=""
+        disabled={false}
         id="radio-1"
         name="radio-1"
         onChange={[Function]}
@@ -230,7 +232,7 @@ exports[`<Forms /> create form item 4`] = `
       <input
         checked={true}
         className="fd-form__control"
-        disabled=""
+        disabled={false}
         id="radio-2"
         name="radio-2"
         onChange={[Function]}
@@ -250,7 +252,7 @@ exports[`<Forms /> create form item 4`] = `
       <input
         checked={false}
         className="fd-form__control"
-        disabled=""
+        disabled={false}
         id="radio-3"
         name="radio-3"
         onChange={[Function]}
@@ -346,7 +348,7 @@ exports[`<Forms /> create form item 4`] = `
         <input
           checked={false}
           className="fd-form__control"
-          disabled=""
+          disabled={false}
           id="radio-7"
           name="radio-7"
           onChange={[Function]}
@@ -366,7 +368,7 @@ exports[`<Forms /> create form item 4`] = `
         <input
           checked={false}
           className="fd-form__control"
-          disabled=""
+          disabled={false}
           id="radio-8"
           name="radio-8"
           onChange={[Function]}
@@ -386,7 +388,7 @@ exports[`<Forms /> create form item 4`] = `
         <input
           checked={true}
           className="fd-form__control"
-          disabled=""
+          disabled={false}
           id="radio-9"
           name="radio-9"
           onChange={[Function]}
@@ -474,6 +476,7 @@ exports[`<Forms /> create form item 4`] = `
       value=""
     />
     <label
+      aria-required={false}
       className="fd-form__label blue"
       forAttr="checkbox-1"
     >

From 7af0680822eb062285a9b6576cec65c9d4f64c00 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 09:48:19 -0800
Subject: [PATCH 08/29] InputGroup

---
 src/InputGroup/InputGroup.js | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/InputGroup/InputGroup.js b/src/InputGroup/InputGroup.js
index e043b871c..6e407d1c9 100644
--- a/src/InputGroup/InputGroup.js
+++ b/src/InputGroup/InputGroup.js
@@ -15,8 +15,8 @@ export class InputGroup extends Component {
         super(props);
 
         this.state = {
-            value: this.props.inputValue || '',
-            searchValue: this.props.inputValue || ''
+            value: this.props.inputValue,
+            searchValue: this.props.inputValue
         };
     }
 
@@ -265,3 +265,11 @@ InputGroup.propTypes = {
     numberUpButtonProps: PropTypes.object,
     searchButtonProps: PropTypes.object
 };
+
+InputGroup.defaultProps = {
+    actions: false,
+    addonPos: 'after',
+    compact: false,
+    inputType: 'text',
+    inputValue: ''
+};

From ae68a4eedc43dfad5dfdd11ef465994b0e8a196f Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 09:58:30 -0800
Subject: [PATCH 09/29] LocalizationEditor

---
 src/LocalizationEditor/LocalizationEditor.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/LocalizationEditor/LocalizationEditor.js b/src/LocalizationEditor/LocalizationEditor.js
index 76119dac5..bcfe3fbf4 100644
--- a/src/LocalizationEditor/LocalizationEditor.js
+++ b/src/LocalizationEditor/LocalizationEditor.js
@@ -138,3 +138,8 @@ LocalizationEditor.propTypes = {
     popoverProps: PropTypes.object,
     textarea: PropTypes.bool
 };
+
+LocalizationEditor.defaultProps = {
+    compact: false,
+    textarea: false
+};

From 595a223f7705864ede835ccf6f1de081857b32b6 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 10:10:32 -0800
Subject: [PATCH 10/29] Menu

---
 src/Menu/Menu.js | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/Menu/Menu.js b/src/Menu/Menu.js
index 680d4b7ae..eaf3a31f6 100644
--- a/src/Menu/Menu.js
+++ b/src/Menu/Menu.js
@@ -25,6 +25,10 @@ Menu.propTypes = {
     className: PropTypes.string
 };
 
+Menu.defaultProps = {
+    addonBefore: false
+};
+
 // ---------------------------------------- Menu List ----------------------------------------
 export const MenuList = ({ children, className, ...props }) => {
     const menuListClasses = classnames(
@@ -81,6 +85,11 @@ MenuItem.propTypes = {
     urlProps: PropTypes.object
 };
 
+MenuItem.defaultProps = {
+    isLink: false,
+    separator: false
+};
+
 // ---------------------------------------- Menu Group ----------------------------------------
 export const MenuGroup = ({ title, children, className, titleProps, ...props }) => {
     const menuGroupClasses = classnames(
@@ -97,7 +106,7 @@ export const MenuGroup = ({ title, children, className, titleProps, ...props })
 };
 
 MenuGroup.propTypes = {
+    title: PropTypes.string.isRequired,
     className: PropTypes.string,
-    title: PropTypes.string,
     titleProps: PropTypes.object
 };

From f3a2d1be661b66259966f21931a21b7235fc40fb Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 10:38:17 -0800
Subject: [PATCH 11/29] Modal

---
 src/Modal/Modal.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/Modal/Modal.js b/src/Modal/Modal.js
index e65865699..f8372cc2c 100644
--- a/src/Modal/Modal.js
+++ b/src/Modal/Modal.js
@@ -95,3 +95,7 @@ Modal.propTypes = {
     show: PropTypes.bool,
     titleProps: PropTypes.object
 };
+
+Modal.defaultProps = {
+    show: false
+};

From 9c8bc5b2f1e4441e407dcc37af181d6d24e961da Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 10:50:35 -0800
Subject: [PATCH 12/29] MultiInput

---
 src/MultiInput/MultiInput.js | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/MultiInput/MultiInput.js b/src/MultiInput/MultiInput.js
index 18572582f..4a62c4dd1 100644
--- a/src/MultiInput/MultiInput.js
+++ b/src/MultiInput/MultiInput.js
@@ -198,3 +198,7 @@ MultiInput.propTypes = {
     tagProps: PropTypes.object,
     onTagsUpdate: PropTypes.func
 };
+
+MultiInput.defaultProps = {
+    compact: false
+};

From 55a0c8180fcbe2689faf6d7d76954b5c35eae252 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 13:10:07 -0800
Subject: [PATCH 13/29] Pagination

---
 src/Pagination/Pagination.js | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/src/Pagination/Pagination.js b/src/Pagination/Pagination.js
index e159f4bdf..2fe56f869 100644
--- a/src/Pagination/Pagination.js
+++ b/src/Pagination/Pagination.js
@@ -7,7 +7,7 @@ export class Pagination extends Component {
         super(props, context);
 
         this.state = {
-            selectedPage: this.props.initialPage || 1
+            selectedPage: this.props.initialPage
         };
     }
 
@@ -74,21 +74,22 @@ export class Pagination extends Component {
     render() {
         const {
             itemsTotal,
-            itemsPerPage = 10,
-            displayTotal = true,
+            itemsPerPage,
+            displayTotal,
             totalText,
             className,
             linkProps,
             displayTotalProps,
             prevProps,
             nextProps,
+            initialPage,
             ...props
         } = this.props;
 
         // calculate number of pages based on item total and items per page
         // check to make sure itemsPerPage != 0
         this.numberOfPages = Math.ceil(
-            itemsTotal / (itemsPerPage ? itemsPerPage : 10)
+            itemsTotal / (itemsPerPage || 10)
         );
 
         const paginationClasses = classnames(
@@ -104,7 +105,7 @@ export class Pagination extends Component {
                     <span
                         {...displayTotalProps}
                         className='fd-pagination__total'>
-                        {itemsTotal} {totalText || 'items'}
+                        {itemsTotal} {totalText}
                     </span>
                 ) : (
                     ''
@@ -146,3 +147,10 @@ Pagination.propTypes = {
     prevProps: PropTypes.object,
     totalText: PropTypes.string
 };
+
+Pagination.defaultProps = {
+    displayTotal: true,
+    initialPage: 1,
+    itemsPerPage: 10,
+    totalText: 'items'
+};

From 2a2b7186c2aacd5baa9327941268c6918c528913 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 13:41:17 -0800
Subject: [PATCH 14/29] SearchInput

---
 src/SearchInput/SearchInput.js | 17 ++++++++++++++++-
 1 file changed, 16 insertions(+), 1 deletion(-)

diff --git a/src/SearchInput/SearchInput.js b/src/SearchInput/SearchInput.js
index 96c6e54f1..e69ffbcb4 100644
--- a/src/SearchInput/SearchInput.js
+++ b/src/SearchInput/SearchInput.js
@@ -267,7 +267,22 @@ export class SearchInput extends Component {
 }
 
 SearchInput.propTypes = {
+    className: PropTypes.string,
+    compact: PropTypes.bool,
     inputProps: PropTypes.object,
+    inShellbar: PropTypes.bool,
     listProps: PropTypes.object,
-    searchBtnProps: PropTypes.object
+    noSearchBtn: PropTypes.bool,
+    placeholder: PropTypes.string,
+    searchBtnProps: PropTypes.object,
+    searchList: PropTypes.array,
+    onChange: PropTypes.func,
+    onEnter: PropTypes.func,
+    onSearch: PropTypes.func
+};
+
+SearchInput.defaultProps = {
+    compact: false,
+    inShellbar: false,
+    noSearchBtn: false
 };

From c87a10df2dfe11f7a21f8fa3a24b525a553d53e3 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 14:44:30 -0800
Subject: [PATCH 15/29] Shellbar

---
 src/Shellbar/Shellbar.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/Shellbar/Shellbar.js b/src/Shellbar/Shellbar.js
index 898fe91b7..232ee38c8 100644
--- a/src/Shellbar/Shellbar.js
+++ b/src/Shellbar/Shellbar.js
@@ -405,5 +405,7 @@ Shellbar.propTypes = {
 };
 
 Shellbar.defaultProps = {
-    actions: null
+    actions: null,
+    copilot: false,
+    logoSAP: false
 };

From 502c0b9923379aa3bc208209ae440b2da57ecba9 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 15:03:21 -0800
Subject: [PATCH 16/29] SideNavigation

---
 src/SideNavigation/SideNavigation.js      | 6 +++++-
 src/SideNavigation/SideNavigation.test.js | 4 ++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/SideNavigation/SideNavigation.js b/src/SideNavigation/SideNavigation.js
index 85dccb5fe..997951c75 100644
--- a/src/SideNavigation/SideNavigation.js
+++ b/src/SideNavigation/SideNavigation.js
@@ -24,6 +24,10 @@ SideNav.propTypes = {
     icons: PropTypes.bool
 };
 
+SideNav.defaultProps = {
+    icons: false
+};
+
 export class SideNavList extends Component {
     constructor(props) {
         super(props);
@@ -190,7 +194,7 @@ export const SideNavGroup = props => {
 };
 
 SideNavGroup.propTypes = {
+    title: PropTypes.string.isRequired,
     className: PropTypes.string,
-    title: PropTypes.string,
     titleProps: PropTypes.object
 };
diff --git a/src/SideNavigation/SideNavigation.test.js b/src/SideNavigation/SideNavigation.test.js
index 5fd61513b..c57c3192c 100644
--- a/src/SideNavigation/SideNavigation.test.js
+++ b/src/SideNavigation/SideNavigation.test.js
@@ -218,7 +218,7 @@ describe('<SideNavigation />', () => {
         });
 
         test('should allow props to be spread to the SideNavGroup component', () => {
-            const element = mount(<SideNavGroup data-sample='Sample' />);
+            const element = mount(<SideNavGroup data-sample='Sample' title='test' />);
 
             expect(
                 element.getDOMNode().attributes['data-sample'].value
@@ -226,7 +226,7 @@ describe('<SideNavigation />', () => {
         });
 
         test('should allow props to be spread to the SideNavGroup component\'s h1 element', () => {
-            const element = mount(<SideNavGroup titleProps={{'data-sample': 'Sample'}} />);
+            const element = mount(<SideNavGroup title='test' titleProps={{'data-sample': 'Sample'}} />);
 
             expect(
                 element.find('h1').getDOMNode().attributes['data-sample'].value

From cc027d5c2cd9c640de1bf66fe035328382b91436 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 15:22:19 -0800
Subject: [PATCH 17/29] Table

---
 src/Table/Table.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Table/Table.js b/src/Table/Table.js
index 961d4682c..31d05b7da 100644
--- a/src/Table/Table.js
+++ b/src/Table/Table.js
@@ -49,13 +49,13 @@ export const Table = props => {
     );
 };
 Table.propTypes = {
+    headers: PropTypes.array.isRequired,
     tableData: PropTypes.arrayOf(
         PropTypes.shape({
             rowData: PropTypes.array
         }).isRequired
     ).isRequired,
     className: PropTypes.string,
-    headers: PropTypes.array,
     tableBodyProps: PropTypes.object,
     tableBodyRowProps: PropTypes.oneOfType([
         PropTypes.object,

From 79c33a7539206391ba804cc64adb167d972ebaa0 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Mon, 28 Jan 2019 16:05:47 -0800
Subject: [PATCH 18/29] Tabs

---
 src/Tabs/Tabs.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/Tabs/Tabs.js b/src/Tabs/Tabs.js
index 188bcb60e..0e0a987a6 100644
--- a/src/Tabs/Tabs.js
+++ b/src/Tabs/Tabs.js
@@ -20,6 +20,11 @@ export const Tabs = props => {
     );
 };
 
+Tabs.propTypes = {
+    children: PropTypes.node,
+    classnames: PropTypes.string
+};
+
 export class TabComponent extends Component {
     constructor(props) {
         super(props);

From c1f24c21d2bc81eb41ba787a6ef0f096289e02ed Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Tue, 29 Jan 2019 09:30:12 -0800
Subject: [PATCH 19/29] Tile

---
 src/Tile/Tile.js | 30 +++++++++++++++++++++++++-----
 1 file changed, 25 insertions(+), 5 deletions(-)

diff --git a/src/Tile/Tile.js b/src/Tile/Tile.js
index 300ebbac5..b25f19ebe 100644
--- a/src/Tile/Tile.js
+++ b/src/Tile/Tile.js
@@ -44,6 +44,10 @@ Tile.propTypes = {
     rowSpan: PropTypes.number
 };
 
+Tile.defaultProps = {
+    disabled: false
+};
+
 export const TileContent = props => {
     const { title, children, className, titleProps, ...rest } = props;
 
@@ -61,8 +65,8 @@ export const TileContent = props => {
 };
 
 TileContent.propTypes = {
+    title: PropTypes.string.isRequired,
     className: PropTypes.string,
-    title: PropTypes.string,
     titleProps: PropTypes.object
 };
 
@@ -77,6 +81,11 @@ export const TileMedia = props => {
     return <div {...rest} className={tileMediaClasses}>{children}</div>;
 };
 
+TileMedia.propTypes = {
+    children: PropTypes.node,
+    className: PropTypes.string
+};
+
 export const TileActions = props => {
     const { children, className, ...rest } = props;
 
@@ -88,6 +97,11 @@ export const TileActions = props => {
     return <div {...rest} className={tileActionsClasses}>{children}</div>;
 };
 
+TileActions.propTypes = {
+    children: PropTypes.node,
+    className: PropTypes.string
+};
+
 export const ProductTile = props => {
     const { disabled, children, className, ...rest } = props;
 
@@ -109,10 +123,15 @@ export const ProductTile = props => {
 };
 
 ProductTile.propTypes = {
+    children: PropTypes.node,
     className: PropTypes.string,
     disabled: PropTypes.bool
 };
 
+ProductTile.defaultProps = {
+    disabled: false
+};
+
 export const ProductTileContent = props => {
     const { title, children, className, titleProps, ...rest } = props;
 
@@ -161,10 +180,7 @@ export const TileGrid = props => {
 
     const tileGridClasses = classnames(
         'fd-tile-grid',
-        {
-            [`fd-tile-grid--${col}col`]: !!col,
-            'fd-tile-grid--3col': !col
-        },
+        `fd-tile-grid--${col}col`,
         className
     );
 
@@ -181,3 +197,7 @@ TileGrid.propTypes = {
     className: PropTypes.string,
     col: PropTypes.number
 };
+
+TileGrid.defaultProps = {
+    col: 3
+};

From ea613288b1880693737aa8559815fd60b0566074 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Tue, 29 Jan 2019 09:30:37 -0800
Subject: [PATCH 20/29] Tile

---
 src/Tile/Tile.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/Tile/Tile.js b/src/Tile/Tile.js
index b25f19ebe..f030f8f1e 100644
--- a/src/Tile/Tile.js
+++ b/src/Tile/Tile.js
@@ -194,6 +194,7 @@ export const TileGrid = props => {
 };
 
 TileGrid.propTypes = {
+    children: PropTypes.node,
     className: PropTypes.string,
     col: PropTypes.number
 };

From 4201a6acdb59b114810a6abf3633f255e32d5ecc Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Tue, 29 Jan 2019 11:32:48 -0800
Subject: [PATCH 21/29] Time

---
 src/Time/Time.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/Time/Time.js b/src/Time/Time.js
index 656d63416..085af06dc 100644
--- a/src/Time/Time.js
+++ b/src/Time/Time.js
@@ -23,9 +23,12 @@ class TimeItem extends Component {
         value: PropTypes.oneOfType([PropTypes.number, PropTypes.string])
     };
     static defaultProps = {
+        arialabel: '',
+        disabled: false,
+        format12hours: false,
         id: '',
-        value: null,
-        arialabel: ''
+        spinners: false,
+        value: null
     };
 
     constructor(props) {

From 9a7d309f9cce3cee44a39d9fd13d39478bb3866c Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Tue, 29 Jan 2019 11:48:54 -0800
Subject: [PATCH 22/29] TimePicker

---
 src/TimePicker/TimePicker.js | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/TimePicker/TimePicker.js b/src/TimePicker/TimePicker.js
index 8dab8f36e..8c131e7fe 100644
--- a/src/TimePicker/TimePicker.js
+++ b/src/TimePicker/TimePicker.js
@@ -32,7 +32,12 @@ class TimePickerItem extends Component {
         buttonID: '',
         style: '',
         isValid: false,
-        length: 0
+        length: 0,
+        disabled: false,
+        format12Hours: false,
+        showHour: false,
+        showMinute: false,
+        showSecond: false
     };
 
     constructor(props) {

From bf92ab12fbf1a8ba5a3554840199d2aec5f4d5a5 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Tue, 29 Jan 2019 12:01:47 -0800
Subject: [PATCH 23/29] Toggle

---
 src/Toggle/Toggle.js                         | 4 ++++
 src/Toggle/__snapshots__/Toggle.test.js.snap | 2 ++
 2 files changed, 6 insertions(+)

diff --git a/src/Toggle/Toggle.js b/src/Toggle/Toggle.js
index 377be1bf7..8b5428b2e 100644
--- a/src/Toggle/Toggle.js
+++ b/src/Toggle/Toggle.js
@@ -63,3 +63,7 @@ Toggle.propTypes = {
     labelProps: PropTypes.object,
     size: PropTypes.oneOf(['', 'xs', 's', 'l'])
 };
+
+Toggle.defaultProps = {
+    disabled: false
+};
diff --git a/src/Toggle/__snapshots__/Toggle.test.js.snap b/src/Toggle/__snapshots__/Toggle.test.js.snap
index 670f435a2..b0aca1dd6 100644
--- a/src/Toggle/__snapshots__/Toggle.test.js.snap
+++ b/src/Toggle/__snapshots__/Toggle.test.js.snap
@@ -11,6 +11,7 @@ exports[`<Toggle /> create toggle component 1`] = `
       className="fd-toggle fd-form__control"
     >
       <input
+        disabled={false}
         onChange={[Function]}
         type="checkbox"
       />
@@ -37,6 +38,7 @@ exports[`<Toggle /> create toggle component 2`] = `
     >
       <input
         checked={true}
+        disabled={false}
         onChange={[Function]}
         type="checkbox"
       />

From af7bd50ed29c14ac43dd05ab15c234c069b62f76 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Wed, 30 Jan 2019 15:16:39 -0800
Subject: [PATCH 24/29] Remove false default props

---
 src/Alert/Alert.js                            |  4 --
 src/Button/Button.js                          |  8 ----
 src/Button/__snapshots__/Button.test.js.snap  | 21 ---------
 src/Calendar/Calendar.js                      |  6 ---
 src/ComboboxInput/ComboboxInput.js            |  5 ---
 src/DatePicker/DatePicker.js                  |  5 ---
 src/Dropdown/Dropdown.js                      |  4 --
 .../__snapshots__/Dropdown.test.js.snap       |  9 ----
 src/InputGroup/InputGroup.js                  |  2 -
 .../__snapshots__/InputGroup.test.js.snap     |  4 --
 .../__snapshots__/ListGroup.test.js.snap      |  4 --
 src/LocalizationEditor/LocalizationEditor.js  |  5 ---
 src/Menu/Menu.js                              |  9 ----
 src/Modal/Modal.js                            |  4 --
 src/MultiInput/MultiInput.js                  |  4 --
 src/Panel/Panel.js                            |  9 ----
 src/Panel/__snapshots__/Panel.test.js.snap    |  4 --
 src/SearchInput/SearchInput.js                |  6 ---
 src/Shellbar/Shellbar.js                      |  6 ---
 src/SideNavigation/SideNavigation.js          |  4 --
 src/Tile/Tile.js                              |  8 ----
 src/Tile/__snapshots__/Tile.test.js.snap      |  4 --
 src/Time/Time.js                              |  9 ----
 src/Time/__snapshots__/Time.test.js.snap      |  8 ----
 src/TimePicker/TimePicker.js                  | 32 ++-----------
 .../__snapshots__/TimePicker.test.js.snap     | 45 -------------------
 src/Toggle/Toggle.js                          |  4 +-
 src/Toggle/__snapshots__/Toggle.test.js.snap  |  2 -
 28 files changed, 4 insertions(+), 231 deletions(-)

diff --git a/src/Alert/Alert.js b/src/Alert/Alert.js
index f2963591c..05ad7e925 100644
--- a/src/Alert/Alert.js
+++ b/src/Alert/Alert.js
@@ -81,10 +81,6 @@ Alert.propTypes = {
     type: PropTypes.oneOf(ALERT_TYPES)
 };
 
-Alert.defaultProps = {
-    dismissible: false
-};
-
 Alert.propDescriptions = {
     dismissible: 'Set to **true** to show a dismiss button.',
     link: 'Value to be applied to the anchor\'s `href` attribute.',
diff --git a/src/Button/Button.js b/src/Button/Button.js
index f31fbd8cf..0be512c0d 100644
--- a/src/Button/Button.js
+++ b/src/Button/Button.js
@@ -55,14 +55,6 @@ Button.propTypes = {
     onClick: PropTypes.func
 };
 
-Button.defaultProps = {
-    compact: false,
-    disabled: false,
-    dropdown: false,
-    navbar: false,
-    selected: false
-};
-
 Button.propDescriptions = {
     dropdown: 'Set to **true** if button triggers a dropdown list. ',
     navbar: 'Set to **true** if button is part of global navbar.',
diff --git a/src/Button/__snapshots__/Button.test.js.snap b/src/Button/__snapshots__/Button.test.js.snap
index 835318a11..7c87481e6 100644
--- a/src/Button/__snapshots__/Button.test.js.snap
+++ b/src/Button/__snapshots__/Button.test.js.snap
@@ -8,18 +8,13 @@ exports[`<Button /> create button group 1`] = `
 >
   <button
     className="fd-button sap-icon--survey blue"
-    disabled={false}
-    selected={false}
   />
   <button
     className="fd-button sap-icon--pie-chart is-selected"
-    disabled={false}
     selected={true}
   />
   <button
     className="fd-button sap-icon--pool"
-    disabled={false}
-    selected={false}
   />
 </div>
 `;
@@ -27,8 +22,6 @@ exports[`<Button /> create button group 1`] = `
 exports[`<Button /> create buttons 1`] = `
 <button
   className="fd-button"
-  disabled={false}
-  selected={false}
 >
   Regular Button
 </button>
@@ -37,8 +30,6 @@ exports[`<Button /> create buttons 1`] = `
 exports[`<Button /> create buttons 2`] = `
 <button
   className="fd-button--emphasized"
-  disabled={false}
-  selected={false}
 >
   Emphasized Button
 </button>
@@ -47,8 +38,6 @@ exports[`<Button /> create buttons 2`] = `
 exports[`<Button /> create buttons 3`] = `
 <button
   className="fd-button fd-button--positive"
-  disabled={false}
-  selected={false}
 >
   Positive Button
 </button>
@@ -57,8 +46,6 @@ exports[`<Button /> create buttons 3`] = `
 exports[`<Button /> create buttons 4`] = `
 <button
   className="fd-button sap-icon--cart"
-  disabled={false}
-  selected={false}
 >
   Icon Button
 </button>
@@ -67,8 +54,6 @@ exports[`<Button /> create buttons 4`] = `
 exports[`<Button /> create buttons 5`] = `
 <button
   className="fd-button fd-button--compact"
-  disabled={false}
-  selected={false}
 >
   Compact
 </button>
@@ -77,7 +62,6 @@ exports[`<Button /> create buttons 5`] = `
 exports[`<Button /> create buttons 6`] = `
 <button
   className="fd-button--emphasized is-selected"
-  disabled={false}
   selected={true}
 >
   Selected State
@@ -88,7 +72,6 @@ exports[`<Button /> create buttons 7`] = `
 <button
   className="fd-button--emphasized is-disabled"
   disabled={true}
-  selected={false}
 >
   Disabled State
 </button>
@@ -97,8 +80,6 @@ exports[`<Button /> create buttons 7`] = `
 exports[`<Button /> create buttons 8`] = `
 <button
   className="fd-button fd-dropdown__control"
-  disabled={false}
-  selected={false}
 >
   Dropdown Button
 </button>
@@ -107,8 +88,6 @@ exports[`<Button /> create buttons 8`] = `
 exports[`<Button /> create buttons 9`] = `
 <button
   className="fd-button fd-global-nav__btn"
-  disabled={false}
-  selected={false}
 >
   Navbar button
 </button>
diff --git a/src/Calendar/Calendar.js b/src/Calendar/Calendar.js
index 5433a604c..80734308e 100644
--- a/src/Calendar/Calendar.js
+++ b/src/Calendar/Calendar.js
@@ -583,12 +583,6 @@ Calendar.propTypes = {
     onChange: PropTypes.func
 };
 
-Calendar.defaultProps = {
-    disableFutureDates: false,
-    disablePastDates: false,
-    disableWeekends: false
-};
-
 Calendar.propDescriptions = {
     blockedDates: 'Blocks dates that are in between the blocked dates.',
     disableAfterDate: 'Disables dates of a calendar that come after the specified date.',
diff --git a/src/ComboboxInput/ComboboxInput.js b/src/ComboboxInput/ComboboxInput.js
index 1ccb5c43f..3e716852d 100644
--- a/src/ComboboxInput/ComboboxInput.js
+++ b/src/ComboboxInput/ComboboxInput.js
@@ -60,11 +60,6 @@ ComboboxInput.propTypes = {
     popoverProps: PropTypes.object
 };
 
-ComboboxInput.defaultTypes = {
-    compact: false,
-    placeholder: ''
-};
-
 ComboboxInput.propDescriptions = {
     menu: 'An object containing a `Menu` component.'
 };
diff --git a/src/DatePicker/DatePicker.js b/src/DatePicker/DatePicker.js
index 516ae65ec..7025a93ed 100644
--- a/src/DatePicker/DatePicker.js
+++ b/src/DatePicker/DatePicker.js
@@ -445,11 +445,6 @@ DatePicker.propTypes = {
     inputProps: PropTypes.object
 };
 
-DatePicker.defaultProps = {
-    compact: false,
-    enableRangeSelection: false
-};
-
 DatePicker.propDescriptions = {
     ...Calendar.propDescriptions,
     enableRangeSelection: 'Set to **true** to enable the selection of a date range (begin and end).'
diff --git a/src/Dropdown/Dropdown.js b/src/Dropdown/Dropdown.js
index 8378b0592..2dbadaf22 100644
--- a/src/Dropdown/Dropdown.js
+++ b/src/Dropdown/Dropdown.js
@@ -26,10 +26,6 @@ Dropdown.propTypes = {
     standard: PropTypes.bool
 };
 
-Dropdown.defaultProps = {
-    standard: false
-};
-
 Dropdown.propDescriptions = {
     standard: 'Set to **true** to enable a dropdown for toolbar.'
 };
diff --git a/src/Dropdown/__snapshots__/Dropdown.test.js.snap b/src/Dropdown/__snapshots__/Dropdown.test.js.snap
index ecbaa7ce8..796c4c50e 100644
--- a/src/Dropdown/__snapshots__/Dropdown.test.js.snap
+++ b/src/Dropdown/__snapshots__/Dropdown.test.js.snap
@@ -14,8 +14,6 @@ exports[`<Dropdown /> create dropdown component 1`] = `
     >
       <button
         className="fd-button fd-dropdown__control"
-        disabled={false}
-        selected={false}
       >
         Select
       </button>
@@ -83,8 +81,6 @@ exports[`<Dropdown /> create dropdown component 2`] = `
     >
       <button
         className="fd-button fd-dropdown__control fd-button--compact"
-        disabled={false}
-        selected={false}
       >
         Select
       </button>
@@ -152,8 +148,6 @@ exports[`<Dropdown /> create dropdown component 3`] = `
     >
       <button
         className="fd-button fd-button--standard fd-dropdown__control"
-        disabled={false}
-        selected={false}
       >
         Select
       </button>
@@ -222,8 +216,6 @@ exports[`<Dropdown /> create dropdown component 4`] = `
     >
       <button
         className="fd-button fd-dropdown__control sap-icon--filter"
-        disabled={false}
-        selected={false}
       >
         Select
       </button>
@@ -295,7 +287,6 @@ exports[`<Dropdown /> create dropdown component 5`] = `
       <button
         className="fd-button fd-dropdown__control sap-icon--filter is-disabled"
         disabled={true}
-        selected={false}
       >
         Select
       </button>
diff --git a/src/InputGroup/InputGroup.js b/src/InputGroup/InputGroup.js
index 551c61a6b..c0d7f101c 100644
--- a/src/InputGroup/InputGroup.js
+++ b/src/InputGroup/InputGroup.js
@@ -268,9 +268,7 @@ InputGroup.propTypes = {
 };
 
 InputGroup.defaultProps = {
-    actions: false,
     addonPos: 'after',
-    compact: false,
     inputType: 'text',
     inputValue: ''
 };
diff --git a/src/InputGroup/__snapshots__/InputGroup.test.js.snap b/src/InputGroup/__snapshots__/InputGroup.test.js.snap
index 444a65d22..2a56df382 100644
--- a/src/InputGroup/__snapshots__/InputGroup.test.js.snap
+++ b/src/InputGroup/__snapshots__/InputGroup.test.js.snap
@@ -271,8 +271,6 @@ exports[`<InputGroup /> create input group items 13`] = `
   >
     <button
       className="fd-button--light"
-      disabled={false}
-      selected={false}
     >
       Button
     </button>
@@ -295,8 +293,6 @@ exports[`<InputGroup /> create input group items 14`] = `
   >
     <button
       className="fd-button--light"
-      disabled={false}
-      selected={false}
     >
       Button
     </button>
diff --git a/src/ListGroup/__snapshots__/ListGroup.test.js.snap b/src/ListGroup/__snapshots__/ListGroup.test.js.snap
index 5ac54ee45..94ae791a1 100644
--- a/src/ListGroup/__snapshots__/ListGroup.test.js.snap
+++ b/src/ListGroup/__snapshots__/ListGroup.test.js.snap
@@ -49,8 +49,6 @@ exports[`<ListGroup /> create list group 1`] = `
     >
       <button
         className="fd-button fd-button--standard sap-icon--edit"
-        disabled={false}
-        selected={false}
       />
     </span>
   </li>
@@ -70,8 +68,6 @@ exports[`<ListGroup /> create list group 2`] = `
     >
       <button
         className="fd-button fd-button--standard sap-icon--edit"
-        disabled={false}
-        selected={false}
       />
     </span>
   </li>
diff --git a/src/LocalizationEditor/LocalizationEditor.js b/src/LocalizationEditor/LocalizationEditor.js
index 5f44bc16e..39f7ec354 100644
--- a/src/LocalizationEditor/LocalizationEditor.js
+++ b/src/LocalizationEditor/LocalizationEditor.js
@@ -139,11 +139,6 @@ LocalizationEditor.propTypes = {
     textarea: PropTypes.bool
 };
 
-LocalizationEditor.defaultProps = {
-    compact: false,
-    textarea: false
-};
-
 LocalizationEditor.propDescriptions = {
     control: 'An object of shape `{ label: string, placeholder: string, language: string, labelProps: object, inputProps: object, buttonProps: object }` containing the values of the control localization editor.',
     menu: 'An array of objects that represent the values of the elements in the dropdown menu. The shape of the objects in the array is `{ placeholder: string, language: string, inputProps: object }`.',
diff --git a/src/Menu/Menu.js b/src/Menu/Menu.js
index 8efbbaa15..94588acf2 100644
--- a/src/Menu/Menu.js
+++ b/src/Menu/Menu.js
@@ -24,10 +24,6 @@ Menu.propTypes = {
     className: PropTypes.string
 };
 
-Menu.defaultProps = {
-    addonBefore: false
-};
-
 Menu.propDescriptions = {
     addonBefore: 'Set to **true** enables menu items with add-on before.'
 };
@@ -106,11 +102,6 @@ MenuItem.propTypes = {
     urlProps: PropTypes.object
 };
 
-MenuItem.defaultProps = {
-    isLink: false,
-    separator: false
-};
-
 MenuItem.propDescriptions = {
     addon: 'Name of the SAP icon to be applied as an add-on before.',
     addonProps: 'Additional props to be spread to the add-on section.',
diff --git a/src/Modal/Modal.js b/src/Modal/Modal.js
index b5660efac..5218d87f7 100644
--- a/src/Modal/Modal.js
+++ b/src/Modal/Modal.js
@@ -97,10 +97,6 @@ Modal.propTypes = {
     titleProps: PropTypes.object
 };
 
-Modal.defaultProps = {
-    show: false
-};
-
 Modal.propDescriptions = {
     actions: 'Node(s) to render within the footer of the dialog.',
     bodyProps: 'Additional props to be spread to the body section of the dialog.',
diff --git a/src/MultiInput/MultiInput.js b/src/MultiInput/MultiInput.js
index fc44a1f89..fb2f7a485 100644
--- a/src/MultiInput/MultiInput.js
+++ b/src/MultiInput/MultiInput.js
@@ -199,10 +199,6 @@ MultiInput.propTypes = {
     onTagsUpdate: PropTypes.func
 };
 
-MultiInput.defaultProps = {
-    compact: false
-};
-
 MultiInput.propDescriptions = {
     data: 'Collection of items to display in the list.',
     onTagsUpdate: 'Callback function when a tag is added or removed. Returns array of tags selected.',
diff --git a/src/Panel/Panel.js b/src/Panel/Panel.js
index 4e561862d..29be9fff0 100644
--- a/src/Panel/Panel.js
+++ b/src/Panel/Panel.js
@@ -23,10 +23,6 @@ Panel.propTypes = {
     colSpan: CustomPropTypes.range(1, 6)
 };
 
-Panel.defaultProps = {
-    colSpan: null
-};
-
 Panel.propDescriptions = {
     colSpan: 'The number of columns to span inside a `PanelGrid`.'
 };
@@ -59,11 +55,6 @@ PanelGrid.propTypes = {
     nogap: PropTypes.bool
 };
 
-PanelGrid.defaultProps = {
-    nogap: false,
-    cols: null
-};
-
 PanelGrid.propDescriptions = {
     cols: 'The number of columns in the grid.',
     nogap: 'Set to **true** to remove the margins between the panels.'
diff --git a/src/Panel/__snapshots__/Panel.test.js.snap b/src/Panel/__snapshots__/Panel.test.js.snap
index e620cbe4f..8df26e282 100644
--- a/src/Panel/__snapshots__/Panel.test.js.snap
+++ b/src/Panel/__snapshots__/Panel.test.js.snap
@@ -26,8 +26,6 @@ exports[`<Panel /> create panels 1`] = `
     >
       <button
         className="fd-button fd-button--compact sap-icon--add"
-        disabled={false}
-        selected={false}
       >
         Add New Button
       </button>
@@ -162,8 +160,6 @@ exports[`<Panel /> create panels 3`] = `
     >
       <button
         className="fd-button fd-button--compact sap-icon--add"
-        disabled={false}
-        selected={false}
       >
         Add New Button
       </button>
diff --git a/src/SearchInput/SearchInput.js b/src/SearchInput/SearchInput.js
index 7ca859bf0..02de93abf 100644
--- a/src/SearchInput/SearchInput.js
+++ b/src/SearchInput/SearchInput.js
@@ -281,12 +281,6 @@ SearchInput.propTypes = {
     onSearch: PropTypes.func
 };
 
-SearchInput.defaultProps = {
-    compact: false,
-    inShellbar: false,
-    noSearchBtn: false
-};
-
 SearchInput.propDescriptions = {
     noSearchBtn: 'Set to **true** to render without a search button.',
     onEnter: 'Callback function when the user hits the <Enter> key.',
diff --git a/src/Shellbar/Shellbar.js b/src/Shellbar/Shellbar.js
index ee44b27d3..d02928942 100644
--- a/src/Shellbar/Shellbar.js
+++ b/src/Shellbar/Shellbar.js
@@ -404,12 +404,6 @@ Shellbar.propTypes = {
     subtitle: PropTypes.string
 };
 
-Shellbar.defaultProps = {
-    actions: null,
-    copilot: false,
-    logoSAP: false
-};
-
 Shellbar.propDescriptions = {
     actions: 'Holds all product actions and links.',
     copilot: 'For use with applications that utilize CoPilot.',
diff --git a/src/SideNavigation/SideNavigation.js b/src/SideNavigation/SideNavigation.js
index b7510a0f2..403e1dd5b 100644
--- a/src/SideNavigation/SideNavigation.js
+++ b/src/SideNavigation/SideNavigation.js
@@ -24,10 +24,6 @@ SideNav.propTypes = {
     icons: PropTypes.bool
 };
 
-SideNav.defaultProps = {
-    icons: false
-};
-
 SideNav.propDescriptions = {
     icons: 'Set to **true** enables side navigation collapsed with icons.'
 };
diff --git a/src/Tile/Tile.js b/src/Tile/Tile.js
index bf1a6dddd..f3a6db486 100644
--- a/src/Tile/Tile.js
+++ b/src/Tile/Tile.js
@@ -45,10 +45,6 @@ Tile.propTypes = {
     rowSpan: PropTypes.number
 };
 
-Tile.defaultProps = {
-    disabled: false
-};
-
 Tile.propDescriptions = {
     backgroundColor: 'Sets a background color class.',
     colorAccent: 'Sets a background color accent class. Options include numbers from 1 to 9.',
@@ -136,10 +132,6 @@ ProductTile.propTypes = {
     disabled: PropTypes.bool
 };
 
-ProductTile.defaultProps = {
-    disabled: false
-};
-
 export const ProductTileContent = props => {
     const { title, children, className, titleProps, ...rest } = props;
 
diff --git a/src/Tile/__snapshots__/Tile.test.js.snap b/src/Tile/__snapshots__/Tile.test.js.snap
index fb74dbf81..9a4b2869e 100644
--- a/src/Tile/__snapshots__/Tile.test.js.snap
+++ b/src/Tile/__snapshots__/Tile.test.js.snap
@@ -92,8 +92,6 @@ exports[`<Tile /> create tile component 4`] = `
       >
         <button
           className="fd-button fd-button--standard sap-icon--vertical-grip"
-          disabled={false}
-          selected={false}
         />
       </div>
       <div
@@ -200,8 +198,6 @@ exports[`<Tile /> create tile component 6`] = `
       >
         <button
           className="fd-button fd-button--standard sap-icon--vertical-grip"
-          disabled={false}
-          selected={false}
         />
       </div>
       <div
diff --git a/src/Time/Time.js b/src/Time/Time.js
index a54ce2ede..62a3cb7d9 100644
--- a/src/Time/Time.js
+++ b/src/Time/Time.js
@@ -286,12 +286,6 @@ TimeItem.propTypes = {
     value: PropTypes.oneOfType([PropTypes.number, PropTypes.string])
 };
 
-TimeItem.defaultProps = {
-    id: '',
-    value: null,
-    arialabel: ''
-};
-
 export class Time extends Component {
     constructor(props) {
         super(props);
@@ -494,12 +488,9 @@ Time.propTypes = {
 };
 
 Time.defaultProps = {
-    id: '',
     showHour: true,
     showMinute: true,
     showSecond: true,
-    format12Hours: false,
-    disabled: false,
     spinners: true,
     time: {
         hour: '00',
diff --git a/src/Time/__snapshots__/Time.test.js.snap b/src/Time/__snapshots__/Time.test.js.snap
index f1bdd1e5c..fe975335c 100644
--- a/src/Time/__snapshots__/Time.test.js.snap
+++ b/src/Time/__snapshots__/Time.test.js.snap
@@ -3,7 +3,6 @@
 exports[`<Time /> create time component 1`] = `
 <div
   className="fd-time"
-  id=""
 >
   <div
     className="fd-time__item"
@@ -129,7 +128,6 @@ exports[`<Time /> create time component 1`] = `
 exports[`<Time /> create time component 2`] = `
 <div
   className="fd-time"
-  id=""
 >
   <div
     className="fd-time__item"
@@ -255,7 +253,6 @@ exports[`<Time /> create time component 2`] = `
 exports[`<Time /> create time component 3`] = `
 <div
   className="fd-time"
-  id=""
 >
   <div
     className="fd-time__item"
@@ -381,7 +378,6 @@ exports[`<Time /> create time component 3`] = `
 exports[`<Time /> create time component 4`] = `
 <div
   className="fd-time"
-  id=""
 >
   <div
     className="fd-time__item"
@@ -507,7 +503,6 @@ exports[`<Time /> create time component 4`] = `
 exports[`<Time /> create time component 5`] = `
 <div
   className="fd-time"
-  id=""
 >
   <div
     className="fd-time__item"
@@ -670,7 +665,6 @@ exports[`<Time /> create time component 5`] = `
 exports[`<Time /> create time component 6`] = `
 <div
   className="fd-time"
-  id=""
 >
   <div
     className="fd-time__item"
@@ -742,7 +736,6 @@ exports[`<Time /> create time component 6`] = `
 exports[`<Time /> create time component 7`] = `
 <div
   className="fd-time"
-  id=""
 >
   <div
     className="fd-time__item"
@@ -830,7 +823,6 @@ exports[`<Time /> create time component 7`] = `
 exports[`<Time /> create time component 8`] = `
 <div
   className="fd-time"
-  id=""
 >
   <div
     className="fd-time__item"
diff --git a/src/TimePicker/TimePicker.js b/src/TimePicker/TimePicker.js
index 856c64c38..4fbeeacf2 100644
--- a/src/TimePicker/TimePicker.js
+++ b/src/TimePicker/TimePicker.js
@@ -26,27 +26,14 @@ class TimePickerItem extends Component {
         value: PropTypes.oneOfType([PropTypes.number, PropTypes.string]),
         onChange: PropTypes.func
     };
-    static defaultProps = {
-        value: null,
-        inputId: '',
-        buttonID: '',
-        style: '',
-        isValid: false,
-        length: 0,
-        disabled: false,
-        format12Hours: false,
-        showHour: false,
-        showMinute: false,
-        showSecond: false
-    };
 
     constructor(props) {
         super(props);
         var length = this.setLength(props);
         this.state = {
             value: this.props.value,
-            inputId: this.props.id + '-input',
-            buttonID: this.props.id + '-button',
+            inputId: this.props.id && this.props.id + '-input',
+            buttonID: this.props.id && this.props.id + '-button',
             style: 'fd-input',
             isValid: false,
             length: length
@@ -292,15 +279,6 @@ TimePickerItem.propTypes = {
     onChange: PropTypes.func
 };
 
-TimePickerItem.defaultProps = {
-    value: null,
-    inputId: '',
-    buttonID: '',
-    style: '',
-    isValid: false,
-    length: 0
-};
-
 export class TimePicker extends React.Component {
     constructor(props) {
         super(props);
@@ -459,20 +437,16 @@ TimePicker.propTypes = {
 };
 
 TimePicker.defaultProps = {
-    id: '',
     showHour: true,
     showMinute: true,
     showSecond: true,
-    format12Hours: false,
-    disabled: false,
     spinners: true,
     time: {
         hour: '00',
         minute: '00',
         second: '00',
         meridiem: 0
-    },
-    value: ''
+    }
 };
 
 TimePicker.propDescriptions = {
diff --git a/src/TimePicker/__snapshots__/TimePicker.test.js.snap b/src/TimePicker/__snapshots__/TimePicker.test.js.snap
index 1bcef67d5..88308af7d 100644
--- a/src/TimePicker/__snapshots__/TimePicker.test.js.snap
+++ b/src/TimePicker/__snapshots__/TimePicker.test.js.snap
@@ -29,7 +29,6 @@ exports[`<TimePicker /> create time picker 1`] = `
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="hh:mm:ss"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -41,7 +40,6 @@ exports[`<TimePicker /> create time picker 1`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
                 id="myTime-button"
               />
             </span>
@@ -185,7 +183,6 @@ exports[`<TimePicker /> create time picker 1`] = `
 exports[`<TimePicker /> create time picker 2`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -207,11 +204,9 @@ exports[`<TimePicker /> create time picker 2`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="hh:mm:ss am/pm"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -223,8 +218,6 @@ exports[`<TimePicker /> create time picker 2`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
-                id="-button"
               />
             </span>
           </div>
@@ -404,7 +397,6 @@ exports[`<TimePicker /> create time picker 2`] = `
 exports[`<TimePicker /> create time picker 3`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -426,11 +418,9 @@ exports[`<TimePicker /> create time picker 3`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="hh:mm"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -442,8 +432,6 @@ exports[`<TimePicker /> create time picker 3`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
-                id="-button"
               />
             </span>
           </div>
@@ -548,7 +536,6 @@ exports[`<TimePicker /> create time picker 3`] = `
 exports[`<TimePicker /> create time picker 4`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -570,7 +557,6 @@ exports[`<TimePicker /> create time picker 4`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="hh:mm:ss"
@@ -587,7 +573,6 @@ exports[`<TimePicker /> create time picker 4`] = `
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
                 disabled={true}
-                id="-button"
               />
             </span>
           </div>
@@ -730,7 +715,6 @@ exports[`<TimePicker /> create time picker 4`] = `
 exports[`<TimePicker /> create time picker 5`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -752,11 +736,9 @@ exports[`<TimePicker /> create time picker 5`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="hh:mm:ss am/pm"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -768,8 +750,6 @@ exports[`<TimePicker /> create time picker 5`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
-                id="-button"
               />
             </span>
           </div>
@@ -949,7 +929,6 @@ exports[`<TimePicker /> create time picker 5`] = `
 exports[`<TimePicker /> create time picker 6`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -971,11 +950,9 @@ exports[`<TimePicker /> create time picker 6`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="hh:mm:ss am/pm"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -987,8 +964,6 @@ exports[`<TimePicker /> create time picker 6`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
-                id="-button"
               />
             </span>
           </div>
@@ -1168,7 +1143,6 @@ exports[`<TimePicker /> create time picker 6`] = `
 exports[`<TimePicker /> create time picker 7`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -1190,11 +1164,9 @@ exports[`<TimePicker /> create time picker 7`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="hh:mm:ss am/pm"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -1206,8 +1178,6 @@ exports[`<TimePicker /> create time picker 7`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
-                id="-button"
               />
             </span>
           </div>
@@ -1387,7 +1357,6 @@ exports[`<TimePicker /> create time picker 7`] = `
 exports[`<TimePicker /> create time picker 8`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -1409,11 +1378,9 @@ exports[`<TimePicker /> create time picker 8`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="mm:ss"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -1425,8 +1392,6 @@ exports[`<TimePicker /> create time picker 8`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
-                id="-button"
               />
             </span>
           </div>
@@ -1531,7 +1496,6 @@ exports[`<TimePicker /> create time picker 8`] = `
 exports[`<TimePicker /> create time picker 9`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -1553,11 +1517,9 @@ exports[`<TimePicker /> create time picker 9`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="mm:ss am/pm"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -1569,8 +1531,6 @@ exports[`<TimePicker /> create time picker 9`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
-                id="-button"
               />
             </span>
           </div>
@@ -1712,7 +1672,6 @@ exports[`<TimePicker /> create time picker 9`] = `
 exports[`<TimePicker /> create time picker 10`] = `
 <div
   className="fd-time-picker"
-  id=""
 >
   <div
     className="fd-popover fd-popover--no-arrow"
@@ -1734,11 +1693,9 @@ exports[`<TimePicker /> create time picker 10`] = `
           >
             <input
               className="fd-input"
-              id="-input"
               onBlur={[Function]}
               onChange={[Function]}
               placeholder="hh:ss am/pm"
-              readOnly={false}
               type="text"
               value=""
             />
@@ -1750,8 +1707,6 @@ exports[`<TimePicker /> create time picker 10`] = `
                 aria-expanded="false"
                 aria-haspopup="true"
                 className="fd-button--light fd-button--compact sap-icon--fob-watch fd-popover__control"
-                disabled={false}
-                id="-button"
               />
             </span>
           </div>
diff --git a/src/Toggle/Toggle.js b/src/Toggle/Toggle.js
index de8905537..9ca8ecf6c 100644
--- a/src/Toggle/Toggle.js
+++ b/src/Toggle/Toggle.js
@@ -6,7 +6,7 @@ import {TOGGLE_SIZES} from '../utils/constants';
 export class Toggle extends React.Component {
     constructor(props) {
         super(props);
-        this.state = { checked: props.checked };
+        this.state = { checked: !!props.checked };
     }
 
     handleChange = (e) => {
@@ -73,8 +73,6 @@ Toggle.propTypes = {
 };
 
 Toggle.defaultProps = {
-    checked: false,
-    disabled: false,
     onChange: () => {}
 };
 
diff --git a/src/Toggle/__snapshots__/Toggle.test.js.snap b/src/Toggle/__snapshots__/Toggle.test.js.snap
index f6169feae..bc9a938e8 100644
--- a/src/Toggle/__snapshots__/Toggle.test.js.snap
+++ b/src/Toggle/__snapshots__/Toggle.test.js.snap
@@ -12,7 +12,6 @@ exports[`<Toggle /> create toggle component 1`] = `
     >
       <input
         checked={false}
-        disabled={false}
         onChange={[Function]}
         type="checkbox"
       />
@@ -38,7 +37,6 @@ exports[`<Toggle /> create toggle component 2`] = `
     >
       <input
         checked={true}
-        disabled={false}
         onChange={[Function]}
         type="checkbox"
       />

From 9f500977ea948fdc95dd1226a73e4f5c38dd905a Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Thu, 31 Jan 2019 09:28:05 -0800
Subject: [PATCH 25/29] Make description optional

---
 src/ActionBar/ActionBar.js                         | 10 ++++++----
 src/ActionBar/ActionBar.test.js                    |  2 +-
 src/ActionBar/__snapshots__/ActionBar.test.js.snap |  4 ----
 3 files changed, 7 insertions(+), 9 deletions(-)

diff --git a/src/ActionBar/ActionBar.js b/src/ActionBar/ActionBar.js
index 15e95287a..f3e8311ba 100644
--- a/src/ActionBar/ActionBar.js
+++ b/src/ActionBar/ActionBar.js
@@ -70,17 +70,19 @@ export const ActionBarHeader = ({ className, description, descriptionProps, titl
             <h1
                 {...titleProps}
                 className='fd-action-bar__title'>{title}</h1>
-            <p
-                {...descriptionProps}
-                className='fd-action-bar__description'>{description} </p>
+            {description &&
+                <p
+                    {...descriptionProps}
+                    className='fd-action-bar__description'>{description}</p>
+            }
         </div>
     );
 };
 
 ActionBarHeader.propTypes = {
-    description: PropTypes.string.isRequired,
     title: PropTypes.string.isRequired,
     className: PropTypes.string,
+    description: PropTypes.string,
     descriptionProps: PropTypes.object,
     titleProps: PropTypes.object
 };
diff --git a/src/ActionBar/ActionBar.test.js b/src/ActionBar/ActionBar.test.js
index 5d8dc1599..60f2060b2 100644
--- a/src/ActionBar/ActionBar.test.js
+++ b/src/ActionBar/ActionBar.test.js
@@ -142,7 +142,7 @@ describe('<ActionBar />', () => {
         test('should allow props to be spread to the ActionBarHeader component\'s p element', () => {
             const element = mount(
                 <ActionBarHeader
-                    description=''
+                    description='Sample Description'
                     descriptionProps={{'data-sample': 'Sample'}}
                     title='' />
             );
diff --git a/src/ActionBar/__snapshots__/ActionBar.test.js.snap b/src/ActionBar/__snapshots__/ActionBar.test.js.snap
index b562e2439..33da7a377 100644
--- a/src/ActionBar/__snapshots__/ActionBar.test.js.snap
+++ b/src/ActionBar/__snapshots__/ActionBar.test.js.snap
@@ -23,7 +23,6 @@ exports[`<ActionBar /> create basic Action Bar 1`] = `
       className="fd-action-bar__description"
     >
       Action Bar Description
-       
     </p>
   </div>
   <div
@@ -59,7 +58,6 @@ exports[`<ActionBar /> create basic Action Bar 2`] = `
       className="fd-action-bar__description"
     >
       Action Bar Description
-       
     </p>
   </div>
   <div
@@ -102,7 +100,6 @@ exports[`<ActionBar /> create basic mobile Action Bar 1`] = `
         className="fd-action-bar__description"
       >
         Action Bar Description
-         
       </p>
     </div>
     <div
@@ -146,7 +143,6 @@ exports[`<ActionBar /> create basic mobile Action Bar 2`] = `
         className="fd-action-bar__description"
       >
         Action Bar Description
-         
       </p>
     </div>
     <div

From bdbca88081bd6d87a171f984fc30000cbfcf075d Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Thu, 31 Jan 2019 11:35:39 -0800
Subject: [PATCH 26/29] Add default prop for funcs

---
 src/Calendar/Calendar.js                      | 12 +++----
 src/Forms/Forms.js                            | 18 -----------
 src/Forms/__snapshots__/Forms.test.js.snap    | 13 --------
 src/Icon/Icon.js                              |  4 ---
 src/Icon/__snapshots__/Icon.test.js.snap      |  1 -
 src/MultiInput/MultiInput.js                  |  4 +++
 .../__snapshots__/Popover.test.js.snap        |  4 ---
 src/SearchInput/SearchInput.js                |  8 +++--
 .../__snapshots__/Shellbar.test.js.snap       | 32 -------------------
 src/Time/Time.js                              |  8 ++---
 10 files changed, 18 insertions(+), 86 deletions(-)

diff --git a/src/Calendar/Calendar.js b/src/Calendar/Calendar.js
index a1777e7b4..ce7ef25de 100644
--- a/src/Calendar/Calendar.js
+++ b/src/Calendar/Calendar.js
@@ -373,13 +373,7 @@ export class Calendar extends Component {
     }
 
     returnDateSelected = (dates) => {
-        if (typeof this.props.onChange !== 'undefined') {
-            if (this.props.enableRangeSelection === true) {
-                this.props.onChange(dates);
-            } else {
-                this.props.onChange(dates);
-            }
-        }
+        this.props.onChange(dates);
     }
 
     generateNavigation = () => {
@@ -563,6 +557,10 @@ Calendar.propTypes = {
     onChange: PropTypes.func
 };
 
+Calendar.defaultProps = {
+    onChange: () => {}
+};
+
 Calendar.propDescriptions = {
     blockedDates: 'Blocks dates that are in between the blocked dates.',
     disableAfterDate: 'Disables dates of a calendar that come after the specified date.',
diff --git a/src/Forms/Forms.js b/src/Forms/Forms.js
index a110d67a0..ee8fd84f1 100644
--- a/src/Forms/Forms.js
+++ b/src/Forms/Forms.js
@@ -47,11 +47,6 @@ FormItem.propTypes = {
     isInline: PropTypes.bool
 };
 
-FormItem.defaultProps = {
-    isCheck: false,
-    isInline: false
-};
-
 FormItem.propDescriptions = {
     isCheck: 'Set to **true** to render an `<input>` with `type` of **checkbox**.',
     isInline: 'Set to **true** to display radio buttons and checkboxes in a row.'
@@ -79,10 +74,6 @@ FormLabel.propTypes = {
     required: PropTypes.bool
 };
 
-FormLabel.defaultProps = {
-    required: false
-};
-
 FormLabel.propDescriptions = {
     required: 'Set to **true** for required input fields.'
 };
@@ -217,10 +208,6 @@ FormSelect.propTypes = {
     disabled: PropTypes.bool
 };
 
-FormSelect.defaultProps = {
-    disabled: false
-};
-
 // ------------------------------------------------- Form Radio ----------------------------------------------
 export class FormRadio extends Component {
     constructor(props) {
@@ -288,11 +275,6 @@ FormRadio.propTypes = {
     isInline: PropTypes.bool
 };
 
-FormRadio.defaultProps = {
-    disabled: false,
-    isInline: false
-};
-
 FormRadio.propDescriptions = {
     defaultChecked: 'The `id` of the element selected by default.',
     inputs: 'Configuration settings for each radio button in the group.',
diff --git a/src/Forms/__snapshots__/Forms.test.js.snap b/src/Forms/__snapshots__/Forms.test.js.snap
index ad96f07c6..f4679f540 100644
--- a/src/Forms/__snapshots__/Forms.test.js.snap
+++ b/src/Forms/__snapshots__/Forms.test.js.snap
@@ -8,7 +8,6 @@ exports[`<Forms /> create form item 1`] = `
     className="fd-form__item fd-form__item--inline fd-form__item--check"
   >
     <label
-      aria-required={false}
       className="fd-form__label"
       forAttr="input-1"
     >
@@ -63,7 +62,6 @@ exports[`<Forms /> create form item 1`] = `
     </span>
     <select
       className="fd-form__control"
-      disabled={false}
       id="select-1"
     >
       <option
@@ -105,7 +103,6 @@ exports[`<Forms /> create form item 2`] = `
     className="fd-form__item fd-form__item--inline fd-form__item--check"
   >
     <label
-      aria-required={false}
       className="fd-form__label"
       forAttr="input-1"
     >
@@ -138,7 +135,6 @@ exports[`<Forms /> create form item 3`] = `
       <input
         checked={false}
         className="fd-form__control"
-        disabled={false}
         id="radio-1"
         name="radio-1"
         onChange={[Function]}
@@ -158,7 +154,6 @@ exports[`<Forms /> create form item 3`] = `
       <input
         checked={true}
         className="fd-form__control"
-        disabled={false}
         id="radio-2"
         name="radio-2"
         onChange={[Function]}
@@ -178,7 +173,6 @@ exports[`<Forms /> create form item 3`] = `
       <input
         checked={false}
         className="fd-form__control"
-        disabled={false}
         id="radio-3"
         name="radio-3"
         onChange={[Function]}
@@ -212,7 +206,6 @@ exports[`<Forms /> create form item 4`] = `
       <input
         checked={false}
         className="fd-form__control"
-        disabled={false}
         id="radio-1"
         name="radio-1"
         onChange={[Function]}
@@ -232,7 +225,6 @@ exports[`<Forms /> create form item 4`] = `
       <input
         checked={true}
         className="fd-form__control"
-        disabled={false}
         id="radio-2"
         name="radio-2"
         onChange={[Function]}
@@ -252,7 +244,6 @@ exports[`<Forms /> create form item 4`] = `
       <input
         checked={false}
         className="fd-form__control"
-        disabled={false}
         id="radio-3"
         name="radio-3"
         onChange={[Function]}
@@ -348,7 +339,6 @@ exports[`<Forms /> create form item 4`] = `
         <input
           checked={false}
           className="fd-form__control"
-          disabled={false}
           id="radio-7"
           name="radio-7"
           onChange={[Function]}
@@ -368,7 +358,6 @@ exports[`<Forms /> create form item 4`] = `
         <input
           checked={false}
           className="fd-form__control"
-          disabled={false}
           id="radio-8"
           name="radio-8"
           onChange={[Function]}
@@ -388,7 +377,6 @@ exports[`<Forms /> create form item 4`] = `
         <input
           checked={true}
           className="fd-form__control"
-          disabled={false}
           id="radio-9"
           name="radio-9"
           onChange={[Function]}
@@ -476,7 +464,6 @@ exports[`<Forms /> create form item 4`] = `
       value=""
     />
     <label
-      aria-required={false}
       className="fd-form__label blue"
       forAttr="checkbox-1"
     >
diff --git a/src/Icon/Icon.js b/src/Icon/Icon.js
index a3858f7f1..77def5b59 100644
--- a/src/Icon/Icon.js
+++ b/src/Icon/Icon.js
@@ -20,10 +20,6 @@ export const Icon = ({ glyph, size, clickHandler, className, ...props }) => {
     );
 };
 
-Icon.defaultProps = {
-    clickHandler: () => {}
-};
-
 Icon.propTypes = {
     glyph: PropTypes.string.isRequired,
     className: PropTypes.string,
diff --git a/src/Icon/__snapshots__/Icon.test.js.snap b/src/Icon/__snapshots__/Icon.test.js.snap
index b129d1974..7feee51d7 100644
--- a/src/Icon/__snapshots__/Icon.test.js.snap
+++ b/src/Icon/__snapshots__/Icon.test.js.snap
@@ -10,6 +10,5 @@ exports[`<Icon /> create icon 1`] = `
 exports[`<Icon /> create icon 2`] = `
 <span
   className="sap-icon--cart sap-icon--s"
-  onClick={[Function]}
 />
 `;
diff --git a/src/MultiInput/MultiInput.js b/src/MultiInput/MultiInput.js
index fb2f7a485..d5734e7b9 100644
--- a/src/MultiInput/MultiInput.js
+++ b/src/MultiInput/MultiInput.js
@@ -199,6 +199,10 @@ MultiInput.propTypes = {
     onTagsUpdate: PropTypes.func
 };
 
+MultiInput.defaultProps = {
+    onTagsUpdate: () => {}
+};
+
 MultiInput.propDescriptions = {
     data: 'Collection of items to display in the list.',
     onTagsUpdate: 'Callback function when a tag is added or removed. Returns array of tags selected.',
diff --git a/src/Popover/__snapshots__/Popover.test.js.snap b/src/Popover/__snapshots__/Popover.test.js.snap
index 29ee4d6ec..b17f6c3f4 100644
--- a/src/Popover/__snapshots__/Popover.test.js.snap
+++ b/src/Popover/__snapshots__/Popover.test.js.snap
@@ -11,7 +11,6 @@ exports[`<Popover /> create Popover 1`] = `
   >
     <span
       className="sap-icon--cart sap-icon--xl"
-      onClick={[Function]}
     />
   </div>
   <div
@@ -74,7 +73,6 @@ exports[`<Popover /> create Popover 2`] = `
   >
     <span
       className="sap-icon--cart sap-icon--xl"
-      onClick={[Function]}
     />
   </div>
   <div
@@ -136,7 +134,6 @@ exports[`<Popover /> create Popover 3`] = `
   >
     <span
       className="sap-icon--cart sap-icon--xl"
-      onClick={[Function]}
     />
   </div>
   <div
@@ -198,7 +195,6 @@ exports[`<Popover /> create Popover 4`] = `
   >
     <span
       className="sap-icon--cart sap-icon--xl"
-      onClick={[Function]}
     />
   </div>
   <div
diff --git a/src/SearchInput/SearchInput.js b/src/SearchInput/SearchInput.js
index 02de93abf..e5dad746e 100644
--- a/src/SearchInput/SearchInput.js
+++ b/src/SearchInput/SearchInput.js
@@ -125,7 +125,6 @@ export class SearchInput extends Component {
         const {
             placeholder,
             inShellbar,
-            onSearch,
             onEnter,
             searchList,
             onChange,
@@ -277,8 +276,11 @@ SearchInput.propTypes = {
     searchBtnProps: PropTypes.object,
     searchList: PropTypes.array,
     onChange: PropTypes.func,
-    onEnter: PropTypes.func,
-    onSearch: PropTypes.func
+    onEnter: PropTypes.func
+};
+
+SearchInput.defaultProps = {
+    onEnter: () => {}
 };
 
 SearchInput.propDescriptions = {
diff --git a/src/Shellbar/__snapshots__/Shellbar.test.js.snap b/src/Shellbar/__snapshots__/Shellbar.test.js.snap
index 4524318a1..6384ad209 100644
--- a/src/Shellbar/__snapshots__/Shellbar.test.js.snap
+++ b/src/Shellbar/__snapshots__/Shellbar.test.js.snap
@@ -76,7 +76,6 @@ exports[`<Shellbar /> create shellbar 1`] = `
                     >
                       <span
                         className="sap-icon--action-settings sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Settings
@@ -89,7 +88,6 @@ exports[`<Shellbar /> create shellbar 1`] = `
                     >
                       <span
                         className="sap-icon--log sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Sign Out
@@ -182,7 +180,6 @@ exports[`<Shellbar /> create shellbar 2`] = `
                     >
                       <span
                         className="sap-icon--action-settings sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Settings
@@ -195,7 +192,6 @@ exports[`<Shellbar /> create shellbar 2`] = `
                     >
                       <span
                         className="sap-icon--log sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Sign Out
@@ -268,7 +264,6 @@ exports[`<Shellbar /> create shellbar 3`] = `
                   >
                     <span
                       className="sap-icon--log"
-                      onClick={[Function]}
                     />
                        
                     Application A
@@ -541,7 +536,6 @@ exports[`<Shellbar /> create shellbar 3`] = `
                     >
                       <span
                         className="sap-icon--search"
-                        onClick={[Function]}
                       />
                        
                       Search
@@ -554,7 +548,6 @@ exports[`<Shellbar /> create shellbar 3`] = `
                     >
                       <span
                         className="sap-icon--settings"
-                        onClick={[Function]}
                       >
                         <span
                           aria-label="Unread count"
@@ -574,7 +567,6 @@ exports[`<Shellbar /> create shellbar 3`] = `
                     >
                       <span
                         className="sap-icon--bell"
-                        onClick={[Function]}
                       >
                         <span
                           aria-label="Unread count"
@@ -594,7 +586,6 @@ exports[`<Shellbar /> create shellbar 3`] = `
                     >
                       <span
                         className="sap-icon--grid"
-                        onClick={[Function]}
                       />
                        
                       Product Switcher
@@ -651,7 +642,6 @@ exports[`<Shellbar /> create shellbar 3`] = `
                     >
                       <span
                         className="sap-icon--action-settings sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Settings
@@ -664,7 +654,6 @@ exports[`<Shellbar /> create shellbar 3`] = `
                     >
                       <span
                         className="sap-icon--log sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Sign Out
@@ -803,7 +792,6 @@ exports[`<Shellbar /> create shellbar 4`] = `
                   >
                     <span
                       className="sap-icon--log"
-                      onClick={[Function]}
                     />
                        
                     Application A
@@ -1052,7 +1040,6 @@ exports[`<Shellbar /> create shellbar 4`] = `
                     >
                       <span
                         className="sap-icon--search"
-                        onClick={[Function]}
                       />
                        
                       Search
@@ -1065,7 +1052,6 @@ exports[`<Shellbar /> create shellbar 4`] = `
                     >
                       <span
                         className="sap-icon--settings"
-                        onClick={[Function]}
                       >
                         <span
                           aria-label="Unread count"
@@ -1085,7 +1071,6 @@ exports[`<Shellbar /> create shellbar 4`] = `
                     >
                       <span
                         className="sap-icon--bell"
-                        onClick={[Function]}
                       />
                        
                       Notifications
@@ -1098,7 +1083,6 @@ exports[`<Shellbar /> create shellbar 4`] = `
                     >
                       <span
                         className="sap-icon--grid"
-                        onClick={[Function]}
                       />
                        
                       Product Switcher
@@ -1155,7 +1139,6 @@ exports[`<Shellbar /> create shellbar 4`] = `
                     >
                       <span
                         className="sap-icon--action-settings sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Settings
@@ -1168,7 +1151,6 @@ exports[`<Shellbar /> create shellbar 4`] = `
                     >
                       <span
                         className="sap-icon--log sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Sign Out
@@ -1307,7 +1289,6 @@ exports[`<Shellbar /> create shellbar 5`] = `
                   >
                     <span
                       className="sap-icon--log"
-                      onClick={[Function]}
                     />
                        
                     Application A
@@ -1510,7 +1491,6 @@ exports[`<Shellbar /> create shellbar 5`] = `
                     >
                       <span
                         className="sap-icon--search"
-                        onClick={[Function]}
                       />
                        
                       Search
@@ -1523,7 +1503,6 @@ exports[`<Shellbar /> create shellbar 5`] = `
                     >
                       <span
                         className="sap-icon--settings"
-                        onClick={[Function]}
                       >
                         <span
                           aria-label="Unread count"
@@ -1543,7 +1522,6 @@ exports[`<Shellbar /> create shellbar 5`] = `
                     >
                       <span
                         className="sap-icon--bell"
-                        onClick={[Function]}
                       />
                        
                       Notifications
@@ -1556,7 +1534,6 @@ exports[`<Shellbar /> create shellbar 5`] = `
                     >
                       <span
                         className="sap-icon--grid"
-                        onClick={[Function]}
                       />
                        
                       Product Switcher
@@ -1613,7 +1590,6 @@ exports[`<Shellbar /> create shellbar 5`] = `
                     >
                       <span
                         className="sap-icon--action-settings sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Settings
@@ -1626,7 +1602,6 @@ exports[`<Shellbar /> create shellbar 5`] = `
                     >
                       <span
                         className="sap-icon--log sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Sign Out
@@ -1765,7 +1740,6 @@ exports[`<Shellbar /> create shellbar 6`] = `
                   >
                     <span
                       className="sap-icon--log"
-                      onClick={[Function]}
                     />
                        
                     Application A
@@ -1992,7 +1966,6 @@ exports[`<Shellbar /> create shellbar 6`] = `
                     >
                       <span
                         className="sap-icon--search"
-                        onClick={[Function]}
                       />
                        
                       Search
@@ -2005,7 +1978,6 @@ exports[`<Shellbar /> create shellbar 6`] = `
                     >
                       <span
                         className="sap-icon--settings"
-                        onClick={[Function]}
                       >
                         <span
                           aria-label="Unread count"
@@ -2025,7 +1997,6 @@ exports[`<Shellbar /> create shellbar 6`] = `
                     >
                       <span
                         className="sap-icon--bell"
-                        onClick={[Function]}
                       >
                         <span
                           aria-label="Unread count"
@@ -2045,7 +2016,6 @@ exports[`<Shellbar /> create shellbar 6`] = `
                     >
                       <span
                         className="sap-icon--grid"
-                        onClick={[Function]}
                       />
                        
                       Product Switcher
@@ -2105,7 +2075,6 @@ exports[`<Shellbar /> create shellbar 6`] = `
                     >
                       <span
                         className="sap-icon--action-settings sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Settings
@@ -2118,7 +2087,6 @@ exports[`<Shellbar /> create shellbar 6`] = `
                     >
                       <span
                         className="sap-icon--log sap-icon--s"
-                        onClick={[Function]}
                       />
                          
                       Sign Out
diff --git a/src/Time/Time.js b/src/Time/Time.js
index 62a3cb7d9..167b3bb80 100644
--- a/src/Time/Time.js
+++ b/src/Time/Time.js
@@ -28,6 +28,7 @@ class TimeItem extends Component {
         format12hours: false,
         id: '',
         spinners: false,
+        updateTime: () => {},
         value: null
     };
 
@@ -329,10 +330,8 @@ export class Time extends Component {
         this.setState(prevState => ({
             ...prevState.time
         }));
-        //
-        if (this.props.onChange) {
-            this.props.onChange(this.state.time);
-        }
+
+        this.props.onChange(this.state.time);
     };
     // onUpdateTime = () => {
     //   this.props.onUpdateTime();
@@ -488,6 +487,7 @@ Time.propTypes = {
 };
 
 Time.defaultProps = {
+    onChange: () => {},
     showHour: true,
     showMinute: true,
     showSecond: true,

From e09cb7170eb56e673bdd023dc35d7f9f918fd6f4 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Thu, 31 Jan 2019 13:09:19 -0800
Subject: [PATCH 27/29] Default prop consts

---
 src/Badge/Badge.js     | 4 ++--
 src/Forms/Forms.js     | 3 ++-
 src/utils/constants.js | 6 ++++++
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/src/Badge/Badge.js b/src/Badge/Badge.js
index a6438baae..896c061cf 100644
--- a/src/Badge/Badge.js
+++ b/src/Badge/Badge.js
@@ -1,7 +1,7 @@
 import classnames from 'classnames';
 import PropTypes from 'prop-types';
 import React from 'react';
-import {BADGE_TYPES, LABEL_TYPES, STATUS_TYPES} from '../utils/constants';
+import {BADGE_MODIFIERS, BADGE_TYPES, LABEL_TYPES, STATUS_TYPES} from '../utils/constants';
 
 
 export const Badge = ({ type, modifier, children, className, ...props }) => {
@@ -23,7 +23,7 @@ export const Badge = ({ type, modifier, children, className, ...props }) => {
 
 Badge.propTypes = {
     className: PropTypes.string,
-    modifier: PropTypes.oneOf(['', 'pill', 'filled']),
+    modifier: PropTypes.oneOf(BADGE_MODIFIERS),
     type: PropTypes.oneOf(BADGE_TYPES)
 };
 
diff --git a/src/Forms/Forms.js b/src/Forms/Forms.js
index ee8fd84f1..8b7fa7304 100644
--- a/src/Forms/Forms.js
+++ b/src/Forms/Forms.js
@@ -1,4 +1,5 @@
 import classnames from 'classnames';
+import { FORM_MESSAGE_TYPES } from '../utils/constants';
 import PropTypes from 'prop-types';
 import React, { Component } from 'react';
 
@@ -98,7 +99,7 @@ export const FormMessage = ({ type, children, className, ...props }) => {
 FormMessage.propTypes = {
     children: PropTypes.node,
     className: PropTypes.string,
-    type: PropTypes.oneOf(['error', 'warning', 'help'])
+    type: PropTypes.oneOf(FORM_MESSAGE_TYPES)
 };
 
 // ------------------------------------------------- Form Input ----------------------------------------------
diff --git a/src/utils/constants.js b/src/utils/constants.js
index 304244c9b..f2aea8fa1 100644
--- a/src/utils/constants.js
+++ b/src/utils/constants.js
@@ -10,6 +10,12 @@ export const BADGE_TYPES = [
     'warning',
     'error'
 ];
+
+export const BADGE_MODIFIERS = [
+    'pill',
+    'filled'
+];
+
 export const LABEL_TYPES = [
     'success',
     'warning',

From 93300e3a1042285fb960ac8ca0e4f348b6781d9a Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Thu, 31 Jan 2019 13:24:52 -0800
Subject: [PATCH 28/29] Remove static proptypes

---
 src/Time/Time.js                              | 31 +------
 src/Time/__snapshots__/Time.test.js.snap      | 57 ------------
 src/TimePicker/TimePicker.js                  | 20 -----
 .../__snapshots__/TimePicker.test.js.snap     | 87 -------------------
 4 files changed, 4 insertions(+), 191 deletions(-)

diff --git a/src/Time/Time.js b/src/Time/Time.js
index 167b3bb80..fae727efd 100644
--- a/src/Time/Time.js
+++ b/src/Time/Time.js
@@ -5,33 +5,6 @@ const INVALID = 'is-invalid';
 const CLOCK = ['am', 'pm'];
 
 class TimeItem extends Component {
-    static propTypes = {
-        arialabel: PropTypes.string,
-        disabled: PropTypes.bool,
-        downButtonProps: PropTypes.object,
-        format12Hours: PropTypes.bool,
-        inputProps: PropTypes.object,
-        max: PropTypes.oneOfType([PropTypes.number, PropTypes.string]),
-        name: PropTypes.string,
-        placeholder: PropTypes.string,
-        spinners: PropTypes.bool,
-        style: PropTypes.string,
-        time: PropTypes.object,
-        type: PropTypes.string,
-        upButtonProps: PropTypes.object,
-        updateTime: PropTypes.func,
-        value: PropTypes.oneOfType([PropTypes.number, PropTypes.string])
-    };
-    static defaultProps = {
-        arialabel: '',
-        disabled: false,
-        format12hours: false,
-        id: '',
-        spinners: false,
-        updateTime: () => {},
-        value: null
-    };
-
     constructor(props) {
         super(props);
         var aria = {};
@@ -287,6 +260,10 @@ TimeItem.propTypes = {
     value: PropTypes.oneOfType([PropTypes.number, PropTypes.string])
 };
 
+TimeItem.defaultProps = {
+    updateTime: () => {}
+};
+
 export class Time extends Component {
     constructor(props) {
         super(props);
diff --git a/src/Time/__snapshots__/Time.test.js.snap b/src/Time/__snapshots__/Time.test.js.snap
index fe975335c..3ba886cda 100644
--- a/src/Time/__snapshots__/Time.test.js.snap
+++ b/src/Time/__snapshots__/Time.test.js.snap
@@ -13,7 +13,6 @@ exports[`<Time /> create time component 1`] = `
       <button
         aria-label="Increase hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -27,7 +26,6 @@ exports[`<Time /> create time component 1`] = `
         name="hour"
         onChange={[Function]}
         placeholder="hh"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -38,7 +36,6 @@ exports[`<Time /> create time component 1`] = `
       <button
         aria-label="Decrease hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -52,7 +49,6 @@ exports[`<Time /> create time component 1`] = `
       <button
         aria-label="Increase minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -66,7 +62,6 @@ exports[`<Time /> create time component 1`] = `
         name="minute"
         onChange={[Function]}
         placeholder="mm"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -77,7 +72,6 @@ exports[`<Time /> create time component 1`] = `
       <button
         aria-label="Decrease minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -91,7 +85,6 @@ exports[`<Time /> create time component 1`] = `
       <button
         aria-label="Increase seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -105,7 +98,6 @@ exports[`<Time /> create time component 1`] = `
         name="second"
         onChange={[Function]}
         placeholder="ss"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -116,7 +108,6 @@ exports[`<Time /> create time component 1`] = `
       <button
         aria-label="Decrease seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -138,7 +129,6 @@ exports[`<Time /> create time component 2`] = `
       <button
         aria-label="Increase hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -152,7 +142,6 @@ exports[`<Time /> create time component 2`] = `
         name="hour"
         onChange={[Function]}
         placeholder="hh"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -163,7 +152,6 @@ exports[`<Time /> create time component 2`] = `
       <button
         aria-label="Decrease hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -177,7 +165,6 @@ exports[`<Time /> create time component 2`] = `
       <button
         aria-label="Increase minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -191,7 +178,6 @@ exports[`<Time /> create time component 2`] = `
         name="minute"
         onChange={[Function]}
         placeholder="mm"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -202,7 +188,6 @@ exports[`<Time /> create time component 2`] = `
       <button
         aria-label="Decrease minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -216,7 +201,6 @@ exports[`<Time /> create time component 2`] = `
       <button
         aria-label="Increase seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -230,7 +214,6 @@ exports[`<Time /> create time component 2`] = `
         name="second"
         onChange={[Function]}
         placeholder="ss"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -241,7 +224,6 @@ exports[`<Time /> create time component 2`] = `
       <button
         aria-label="Decrease seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -263,7 +245,6 @@ exports[`<Time /> create time component 3`] = `
       <button
         aria-label="Increase hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -277,7 +258,6 @@ exports[`<Time /> create time component 3`] = `
         name="hour"
         onChange={[Function]}
         placeholder="hh"
-        readOnly={false}
         type="text"
         value={22}
       />
@@ -288,7 +268,6 @@ exports[`<Time /> create time component 3`] = `
       <button
         aria-label="Decrease hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -302,7 +281,6 @@ exports[`<Time /> create time component 3`] = `
       <button
         aria-label="Increase minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -316,7 +294,6 @@ exports[`<Time /> create time component 3`] = `
         name="minute"
         onChange={[Function]}
         placeholder="mm"
-        readOnly={false}
         type="text"
         value={34}
       />
@@ -327,7 +304,6 @@ exports[`<Time /> create time component 3`] = `
       <button
         aria-label="Decrease minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -341,7 +317,6 @@ exports[`<Time /> create time component 3`] = `
       <button
         aria-label="Increase seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -355,7 +330,6 @@ exports[`<Time /> create time component 3`] = `
         name="second"
         onChange={[Function]}
         placeholder="ss"
-        readOnly={false}
         type="text"
         value={12}
       />
@@ -366,7 +340,6 @@ exports[`<Time /> create time component 3`] = `
       <button
         aria-label="Decrease seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -388,7 +361,6 @@ exports[`<Time /> create time component 4`] = `
       <button
         aria-label="Increase hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -402,7 +374,6 @@ exports[`<Time /> create time component 4`] = `
         name="hour"
         onChange={[Function]}
         placeholder="hh"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -413,7 +384,6 @@ exports[`<Time /> create time component 4`] = `
       <button
         aria-label="Decrease hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -427,7 +397,6 @@ exports[`<Time /> create time component 4`] = `
       <button
         aria-label="Increase minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -441,7 +410,6 @@ exports[`<Time /> create time component 4`] = `
         name="minute"
         onChange={[Function]}
         placeholder="mm"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -452,7 +420,6 @@ exports[`<Time /> create time component 4`] = `
       <button
         aria-label="Decrease minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -466,7 +433,6 @@ exports[`<Time /> create time component 4`] = `
       <button
         aria-label="Increase seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -480,7 +446,6 @@ exports[`<Time /> create time component 4`] = `
         name="second"
         onChange={[Function]}
         placeholder="ss"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -491,7 +456,6 @@ exports[`<Time /> create time component 4`] = `
       <button
         aria-label="Decrease seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -513,7 +477,6 @@ exports[`<Time /> create time component 5`] = `
       <button
         aria-label="Increase hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -527,7 +490,6 @@ exports[`<Time /> create time component 5`] = `
         name="hour"
         onChange={[Function]}
         placeholder="hh"
-        readOnly={false}
         type="text"
         value="12"
       />
@@ -538,7 +500,6 @@ exports[`<Time /> create time component 5`] = `
       <button
         aria-label="Decrease hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -552,7 +513,6 @@ exports[`<Time /> create time component 5`] = `
       <button
         aria-label="Increase minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -566,7 +526,6 @@ exports[`<Time /> create time component 5`] = `
         name="minute"
         onChange={[Function]}
         placeholder="mm"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -577,7 +536,6 @@ exports[`<Time /> create time component 5`] = `
       <button
         aria-label="Decrease minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -591,7 +549,6 @@ exports[`<Time /> create time component 5`] = `
       <button
         aria-label="Increase seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -605,7 +562,6 @@ exports[`<Time /> create time component 5`] = `
         name="second"
         onChange={[Function]}
         placeholder="ss"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -616,7 +572,6 @@ exports[`<Time /> create time component 5`] = `
       <button
         aria-label="Decrease seconds"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -630,7 +585,6 @@ exports[`<Time /> create time component 5`] = `
       <button
         aria-label="Increase period"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -643,7 +597,6 @@ exports[`<Time /> create time component 5`] = `
         maxLength="2"
         name="meridiem"
         onChange={[Function]}
-        readOnly={false}
         type="text"
         value="am"
       />
@@ -654,7 +607,6 @@ exports[`<Time /> create time component 5`] = `
       <button
         aria-label="Decrease period"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -680,7 +632,6 @@ exports[`<Time /> create time component 6`] = `
         name="hour"
         onChange={[Function]}
         placeholder="hh"
-        readOnly={false}
         type="text"
         value="12"
       />
@@ -701,7 +652,6 @@ exports[`<Time /> create time component 6`] = `
         name="minute"
         onChange={[Function]}
         placeholder="mm"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -722,7 +672,6 @@ exports[`<Time /> create time component 6`] = `
         name="second"
         onChange={[Function]}
         placeholder="ss"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -746,7 +695,6 @@ exports[`<Time /> create time component 7`] = `
       <button
         aria-label="Increase hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -760,7 +708,6 @@ exports[`<Time /> create time component 7`] = `
         name="hour"
         onChange={[Function]}
         placeholder="hh"
-        readOnly={false}
         type="text"
         value="12"
       />
@@ -771,7 +718,6 @@ exports[`<Time /> create time component 7`] = `
       <button
         aria-label="Decrease hours"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -785,7 +731,6 @@ exports[`<Time /> create time component 7`] = `
       <button
         aria-label="Increase minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-        disabled={false}
         onClick={[Function]}
       />
     </div>
@@ -799,7 +744,6 @@ exports[`<Time /> create time component 7`] = `
         name="minute"
         onChange={[Function]}
         placeholder="mm"
-        readOnly={false}
         type="text"
         value="00"
       />
@@ -810,7 +754,6 @@ exports[`<Time /> create time component 7`] = `
       <button
         aria-label="Decrease minutes"
         className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-        disabled={false}
         onClick={[Function]}
       />
     </div>
diff --git a/src/TimePicker/TimePicker.js b/src/TimePicker/TimePicker.js
index 4fbeeacf2..133cc1d7e 100644
--- a/src/TimePicker/TimePicker.js
+++ b/src/TimePicker/TimePicker.js
@@ -7,26 +7,6 @@ const CLOCK = ['am', 'pm'];
 const INVALID = 'is-invalid';
 const VALID = 'fd-input';
 class TimePickerItem extends Component {
-    static propTypes = {
-        buttonID: PropTypes.string,
-        buttonProps: PropTypes.object,
-        disabled: PropTypes.bool,
-        format12Hours: PropTypes.bool,
-        id: PropTypes.string,
-        inputId: PropTypes.string,
-        inputProps: PropTypes.object,
-        isValid: PropTypes.bool,
-        length: PropTypes.number,
-        placeholder: PropTypes.string,
-        showHour: PropTypes.bool,
-        showMinute: PropTypes.bool,
-        showSecond: PropTypes.bool,
-        style: PropTypes.string,
-        updateValue: PropTypes.func,
-        value: PropTypes.oneOfType([PropTypes.number, PropTypes.string]),
-        onChange: PropTypes.func
-    };
-
     constructor(props) {
         super(props);
         var length = this.setLength(props);
diff --git a/src/TimePicker/__snapshots__/TimePicker.test.js.snap b/src/TimePicker/__snapshots__/TimePicker.test.js.snap
index 88308af7d..19a550765 100644
--- a/src/TimePicker/__snapshots__/TimePicker.test.js.snap
+++ b/src/TimePicker/__snapshots__/TimePicker.test.js.snap
@@ -64,7 +64,6 @@ exports[`<TimePicker /> create time picker 1`] = `
               <button
                 aria-label="Increase hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -78,7 +77,6 @@ exports[`<TimePicker /> create time picker 1`] = `
                 name="hour"
                 onChange={[Function]}
                 placeholder="hh"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -89,7 +87,6 @@ exports[`<TimePicker /> create time picker 1`] = `
               <button
                 aria-label="Decrease hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -103,7 +100,6 @@ exports[`<TimePicker /> create time picker 1`] = `
               <button
                 aria-label="Increase minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -117,7 +113,6 @@ exports[`<TimePicker /> create time picker 1`] = `
                 name="minute"
                 onChange={[Function]}
                 placeholder="mm"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -128,7 +123,6 @@ exports[`<TimePicker /> create time picker 1`] = `
               <button
                 aria-label="Decrease minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -142,7 +136,6 @@ exports[`<TimePicker /> create time picker 1`] = `
               <button
                 aria-label="Increase seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -156,7 +149,6 @@ exports[`<TimePicker /> create time picker 1`] = `
                 name="second"
                 onChange={[Function]}
                 placeholder="ss"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -167,7 +159,6 @@ exports[`<TimePicker /> create time picker 1`] = `
               <button
                 aria-label="Decrease seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -241,7 +232,6 @@ exports[`<TimePicker /> create time picker 2`] = `
               <button
                 aria-label="Increase hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -255,7 +245,6 @@ exports[`<TimePicker /> create time picker 2`] = `
                 name="hour"
                 onChange={[Function]}
                 placeholder="hh"
-                readOnly={false}
                 type="text"
                 value="12"
               />
@@ -266,7 +255,6 @@ exports[`<TimePicker /> create time picker 2`] = `
               <button
                 aria-label="Decrease hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -280,7 +268,6 @@ exports[`<TimePicker /> create time picker 2`] = `
               <button
                 aria-label="Increase minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -294,7 +281,6 @@ exports[`<TimePicker /> create time picker 2`] = `
                 name="minute"
                 onChange={[Function]}
                 placeholder="mm"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -305,7 +291,6 @@ exports[`<TimePicker /> create time picker 2`] = `
               <button
                 aria-label="Decrease minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -319,7 +304,6 @@ exports[`<TimePicker /> create time picker 2`] = `
               <button
                 aria-label="Increase seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -333,7 +317,6 @@ exports[`<TimePicker /> create time picker 2`] = `
                 name="second"
                 onChange={[Function]}
                 placeholder="ss"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -344,7 +327,6 @@ exports[`<TimePicker /> create time picker 2`] = `
               <button
                 aria-label="Decrease seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -358,7 +340,6 @@ exports[`<TimePicker /> create time picker 2`] = `
               <button
                 aria-label="Increase period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -371,7 +352,6 @@ exports[`<TimePicker /> create time picker 2`] = `
                 maxLength="2"
                 name="meridiem"
                 onChange={[Function]}
-                readOnly={false}
                 type="text"
                 value="am"
               />
@@ -382,7 +362,6 @@ exports[`<TimePicker /> create time picker 2`] = `
               <button
                 aria-label="Decrease period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -455,7 +434,6 @@ exports[`<TimePicker /> create time picker 3`] = `
               <button
                 aria-label="Increase hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -469,7 +447,6 @@ exports[`<TimePicker /> create time picker 3`] = `
                 name="hour"
                 onChange={[Function]}
                 placeholder="hh"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -480,7 +457,6 @@ exports[`<TimePicker /> create time picker 3`] = `
               <button
                 aria-label="Decrease hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -494,7 +470,6 @@ exports[`<TimePicker /> create time picker 3`] = `
               <button
                 aria-label="Increase minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -508,7 +483,6 @@ exports[`<TimePicker /> create time picker 3`] = `
                 name="minute"
                 onChange={[Function]}
                 placeholder="mm"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -519,7 +493,6 @@ exports[`<TimePicker /> create time picker 3`] = `
               <button
                 aria-label="Decrease minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -773,7 +746,6 @@ exports[`<TimePicker /> create time picker 5`] = `
               <button
                 aria-label="Increase hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -787,7 +759,6 @@ exports[`<TimePicker /> create time picker 5`] = `
                 name="hour"
                 onChange={[Function]}
                 placeholder="hh"
-                readOnly={false}
                 type="text"
                 value="12"
               />
@@ -798,7 +769,6 @@ exports[`<TimePicker /> create time picker 5`] = `
               <button
                 aria-label="Decrease hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -812,7 +782,6 @@ exports[`<TimePicker /> create time picker 5`] = `
               <button
                 aria-label="Increase minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -826,7 +795,6 @@ exports[`<TimePicker /> create time picker 5`] = `
                 name="minute"
                 onChange={[Function]}
                 placeholder="mm"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -837,7 +805,6 @@ exports[`<TimePicker /> create time picker 5`] = `
               <button
                 aria-label="Decrease minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -851,7 +818,6 @@ exports[`<TimePicker /> create time picker 5`] = `
               <button
                 aria-label="Increase seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -865,7 +831,6 @@ exports[`<TimePicker /> create time picker 5`] = `
                 name="second"
                 onChange={[Function]}
                 placeholder="ss"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -876,7 +841,6 @@ exports[`<TimePicker /> create time picker 5`] = `
               <button
                 aria-label="Decrease seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -890,7 +854,6 @@ exports[`<TimePicker /> create time picker 5`] = `
               <button
                 aria-label="Increase period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -903,7 +866,6 @@ exports[`<TimePicker /> create time picker 5`] = `
                 maxLength="2"
                 name="meridiem"
                 onChange={[Function]}
-                readOnly={false}
                 type="text"
                 value="am"
               />
@@ -914,7 +876,6 @@ exports[`<TimePicker /> create time picker 5`] = `
               <button
                 aria-label="Decrease period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -987,7 +948,6 @@ exports[`<TimePicker /> create time picker 6`] = `
               <button
                 aria-label="Increase hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1001,7 +961,6 @@ exports[`<TimePicker /> create time picker 6`] = `
                 name="hour"
                 onChange={[Function]}
                 placeholder="hh"
-                readOnly={false}
                 type="text"
                 value="12"
               />
@@ -1012,7 +971,6 @@ exports[`<TimePicker /> create time picker 6`] = `
               <button
                 aria-label="Decrease hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1026,7 +984,6 @@ exports[`<TimePicker /> create time picker 6`] = `
               <button
                 aria-label="Increase minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1040,7 +997,6 @@ exports[`<TimePicker /> create time picker 6`] = `
                 name="minute"
                 onChange={[Function]}
                 placeholder="mm"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1051,7 +1007,6 @@ exports[`<TimePicker /> create time picker 6`] = `
               <button
                 aria-label="Decrease minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1065,7 +1020,6 @@ exports[`<TimePicker /> create time picker 6`] = `
               <button
                 aria-label="Increase seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1079,7 +1033,6 @@ exports[`<TimePicker /> create time picker 6`] = `
                 name="second"
                 onChange={[Function]}
                 placeholder="ss"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1090,7 +1043,6 @@ exports[`<TimePicker /> create time picker 6`] = `
               <button
                 aria-label="Decrease seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1104,7 +1056,6 @@ exports[`<TimePicker /> create time picker 6`] = `
               <button
                 aria-label="Increase period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1117,7 +1068,6 @@ exports[`<TimePicker /> create time picker 6`] = `
                 maxLength="2"
                 name="meridiem"
                 onChange={[Function]}
-                readOnly={false}
                 type="text"
                 value="am"
               />
@@ -1128,7 +1078,6 @@ exports[`<TimePicker /> create time picker 6`] = `
               <button
                 aria-label="Decrease period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1201,7 +1150,6 @@ exports[`<TimePicker /> create time picker 7`] = `
               <button
                 aria-label="Increase hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1215,7 +1163,6 @@ exports[`<TimePicker /> create time picker 7`] = `
                 name="hour"
                 onChange={[Function]}
                 placeholder="hh"
-                readOnly={false}
                 type="text"
                 value="12"
               />
@@ -1226,7 +1173,6 @@ exports[`<TimePicker /> create time picker 7`] = `
               <button
                 aria-label="Decrease hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1240,7 +1186,6 @@ exports[`<TimePicker /> create time picker 7`] = `
               <button
                 aria-label="Increase minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1254,7 +1199,6 @@ exports[`<TimePicker /> create time picker 7`] = `
                 name="minute"
                 onChange={[Function]}
                 placeholder="mm"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1265,7 +1209,6 @@ exports[`<TimePicker /> create time picker 7`] = `
               <button
                 aria-label="Decrease minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1279,7 +1222,6 @@ exports[`<TimePicker /> create time picker 7`] = `
               <button
                 aria-label="Increase seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1293,7 +1235,6 @@ exports[`<TimePicker /> create time picker 7`] = `
                 name="second"
                 onChange={[Function]}
                 placeholder="ss"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1304,7 +1245,6 @@ exports[`<TimePicker /> create time picker 7`] = `
               <button
                 aria-label="Decrease seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1318,7 +1258,6 @@ exports[`<TimePicker /> create time picker 7`] = `
               <button
                 aria-label="Increase period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1331,7 +1270,6 @@ exports[`<TimePicker /> create time picker 7`] = `
                 maxLength="2"
                 name="meridiem"
                 onChange={[Function]}
-                readOnly={false}
                 type="text"
                 value="am"
               />
@@ -1342,7 +1280,6 @@ exports[`<TimePicker /> create time picker 7`] = `
               <button
                 aria-label="Decrease period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1416,7 +1353,6 @@ exports[`<TimePicker /> create time picker 8`] = `
               <button
                 aria-label="Increase minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1430,7 +1366,6 @@ exports[`<TimePicker /> create time picker 8`] = `
                 name="minute"
                 onChange={[Function]}
                 placeholder="mm"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1441,7 +1376,6 @@ exports[`<TimePicker /> create time picker 8`] = `
               <button
                 aria-label="Decrease minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1455,7 +1389,6 @@ exports[`<TimePicker /> create time picker 8`] = `
               <button
                 aria-label="Increase seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1469,7 +1402,6 @@ exports[`<TimePicker /> create time picker 8`] = `
                 name="second"
                 onChange={[Function]}
                 placeholder="ss"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1480,7 +1412,6 @@ exports[`<TimePicker /> create time picker 8`] = `
               <button
                 aria-label="Decrease seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1555,7 +1486,6 @@ exports[`<TimePicker /> create time picker 9`] = `
               <button
                 aria-label="Increase minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1569,7 +1499,6 @@ exports[`<TimePicker /> create time picker 9`] = `
                 name="minute"
                 onChange={[Function]}
                 placeholder="mm"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1580,7 +1509,6 @@ exports[`<TimePicker /> create time picker 9`] = `
               <button
                 aria-label="Decrease minutes"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1594,7 +1522,6 @@ exports[`<TimePicker /> create time picker 9`] = `
               <button
                 aria-label="Increase seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1608,7 +1535,6 @@ exports[`<TimePicker /> create time picker 9`] = `
                 name="second"
                 onChange={[Function]}
                 placeholder="ss"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1619,7 +1545,6 @@ exports[`<TimePicker /> create time picker 9`] = `
               <button
                 aria-label="Decrease seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1633,7 +1558,6 @@ exports[`<TimePicker /> create time picker 9`] = `
               <button
                 aria-label="Increase period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1646,7 +1570,6 @@ exports[`<TimePicker /> create time picker 9`] = `
                 maxLength="2"
                 name="meridiem"
                 onChange={[Function]}
-                readOnly={false}
                 type="text"
                 value="am"
               />
@@ -1657,7 +1580,6 @@ exports[`<TimePicker /> create time picker 9`] = `
               <button
                 aria-label="Decrease period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1730,7 +1652,6 @@ exports[`<TimePicker /> create time picker 10`] = `
               <button
                 aria-label="Increase hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1744,7 +1665,6 @@ exports[`<TimePicker /> create time picker 10`] = `
                 name="hour"
                 onChange={[Function]}
                 placeholder="hh"
-                readOnly={false}
                 type="text"
                 value="12"
               />
@@ -1755,7 +1675,6 @@ exports[`<TimePicker /> create time picker 10`] = `
               <button
                 aria-label="Decrease hours"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1770,7 +1689,6 @@ exports[`<TimePicker /> create time picker 10`] = `
               <button
                 aria-label="Increase seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1784,7 +1702,6 @@ exports[`<TimePicker /> create time picker 10`] = `
                 name="second"
                 onChange={[Function]}
                 placeholder="ss"
-                readOnly={false}
                 type="text"
                 value="00"
               />
@@ -1795,7 +1712,6 @@ exports[`<TimePicker /> create time picker 10`] = `
               <button
                 aria-label="Decrease seconds"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1809,7 +1725,6 @@ exports[`<TimePicker /> create time picker 10`] = `
               <button
                 aria-label="Increase period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-up-arrow "
-                disabled={false}
                 onClick={[Function]}
               />
             </div>
@@ -1822,7 +1737,6 @@ exports[`<TimePicker /> create time picker 10`] = `
                 maxLength="2"
                 name="meridiem"
                 onChange={[Function]}
-                readOnly={false}
                 type="text"
                 value="am"
               />
@@ -1833,7 +1747,6 @@ exports[`<TimePicker /> create time picker 10`] = `
               <button
                 aria-label="Decrease period"
                 className=" fd-button--light fd-button--xs sap-icon--navigation-down-arrow"
-                disabled={false}
                 onClick={[Function]}
               />
             </div>

From 8666112c25328a4440e23e511d84fcbeb909b5d4 Mon Sep 17 00:00:00 2001
From: CJ Skillingstad <cj.skillingstad@sap.com>
Date: Thu, 31 Jan 2019 13:28:24 -0800
Subject: [PATCH 29/29] Remove last false default prop

---
 src/ActionBar/ActionBar.js | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/ActionBar/ActionBar.js b/src/ActionBar/ActionBar.js
index f3e8311ba..d3148f86b 100644
--- a/src/ActionBar/ActionBar.js
+++ b/src/ActionBar/ActionBar.js
@@ -28,7 +28,6 @@ ActionBar.propTypes = {
 };
 
 ActionBar.defaultProps = {
-    mobile: false,
     width: '319px'
 };