alexcrack / angular-ui-notification

Angular.js service providing simple notifications using Bootstrap 3 styles with css transitions for animating
MIT License
536 stars 169 forks source link

Using the options.type doesn't work #80

Open joewashear007 opened 8 years ago

joewashear007 commented 8 years ago

In the readme, it mentions that I can set the type of the notifications in the options using the type property

{ title: "Session Expired!", message: "Please login in again.", type: "warning" };

But this will always show up as a primary notification. Is this the expected behavior? After some debugging, I was unsure if this line should be change to

args.type = t || args.type || '';
AoDev commented 8 years ago

I confirm that I have the same problem. I think the API should be simplified. Right now in the docs there are at least three ways to get the same result.