Uploaded image for project: 'MyFaces Tomahawk'
  1. MyFaces Tomahawk
  2. TOMAHAWK-368

modify NewspaperTable columnClasses to be more useful

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.1.2
    • None
    • None

    Description

      Currently the NewspaperTable columnClasses with multiple comma separated classes work only with the column tags themselves,
      not the resulting html columns in the table. This change allows the html table to have different column classes across all columns in the html table.
      Currently, if you declare:
      <t:newspaperTable newspaperColumns="3" columnClasses="class1,class2,class3" . . . >
      <t:column>
      <f:verbatim>A</f:verbatim>
      </t:column>
      </t:newspaperTable>
      all the columns will be class1

      <t:newspaperTable newspaperColumns="3" columnClasses="class1,class2,class3" . . . >
      <t:column>
      <f:verbatim>A</f:verbatim>
      </t:column>
      <t:column>
      <f:verbatim>B</f:verbatim>
      </t:column>
      </t:newspaperTable>
      all the As will be class1 and Bs will be class2

      with this change
      <t:newspaperTable newspaperColumns="3" columnClasses="class1,class2,class3" . . . >
      <t:column>
      <f:verbatim>A</f:verbatim>
      </t:column>
      </t:newspaperTable>
      will have the first column class1, the second column class2, the third class3

      <t:newspaperTable newspaperColumns="3" columnClasses="class1,class2,class3",class4,class5,class6" . . . >
      <t:column>
      <f:verbatim>A</f:verbatim>
      </t:column>
      <t:column>
      <f:verbatim>B</f:verbatim>
      </t:column>
      </t:newspaperTable>
      the first A column will be class1, the first B column class2, the 2nd A column class 3, the 2nd B column class4, ...

      This makes it more flexible.

      PLEASE email me at frohman@pacbell.net if there is a problem, or if you have any questions.
      Thank you.

      Attachments

        Activity

          People

            Unassigned Unassigned
            lfrohman Lance Frohman
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: