updated propTypes for height and width to be union type of string and…

… number. Also added jest test check no warnings happen when a string is used

lib/__tests__/react-loading.spec.jsx

@@ -1,5 +1,6 @@
 import React from 'react';
 import { shallow } from 'enzyme';
+import sinon from 'sinon';
 
 import Loading from '../react-loading';
 
@@ -51,4 +52,12 @@ describe('test Loading component', () => {
 
     expect(enzymeWrapper.instance().timeout).not.toEqual(undefined);
   });
+
+  it('allows setting height and width with strings without propTypes warnings', () => {
+    const spyConsoleError = sinon.spy(console, 'error');
+    const enzymeWrapper = shallow(<Loading height={'20%'} width={'20%'} />);
+    expect(spyConsoleError.callCount).toEqual(0);
+    spyConsoleError.restore();
+  });
+
 });

lib/react-loading.jsx

@@ -8,8 +8,14 @@ export default class Loading extends Component {
     color: PropTypes.string,
     delay: PropTypes.number,
     type: PropTypes.string,
-    height: PropTypes.number,
-    width: PropTypes.number,
+    height: PropTypes.oneOfType([
+      PropTypes.string,
+      PropTypes.number,
+    ]),
+    width: PropTypes.oneOfType([
+      PropTypes.string,
+      PropTypes.number,
+    ]),
   };
 
   static defaultProps = {

package.json

@@ -66,6 +66,7 @@
     "react-dom": ">=0.14.0",
     "react-test-renderer": "^16.0.0",
     "rimraf": "^2.6.1",
+    "sinon": "^5.0.7",
     "webpack": "^3.8.1"
   },
   "jest": {