search

alertifyjs / alertify.js

JavaScript Alert/Notification System
https://alertifyjs.org
MIT License
521 stars 104 forks source link

Warnings would be cool, in orange! #67

Open gabn88 opened 8 years ago

gabn88 commented 8 years ago

If I look at Bootstrap or Django there are several more options next to 'log', 'error' and 'success'. For example Django:

info success warning error

It would be cool to include those into alertify if possible :)

bradberger commented 8 years ago

I think we could definitely accommodate this, but instead of forcing orange it seems like it might be more useful just to allow custom classes. I think the goal is to keep things as generic as possible for simplicity and size. But if we added an API to allow adding custom classes, it could make it simple to customize the colors on demand.

I'll keep this open. Hopefully someone can jump in and help out with this.

bradberger commented 8 years ago

Actually it looks like we already might have a PR but I think we just need to finalize it.

Vallenain commented 8 years ago

+1 Having info and warn levels of log would be greatly appreciated (and also more consistent with other logging components)