AboutModal accepts a logoImageSrc prop and passes this to AboutModalBoxContainer, but then doesn't do anything with it. I believe it was intended to set the background image on the right side of the modal box. Currently there's no way to override the bg image unless you're using CSS, which isn't always possible on the consuming side.
Since we're not currently using the logoImage{Src,Alt} property for anything, I think we could just rename this to something more appropriate like bgImgSrc and use this value to apply the background image when it's provided. Regardless of how it's implemented, we just need a way to override the background image using pure JS, however that's accomplished.
https://github.com/patternfly/patternfly-react/blob/master/packages/patternfly-4/react-core/src/components/AboutModal/AboutModalContainer.js#L57
@dlabaj
Reactions are currently unavailable
AboutModal accepts a logoImageSrc prop and passes this to AboutModalBoxContainer, but then doesn't do anything with it. I believe it was intended to set the background image on the right side of the modal box. Currently there's no way to override the bg image unless you're using CSS, which isn't always possible on the consuming side.
Since we're not currently using the logoImage{Src,Alt} property for anything, I think we could just rename this to something more appropriate like bgImgSrc and use this value to apply the background image when it's provided. Regardless of how it's implemented, we just need a way to override the background image using pure JS, however that's accomplished.
https://github.com/patternfly/patternfly-react/blob/master/packages/patternfly-4/react-core/src/components/AboutModal/AboutModalContainer.js#L57
@dlabaj