Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.1, 3.1.0.BETA2
    • Fix Version/s: 3.1.0.RC1, 3.1, EE-3.0.0.GA_P01
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Showcase application, Chrome, IE9, Firefox (bug#2 only)
    • Assignee Priority:
      P1

      Description

      Bug#1:
      Wrapping the column content in a <div> lets ice.ace.DataTable.prototype.resizeScrolling from datatable.js count too many columns in the body:

      var bodySingleCols = ice.ace.jq(dataTable.jqId + ' .ui-datatable-scrollable-body:first tbody tr:first td div:first-child').get().reverse();

      Hence bodySingleCols.length is higher than realHeadSingleCols.length / dupeHeadSingleCols.length.

      As bodySingleCols.length is used in the for-loop assigning the column widths, there are invalid indices (e.g. bodySingleCols counts 8, realHeadSingleCols counts 6) in

      for (i = 0; i < bodySingleCols.length; i++) {
      realHeadColumn = ice.ace.jq(realHeadSingleCols[i]);
      [...]
      }

      Bug#2:
      Columns of the ace:dataTable appear misaligned with header when you enable scrolling.
      ----------------------------------
      Use showcase application to reproduce both bugs.
      Screenshot attached.

        Activity

        Evgheni Sadovoi created issue -
        Evgheni Sadovoi made changes -
        Field Original Value New Value
        Salesforce Case [5007000000MIc6Q]
        Evgheni Sadovoi made changes -
        Attachment screenShot.jpg [ 14628 ]
        Evgheni Sadovoi made changes -
        Affects Version/s 3.1.0.BETA2 [ 10336 ]
        Hide
        J Bloemer added a comment -

        Actually the two bugs are both connected to the wrong column count hence could be solved by solving this central issue. The datatable fails to synch headers and body due to the different column count.

        Show
        J Bloemer added a comment - Actually the two bugs are both connected to the wrong column count hence could be solved by solving this central issue. The datatable fails to synch headers and body due to the different column count.
        Ken Fyten made changes -
        Fix Version/s 3.1.0.RC1 [ 10337 ]
        Fix Version/s 3.1 [ 10312 ]
        Fix Version/s EE-3.0.0.GA_P01 [ 10327 ]
        Assignee Priority P1
        Assignee Nils Lundquist [ nils.lundquist ]
        Hide
        Nils Lundquist added a comment -

        Fix for issue was confirmed in latest trunk. Backport to EE will be coming shortly.

        Show
        Nils Lundquist added a comment - Fix for issue was confirmed in latest trunk. Backport to EE will be coming shortly.
        Nils Lundquist made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Ken Fyten added a comment -

        Backported to the icefaces-3.0.x-maintenance branch.

        This patch includes fixes for the following specific issues related to this case:

        ICE-8104 - Multi-row headers that rowspan on the bottom row of the header section cause column/header mis-alignment in scrollable mode (fixes the client's issue)
        ICE-8237 - Scrollable table column misalignement when rendering a table lower in a page
        ICE-8236 - horizontal scrollbar does not scroll the headers along with the body content

        Show
        Ken Fyten added a comment - Backported to the icefaces-3.0.x-maintenance branch. This patch includes fixes for the following specific issues related to this case: ICE-8104 - Multi-row headers that rowspan on the bottom row of the header section cause column/header mis-alignment in scrollable mode (fixes the client's issue) ICE-8237 - Scrollable table column misalignement when rendering a table lower in a page ICE-8236 - horizontal scrollbar does not scroll the headers along with the body content
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: