[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-2148) Support for SelectItemGroup

[Stephen Friedrich (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBSEAM-2148?page=comments#action_12419902 ] 
            
Stephen Friedrich commented on JBSEAM-2148:
-------------------------------------------

Oh my, after a good night's sleep:
Besides being a little too complex, that solution has a specific issue: You can't easily use 
<s:selectItems group="Foo" .../> 
like Pete said in the issue description.
Only this would work
<s:selectItems group="Foo" groupLabel="Foo" .../> 
Hm, either make group default to the groupLabel (and vice versa).
Or simply remove both "group" and "groupVar" and simply only ever group the items using the label.
That would maybe make groupDisabled and groupEscape EL expressions more complex (because they can't refer to the groupVar), but those attributes will probably be used very rarely.

I think I'll rework the patch to the latter alternative.

> Support for SelectItemGroup
> ---------------------------
>
>                 Key: JBSEAM-2148
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-2148
>             Project: Seam
>          Issue Type: Feature Request
>          Components: JSF Controls
>    Affects Versions: 2.0.0.CR2
>            Reporter: Pete Muir
>             Fix For: 2.1.0.BETA2
>
>         Attachments: selectItems_group.patch
>
>
> We can do something like
> <s:selectItemGroup label="#{foo.bar}">
>    <s:selectItems .../>
>    <s:selectItems .../>
> </s:selectItemGroup>
> and, as a shorthand to put each list in a distinct group
>    <s:selectItems group="Foo"/>
> I don't understand why this isn't in JSF core.

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