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

Option for allow only one message inside growl container #21

Merged
merged 2 commits into from
Aug 10, 2014

Conversation

MilosMosovsky
Copy link

When you need validation for one input (async validation for input box inside form) it is very handy to have only one (last response from server) message inside container.

@JanStevens
Copy link
Owner

So you want to use this for specific inputs?

Example form with 8 inputs, every input has its own inline growl with only one message enabled?

@MilosMosovsky
Copy link
Author

Yes , i have dynamic form , with dynamic number of inputs (via NG-repeat) , on each new input, new dynamic growl container with dynamic ID is created. Now i need to show messages for corrensponding input (correndponding growl container with reference ID ) but only last message from server, now server is responding error or ok on each typed character , in current setup screen is spammed with lot of message from server , but it would be handy to replace each collection on new request (its giving more logic especially only when referenceID is not defined becasue thats means its not global container ) . Does it makes sens to you?

@JanStevens
Copy link
Owner

Hello,

Had some busy times in personal live, so I see that this option can be quite useful for limiting the amount of messages you would like to show in one container. It would be better if it was not a boolean value but rather a numerical one so people could use it to show the last x messages in the container.

Any chance you would like to update your pull request?

@MilosMosovsky
Copy link
Author

Hello , offcourse i understand, i just modified my PR, did you mean something like this?

Thanks

JanStevens pushed a commit that referenced this pull request Aug 10, 2014
Option for allow only one message inside growl container
@JanStevens JanStevens merged commit 48afd81 into JanStevens:master Aug 10, 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