Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Request: Toastr Notifications #3

Open
mariohik opened this issue Jul 14, 2018 · 9 comments
Open

Request: Toastr Notifications #3

mariohik opened this issue Jul 14, 2018 · 9 comments
Assignees

Comments

@mariohik
Copy link

mariohik commented Jul 14, 2018

Can we have a Toastr Extension?
I'm trying to implement myself, but I still have a lot to learn.

@mariohik
Copy link
Author

I'm trying to learn about extensions, so I decided to create a project:
https://github.com/arapinha/Blazor.Toastr

@ransagy
Copy link

ransagy commented Jul 17, 2018

From the gitter discussion, There's also https://github.com/sotsera/sotsera.blazor.toaster

@galvesribeiro
Copy link
Member

I'm onboarding HTML5 Notifications API project from @vertonghenb here https://github.com/BlazorExtensions/Notifications

The original repo: https://github.com/vertonghenb/Blazor.Notifications

He is porting his project to our structure and soon will be a nuget package out.

@ransagy
Copy link

ransagy commented Jul 17, 2018

Haven't had a chance to play with HTML5 Notifications. How customizable are they compared to JS solutions? It seems like they are very static, design wise.

@galvesribeiro
Copy link
Member

galvesribeiro commented Jul 17, 2018

I'm adding ToastR as well. I have it done for a while, just didnt had time to push it out. Gimme a bit of time. Will let u guys know.

@galvesribeiro
Copy link
Member

I just realized that it requires JQuery, so I'm dropping it for now. Will implement one pure in C# when I got time.

@vertonghenb
Copy link

vertonghenb commented Jul 17, 2018

@ransagy
They're not that customizable, I tried to implement the spec in the Blazor.Notifications repo.
However since I'm a strong believer in testability I designed it to have a DI first approach and pass on the static.

@galvesribeiro
Copy link
Member

Yeah, avoid statics at all cost 👍

@galvesribeiro
Copy link
Member

Did we have any updates on this?

If that is not going to be maintained, I'll drop the repo for now.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants