Force close when not supplying all options to SimpleSlide.Builder about material-intro HOT 11 CLOSED

You must supply at least a background color, a title, a description and an image.
SimpleSlide is not meant to be used without arguments.

from material-intro.

Sure, but shouldn't there at least be some kind of check or default text, not just a plain crash?

from material-intro.

It throws an IllegalArgumentException saying that you must provide at least a background, title, description and image.

from material-intro.

Hm, ok. In my opinion that's not ideal, but ok. I don't want to meddle in your work, especially as it's so wonderful :)

from material-intro.

What should be the desired behavior?

from material-intro.

Well, if the id is null, just supply no text. Should I create a PR?

from material-intro.

I would appreciate that

from material-intro.

@Maxr1998 Your pull request contains only one small error. Would you please fix that, so that I can merge it?

from material-intro.

Oops, I'm terribly sorry. Didn't see the notifications the last days. Did you fix the issue yourself?

from material-intro.

No, I can't edit your pill request... Would be happy if you edit it

from material-intro.

I fixed it in my code, but I guess I also need to send a new pull request..

from material-intro.