[ http://issues.apache.org/jira/browse/ADFFACES-57?page=comments#action_12422207 ] Adam Winer commented on ADFFACES-57: ------------------------------------
"description" is a fairly esoteric attribute of outputText; I don't see a deep need to support it here. Also, I'm torn about supporting escape="false" here; I'd be more inclined to support a mode (perhaps always on?) where it use the same formatted text code as outputFormatted, so common HTML constructs are supported but no cross-site scripting attacks are possible. Minor code comments: - Use toString() instead of ComponentUtils.resolveString() - Get facet constants directly off of CoreOutputDocument - _isLineFeed() should probably be expanded out as "if..." instead of iterating through the array, since it's such a critical path in the code > Add a new ouputDocument component to print multi-paragraph text > --------------------------------------------------------------- > > Key: ADFFACES-57 > URL: http://issues.apache.org/jira/browse/ADFFACES-57 > Project: MyFaces ADF-Faces > Issue Type: New Feature > Reporter: Simon Lessard > Priority: Minor > Attachments: OutputDocument.patch > > > This looks like a fairly trivial component to make and it could be useful for > some Web sites. > The component would mainly parse its value attribute and change \n and/or \r > with <p>. > Possible facet could be "separator" that would be inserted between paragraph. > The whole text could be in a div and it could supports skin selectors like: > af|outputDocument > af|outputDocument::paragraph > af|outputDocument::separator > as well as derived CSS rules like: > af|outputDocument::paragraph:first-child -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
