ICEfaces
  1. ICEfaces
  2. ICE-8071

ace:list accepts f:selectItem(s) children as model

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: 3.0.1
    • Fix Version/s: 3.1.0.BETA2, 3.1
    • Component/s: None
    • Labels:
      None
    • Environment:
      IF 3.1
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.)

      Description

      ace:list should be able to accept Facelet selectItem definitions in addition to programatic definitions from the bean.

        Activity

        Hide
        Nils Lundquist added a comment -

        Though it seems if we accept List<javax.faces.model.SelectItem> as a model, we should accept child f:selectItems as a model, upon deeper inspection, the component tree is a very ill suited place to keep the model for the list.

        f:selectItem isn't mutable, so these would only be worthwhile in lists where you never want to reorder or migrate your items. Not worth persuing.

        f:selectItems may be mutable, despite not being indented for that use. But then f:selectItems is essentially acting as extra markup to define what you could've defined as part of the value property of the list itself. Redundant.

        Show
        Nils Lundquist added a comment - Though it seems if we accept List<javax.faces.model.SelectItem> as a model, we should accept child f:selectItems as a model, upon deeper inspection, the component tree is a very ill suited place to keep the model for the list. f:selectItem isn't mutable, so these would only be worthwhile in lists where you never want to reorder or migrate your items. Not worth persuing. f:selectItems may be mutable, despite not being indented for that use. But then f:selectItems is essentially acting as extra markup to define what you could've defined as part of the value property of the list itself. Redundant.

          People

          • Assignee:
            Nils Lundquist
            Reporter:
            Nils Lundquist
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: