ICEfaces
  1. ICEfaces
  2. ICE-5013

ice:columns creates 1 extra <td> element than number of <th> elements when table is empty, as a result table renders badly

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 1.8.2
          • Fix Version/s: 1.8.2-EE-Beta, 1.8.2-EE-GA, 1.8.3
          • Component/s: ICE-Components
          • Labels:
            None
          • Environment:
            SUSE Linux 10 (x86_64), ICEFaces 1.8.2

            Description

            This seems to be a regression with 1.8.2.

            ice:columns used inside a ice:dataTable creates 1 extra <td> element than number of <th> elements when table is empty. As a result table renders badly. Screen shot of the table and HTML generated by ICEFaces is attached. Below is the code snippet -

                  <ice:dataTable>
                    <ice:columns>
                      <f:facet name="header">
                        <ice:commandSortHeader>
                          <ice:outputText />
                        </ice:commandSortHeader>
                      </f:facet>
                    </ice:columns>
                    <ice:outputText />
                  </ice:dataTable>

            This problem is not seen if <ice:columns> is replaced by multiple <ice:column> elements, however, this workaround is not useful in our use case, where the file containing this dataTable is a reusable facelet and used by many other pages.
            1. 1.8.2_columns_issue.JPG
              44 kB
            2. screenshot-1.png
              205 kB
            3. screenshot-2.png
              220 kB

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              yip.ng added a comment -

              Issue caused by one-off error in calculation. Fixed. See screen shots 1 and 2.

              Show
              yip.ng added a comment - Issue caused by one-off error in calculation. Fixed. See screen shots 1 and 2.
              Hide
              Permalink
              Joanne Bai added a comment -

              Tested with success on ICEfaces-ee-1.8.2 revision #19954

              Tested on Tomcat 6 + FF3.5 and IE8

              Test app has been committed to repo\qa\trunk\Regression\ICE-5013

              Show
              Joanne Bai added a comment - Tested with success on ICEfaces-ee-1.8.2 revision #19954 Tested on Tomcat 6 + FF3.5 and IE8 Test app has been committed to repo\qa\trunk\Regression\ ICE-5013

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  Abhijit Jere
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: