-
Notifications
You must be signed in to change notification settings - Fork 55
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 new FacesMessage Severity "SUCCESS" #1592
Comments
This would be a good first stepping stone, before allowing custom faces messages. |
would like to do this for 4.1; easy and good enhancement |
+1 for the idea. Put it between INFO and WARN. Bump of ordinals of WARN and higher will however affect backwards compatibility on those who are comparing them with hardcoded integers. Nonetheless, I also strongly feel like we should migrate the whole Severity class to a normal enum. The same for PhaseId. Is there more? This is better to be done in 5.0 rather than 4.1. |
+1 for enums |
move to 5.0 and refactor to enum |
will only handle the new sucess here and moved the enum change to a own ticket |
Adding a "success" severity for
FacesMessages
sounds like a good idea to me.There is an issue of JavaServer Faces Add new FacesMessage Severity "SUCCESS" eclipse-ee4j/mojarra#895 and it is closed.
FacesMessages, <b:messages /> & <b:message /> eclipse-ee4j/mojarra#1000
The text was updated successfully, but these errors were encountered: