Thanks for your review once again Peter but - again - there are some parts I 
don't understand ;)

So here it goes, you said:
anonymous wrote : JSF components need explicit rather than implicit attributes 
and to support EL
What do you mean with explicit & implicit attributes? Is it bad that the email 
tags are derived from the obfuscator tags?
Regarding the EL: if you do something like:
<s:email value="#{user.email}" /> it will work so I'm not sure what you mean.

Regarding nested tag support: while I theoretically agree with you there is not 
much point in it because now both tags are derived from HtmlOutputText and 
therefore behave in a similar way. E.g.:
<p>
  |     <h:outputText styleClass="foo" value="bar" />
  | </p>
  | <p>
  |     <h:outputText styleClass="foo">bar</h:outputText>
  | </p>
results in
<p>
  |     <span class="foo">bar</span>
  | </p>
  | <p>
  |     bar<span class="foo"></span>
  | </p>
So everything in the body is rendered before the tags - which can be done by 
simply writing it before the tag too.

Last but not least you said that the default behavior of the email tag should 
be to do no obfuscation. I don't see why because the whole point of this tag is 
to obfuscate email addresses so this would only result in more writing for the 
normal user while other people still could manually write unobfuscated 
addresses down.
Therefore my suggestion would be to add obfuscation="none" so people can use 
the tag without obfuscation - although I don't see a reason why anyone should 
do that.

The rest is changed like you suggested!

I'm looking forward to your response.

View the original post : 
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4042253#4042253

Reply to the post : 
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4042253
_______________________________________________
jboss-user mailing list
jboss-user@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-user

Reply via email to