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

Customizable message CSS classes #16

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

TheRusskiy
Copy link

Hello, idea of this change is ability to specify what CSS classes I want to use for my messages.
Right now classes are hard-coded to bootstrap.
But it's common to use other frameworks and, since I can't change code of a framework, I want to manually wire my message classes to them.

Also in this commit I provided some tests due to previous lack of such.

No breaking changes, all classes are set bootstrap in default.

@TheRusskiy
Copy link
Author

damn... Your last activity was long ago and I want to use this package with bower, but I have no idea how to point bower to a branch other than the master, so another 'bower config change' commit is stuck to this pull request.
If you accept just ping me anytime at [email protected], I'll remove it and point my bower dependency back to your repo.
Thx for cool module btw 👍

@jtammen
Copy link

jtammen commented Jan 24, 2014

👍 would also like to see the class names to be customizable!

stelcheck pushed a commit to stelcheck/angular-growl that referenced this pull request Jul 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants