Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.2
    • Fix Version/s: None
    • Component/s: Components/Renderers
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      217
    • Status Whiteboard:
      Hide

      cat2 renderkitdoc size_small importance_small

      Show
      cat2 renderkitdoc size_small importance_small

      Description

      Add a styleClass attribute to h:column and relegate the columnClasses attribute
      from h:dataTable to a purpose similar to what rowClasses does: cycle through
      each item in the list repeatedly (the use case being the ability to apply
      alternating styles, like differently-colored even and odd columns, without
      requiring knowledge of the number of columns).

      There should be nothing in the <h:dataTable> attribute list that is dependant
      upon the actual number of columns appearing inside of the table. All
      information related to specific columns should be associated with each column's
      corresponding <h:column> entity. Indeed, this is already the case with the sole
      exception of the columnClasses attribute.

      This is related to: https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=426

        Issue Links

          Activity

          Hide
          codylerum added a comment -

          +1

          Defining classes one per column in order on the datatable via columnClasses is far to brittle. RichFaces has provided this for years in their rich:column via styleClass http://docs.jboss.org/richfaces/4.3.X/4.3.0.Final/vdldoc/

          Show
          codylerum added a comment - +1 Defining classes one per column in order on the datatable via columnClasses is far to brittle. RichFaces has provided this for years in their rich:column via styleClass http://docs.jboss.org/richfaces/4.3.X/4.3.0.Final/vdldoc/
          Hide
          Ed Burns added a comment -

          Set priority to baseline ahead of JSF 2.3 triage. Priorities will be assigned accurately after this exercise.

          Show
          Ed Burns added a comment - Set priority to baseline ahead of JSF 2.3 triage. Priorities will be assigned accurately after this exercise.
          Hide
          Manfred Riem added a comment -

          Setting priority to Major, because of higher vote count

          Show
          Manfred Riem added a comment - Setting priority to Major, because of higher vote count
          Hide
          codylerum added a comment -

          With Richfaces sun-setting in June 2016 it would be very nice to pick this feature up in 2.3

          This is really the only feature that keeps me from using the native JSF datatable

          Show
          codylerum added a comment - With Richfaces sun-setting in June 2016 it would be very nice to pick this feature up in 2.3 This is really the only feature that keeps me from using the native JSF datatable
          Hide
          codylerum added a comment -

          I've gone ahead and implemented styleClass on column and rowClass on datatable.

          This aligns with now Richfaces named the properties.

          https://github.com/codylerum/mojarra/tree/JAVASERVERFACES_SPEC_PUBLIC-217

          I'm going to need some review and pointers to know if I've updated the docs correctly.

          Show
          codylerum added a comment - I've gone ahead and implemented styleClass on column and rowClass on datatable. This aligns with now Richfaces named the properties. https://github.com/codylerum/mojarra/tree/JAVASERVERFACES_SPEC_PUBLIC-217 I'm going to need some review and pointers to know if I've updated the docs correctly.

            People

            • Assignee:
              Unassigned
              Reporter:
              dmlloyd
            • Votes:
              10 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated: