[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-1572) Make s:decorate behave more similarly to ui:decorate (h:panelGrid-friendly)

Wolfgang Schwendt (JIRA) jira-events at lists.jboss.org
Tue Aug 26 20:56:38 EDT 2008


    [ https://jira.jboss.org/jira/browse/JBSEAM-1572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12426829#action_12426829 ] 

Wolfgang Schwendt commented on JBSEAM-1572:
-------------------------------------------

Hello, I just discovered this JIRA entry after posting a patch that adds an optional attribute to s:decorate to disable the rendering of the enclosing div-element. (JBSEAM-3339)

I second the comment of Chris Simons. Why would AJAX need to re-render the enclosing div? Can't it just re-render the children of the s:decorate? And even if there are cases when a div-element is needed (because of AJAX), can't we give the user more control to disable the rendering of the div element when it's not needed?

I have been using a patched version of s:decorate for months already, which disables the rendering of the div-element.  And it works fine.  It would be nice if an enhancement to (optionally) disable the div could be integrated into Seam.

In my view, it's much more important to disable the <div> than to replace the <div> by another html-tag.

> Make s:decorate behave more similarly to ui:decorate (h:panelGrid-friendly)
> ---------------------------------------------------------------------------
>
>                 Key: JBSEAM-1572
>                 URL: https://jira.jboss.org/jira/browse/JBSEAM-1572
>             Project: Seam
>          Issue Type: Feature Request
>          Components: JSF Controls
>            Reporter: Alex Savitsky
>            Priority: Minor
>             Fix For: The future
>
>         Attachments: renderTagDiff.txt
>
>
> ui:decorate preserves its children, while s:decorate groups them as one instead. This makes s:decorate unusable for form fields laid out using h:panelGrid, as there's no way to separate the field label from the field itself (they are grouped together).

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        



More information about the seam-issues mailing list