Removed the React.Fragment and introduced a more general div. By moving the declaration in code our html code is a bit nicer as well. Fixes #74

This commit is contained in:
Rick van Lieshout 2020-06-18 18:14:17 +02:00
parent 6dbd794b6e
commit 47f0e98fe4
2 changed files with 13 additions and 10 deletions

View File

@ -986,17 +986,20 @@ module.exports = /******/ (function (modules) {
buttonsToRender.reverse();
}
var OverlayWrapper = !overlay
var Wrapper = !overlay
? function (props) {
return _react2.default.createElement(_react2.default.Fragment, props);
return _react2.default.createElement("div", props);
}
: function (props) {
return _react2.default.createElement("div", props);
return _react2.default.createElement(
"div",
_extends({}, props, { style: myOverlayStyle, className: overlayClasses })
);
};
return _react2.default.createElement(
OverlayWrapper,
{ style: myOverlayStyle, className: overlayClasses },
Wrapper,
null,
_react2.default.createElement(
"div",
{ className: "" + containerClasses, style: myStyle },

View File

@ -320,12 +320,12 @@ class CookieConsent extends Component {
buttonsToRender.reverse();
}
const OverlayWrapper = !overlay
? (props) => <React.Fragment {...props} />
: (props) => <div {...props} />;
const Wrapper = !overlay
? (props) => <div {...props} />
: (props) => <div {...props} style={myOverlayStyle} className={overlayClasses} />;
return (
<OverlayWrapper style={myOverlayStyle} className={overlayClasses}>
<Wrapper>
<div className={`${containerClasses}`} style={myStyle}>
<div style={myContentStyle} className={contentClasses}>
{this.props.children}
@ -336,7 +336,7 @@ class CookieConsent extends Component {
})}
</div>
</div>
</OverlayWrapper>
</Wrapper>
);
}
}