cgross / angular-notify

Minimalistic and extensible notification service for Angular.
http://cgross.github.io/angular-notify/demo/
MIT License
432 stars 99 forks source link

Display notification(s) at the bottom instead of top #27

Open bl4de opened 9 years ago

bl4de commented 9 years ago

Hi there,

Thanks for this very useful Angular module.

Because I needed to modify notifications to be able to display them at the bottom of the page, I've forked and modified its code - maybe someone also will look for the same issue, so I've prepared working version available here:

https://github.com/bl4de/angular-notify/tree/master/at-the-bottom

Maybe I should create 'mixed' version with both options and send you some PR for reviewing? What do you think? Let me know in comment ;)

Regards,

R.

cgross commented 9 years ago

Sure. If you can work support for showing notifications on the bottom that would be nice. Thanks.

germanger commented 9 years ago

Whats the status of this? :)

sam2x commented 9 years ago

+1 very interested about this. could you merge this feature please ?

bl4de commented 9 years ago

Sorry, hadn't enough time recently, but I'm going to provide this as soon as possible.

fforres commented 9 years ago

BTW, in case anyone wants to hack this on their own, in this pull request there is a really fast hack

https://github.com/cgross/angular-notify/pull/33/files

marco-gagliardi commented 9 years ago

+1

herrherrmann commented 9 years ago

+1 Thanks for the work! Would be awesome to see this in the next version.

amgadfahmi commented 9 years ago

@forres why this is not merged into the main branch !

fforres commented 9 years ago

@amgadfahmi I guess it's because it's a global hack. I didn't add position and animation calculations for every and each notification.

Although... we should ask @cgross, the library it's kind of dead nowadays :/