Custom image per alert type

[lucianomlima]

Turns possible to user set an image for an alert type. Example: custom images for error and success alerts.
Can be new props or an options param for alertWithType method.

[testshallpass]

@lucianomlima If you want you can use imageSrc prop and pass custom type to change the image in the alert.

[lucianomlima]

I know... I use custom type but it would be interesting to be able to have custom image for alert types.

[testshallpass]

true something like the errorImageSrc, etc, it would make it a bit easier. I welcome a PR.

[lucianomlima]

I can help with PR but I'm not familiar with TDD and this can reduce coverage for this component.

[testshallpass]

No worries on the testing I will follow up with a tiny patch on that end. Also, thanks.

[lucianomlima]

Ok! I try to send a PR

[lucianomlima]

  getSourceForType(type) {
    switch (type) {
      case 'info':
        return this.props.infoImageSrc || require('./assets/info.png');
      case 'warn':
        return this.props.warnImageSrc || require('./assets/warn.png');
      case 'error':
        return this.props.errorImageSrc || require('./assets/error.png');
      case 'success':
        return this.props.successImageSrc || require('./assets/success.png');
      default:
        return this.props.imageSrc;
    }
  }

This is good for you?

[testshallpass]

It's good but can be taken a small step further with the props as the return and the defaultProps is the asset. And so: return this.props.infoImageSrc

[lucianomlima]

Ok. I did change. What I have to put for default in Readme for these new props?

[testshallpass]

Basically, infoImageSrc would be require('./assets/info.png'); and etc. See cancelBtnImageSrc prop.

[chhornponleu]

This feature should support version 2.x as well