ICEfaces
  1. ICEfaces
  2. ICE-8228

ace:column - Implement the sortable attribute

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 3.1.0.BETA1
    • Fix Version/s: 3.1.0.RC1, 3.1
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces 3.1.x
    • Assignee Priority:
      P2
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      With the last ICEfaces 3.1.0 BETA1 release a new attribute was added to the TLD for the ace:column. At the moment setting this attribute had no effect. It is possible to dynamically disable sorting via some el expressions on the sortBy attribute but this still causes the sorting buttons to render for the column.

        Activity

        Hide
        Ken Fyten added a comment -

        This either has to work or be removed.

        Show
        Ken Fyten added a comment - This either has to work or be removed.
        Hide
        Nils Lundquist added a comment -

        I think it be removed as the logic it represents is easily expressed by the app in the value binding it gives for sort by.

        A commit will be forthcoming.

        Show
        Nils Lundquist added a comment - I think it be removed as the logic it represents is easily expressed by the app in the value binding it gives for sort by. A commit will be forthcoming.
        Hide
        Nils Lundquist added a comment -

        Revision #29485
        Committed by nils.lundquist
        Moments ago
        ICE-8228 - Removed 'sortable' and other per-feature properties, these are redundant until the column 'value' property is added.

        Show
        Nils Lundquist added a comment - Revision #29485 Committed by nils.lundquist Moments ago ICE-8228 - Removed 'sortable' and other per-feature properties, these are redundant until the column 'value' property is added.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: