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

Add Custom css class to growlUI #47

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

Add Custom css class to growlUI #47

wants to merge 2 commits into from

Conversation

MarcovanMaanen
Copy link

I have made a little change to it:

You can pick a custom css class just by setting the Bool (DifferentCssClasses) to true.
and than give the css class name as second condition when you call the growlUI...

Now you can pick a custom class for succes, error, warning and notice messages (may come in handy at some point)
with each there own image, color, css.

Only 1 this has changed: even if the option is false you must pass a empty string for a class
or just set it to the default class...

Example:
// With empty css class string...
$.growlUI('Error', '', 'the message');
// With the default css class...
$.growlUI('Error', 'growlUI', 'the message');
// With a custom css class...
$.growlUI('Error', 'pick_a_css_class', 'the message');

Now we can use different classes for the GrowlUI.
Now we can choose if we want to use the Default (ok-check.png) whit message,
or use something like: not-check.png (red error sign), warning, notice, unknown, message.

The only change is the you now can set a 'custom' class.
Use: $.growlUI('Error', 'error', 'A Message');
| First 'Error' = title,
| Second 'error' = class

Maby someone else alsow finds it usefull.
You can pick a custom css class just by setting the Bool (DifferentCssClasses) to true.
and than give the css class name as second condition when you call the growlUI...

Now you can pick a custom class for succes, error, warning and notice messages (may come in handy at some point)
with each there own image, color, css.

Only 1 this has changed: even if the option is false you must pass a empty string for a class 
or just set it to the default class...


Example:
// With empty css class string...
$.growlUI('Error', '', 'the message');
// With the default css class...
$.growlUI('Error', 'growlUI', 'the message');
// With a custom css class...
$.growlUI('Error', 'pick_a_css_class', 'the message');
@DiegoYungh
Copy link

I created a fork with all PR I saw here
https://github.com/DiegoYungh/blockui/commits/master

In the near future I will probably create a block feature on top of blockui and open to pull requests

I set the commit message to your PR but was this one: Issue #36 : #37 opened on 15 Aug 2012 by wiggin15

I could not merge your PR because you changed the entire file.

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.

None yet

2 participants