PlaceholderWithoutTracking: removed default width and height values (#58)

Co-authored-by: Aleksandr Strashkov <[email protected]>

Author: strashkov

src/components/PlaceholderWithoutTracking.jsx

@@ -118,11 +118,24 @@ class PlaceholderWithoutTracking extends React.Component {
 			});
 		}
 
+		const styleProp = {
+			display: 'inline-block',
+			...style,
+		};
+
+		if (typeof width !== 'undefined') {
+			styleProp.width = width;
+		}
+
+		if (typeof height !== 'undefined') {
+			styleProp.height = height;
+		}
+
 		return (
 			<span
 				className={className}
 				ref={el => (this.placeholder = el)}
-				style={{ display: 'inline-block', height, width, ...style }}
+				style={styleProp}
 			>
 				{placeholder}
 			</span>
@@ -133,24 +146,22 @@ class PlaceholderWithoutTracking extends React.Component {
 PlaceholderWithoutTracking.propTypes = {
 	onVisible: PropTypes.func.isRequired,
 	className: PropTypes.string,
-	height: PropTypes.number,
+	height: PropTypes.oneOfType([PropTypes.number, PropTypes.string]),
 	placeholder: PropTypes.element,
 	threshold: PropTypes.number,
 	useIntersectionObserver: PropTypes.bool,
 	scrollPosition: PropTypes.shape({
 		x: PropTypes.number.isRequired,
 		y: PropTypes.number.isRequired,
 	}),
-	width: PropTypes.number,
+	width: PropTypes.oneOfType([PropTypes.number, PropTypes.string]),
 };
 
 PlaceholderWithoutTracking.defaultProps = {
 	className: '',
-	height: 0,
 	placeholder: null,
 	threshold: 100,
 	useIntersectionObserver: true,
-	width: 0,
 };
 
 export default PlaceholderWithoutTracking;