search

digipolisantwerp / antwerp-ui_angular

Antwerp UI is a component interface library for building user interfaces and responsive web apps.
https://digipolisantwerp.github.io/antwerp-ui_angular
MIT License
13 stars 19 forks source link

Add toastr #236

Closed MichaelCastiau closed 3 years ago

MichaelCastiau commented 3 years ago

Adding Toastr package

MichaelCastiau commented 3 years ago

@TriangleJuice Op dit ogenblik is er geen notification component meer in de acpaas ui; Verder kan ik natuurlijk het onder een andere naam zetten, maar nu kan je gemakkelijker overschakelen naar de nieuwe versie van acpaas als je nog de oude notifications gebruikt. Je kunt de toastr package alongside installeren en seamless overschakelen op de nieuwe versie. Wat denk jij?

TriangleJuice commented 3 years ago

@TriangleJuice Op dit ogenblik is er geen notification component meer in de acpaas ui; Verder kan ik natuurlijk het onder een andere naam zetten, maar nu kan je gemakkelijker overschakelen naar de nieuwe versie van acpaas als je nog de oude notifications gebruikt. Je kunt de toastr package alongside installeren en seamless overschakelen op de nieuwe versie. Wat denk jij?

De naamgeving zou ik sowieso wel terug aanpassen als je dat ziet zitten. Daar hebben we indertijd ferm over gediscussieerd en het zou niet voor de 'gebruiker' (de developer in dit geval) mogen uitmaken welke achterliggende package er gebruikt wordt voor een component. Upgrade problemen gaan we niet veroorzaken, want die is echt al lang eruit gehaald.