ICEfaces
  1. ICEfaces
  2. ICE-10727

dataTable footer does not resize when rowEditor edit function is activated

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.0, EE-4.0.0.GA, EE-3.3.0.GA_P03
    • Fix Version/s: 4.1, EE-3.3.0.GA_P04
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Tomcat 7, all browsers, ICEfaces 4 trunk r45004.
    • Assignee Priority:
      P2

      Description

      Scenario - a dataTable that has column resizing, a footer facet and ace:rowEditor/ace:cellEditor. When the edit function is activated, and the resulting editing view changes the dataTable width to match the rowEditor width, the footer does not resize to match the new dataTable width. Test apps (/views/cellEditFooter.jsf) located at:
      http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dataTable
      http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/dataTable
      1. 1.PNG
        20 kB
      2. 2.PNG
        20 kB
      3. footer.PNG
        6 kB
      4. footer2.PNG
        21 kB
      5. footer-no-rows.PNG
        41 kB

        Activity

        Hide
        Arturo Zambrano added a comment -

        Committed fix to the 4.1 trunk at revision 45011 and to the 3.3 EE maintenance branch at revision 45013. Added function to adjust the width of the footer to that of the table body when editing rows.

        Show
        Arturo Zambrano added a comment - Committed fix to the 4.1 trunk at revision 45011 and to the 3.3 EE maintenance branch at revision 45013. Added function to adjust the width of the footer to that of the table body when editing rows.
        Hide
        Liana Munroe added a comment -

        Tested with ICEfaces 4 trunk r45020 and EE-3.3.0 maintenance branch r 45023.
        When testing with ICEfaces 4 trunk the issue is resolved however after loading the demo the Footer is a slight bit less than the width of the table. See screen shot footer.PNG
        When testing with ICEfaces EE-3.3.0 maintenance branch the footer is the correct width after loading the demo but it still has the issue where it does not resize to table width when the cellEditor is in use. See screen shot footer2.PNG

        Show
        Liana Munroe added a comment - Tested with ICEfaces 4 trunk r45020 and EE-3.3.0 maintenance branch r 45023. When testing with ICEfaces 4 trunk the issue is resolved however after loading the demo the Footer is a slight bit less than the width of the table. See screen shot footer.PNG When testing with ICEfaces EE-3.3.0 maintenance branch the footer is the correct width after loading the demo but it still has the issue where it does not resize to table width when the cellEditor is in use. See screen shot footer2.PNG
        Hide
        Mistah Mikey added a comment -

        This solves the issue I raised, but unfortunately, it causes a new one. After the footer is resized to match the expanded table width, it no longer will resize when you resize the browser window. The header resizes correctly (I raised an issue in the past about that not working, but now it appears to), but the footer stays the same size as when it was expanded.

        Show
        Mistah Mikey added a comment - This solves the issue I raised, but unfortunately, it causes a new one. After the footer is resized to match the expanded table width, it no longer will resize when you resize the browser window. The header resizes correctly (I raised an issue in the past about that not working, but now it appears to), but the footer stays the same size as when it was expanded.
        Hide
        Arturo Zambrano added a comment -

        r45045,r45046: fixed the issues reported by QA in the 4.0 trunk and the 3.3 EE maintenance branch.

        The width was adjusted by 1, and the table body selector was corrected in the 3.3 branch.

        Show
        Arturo Zambrano added a comment - r45045,r45046: fixed the issues reported by QA in the 4.0 trunk and the 3.3 EE maintenance branch. The width was adjusted by 1, and the table body selector was corrected in the 3.3 branch.
        Hide
        Arturo Zambrano added a comment -

        r45087, r45088: added event listener to adjust the dataTable footer width when the window is resized.

        Show
        Arturo Zambrano added a comment - r45087, r45088: added event listener to adjust the dataTable footer width when the window is resized.
        Hide
        Mistah Mikey added a comment -

        Things are now working as expected from my end. Thanks very much.

        Show
        Mistah Mikey added a comment - Things are now working as expected from my end. Thanks very much.
        Hide
        Mistah Mikey added a comment -

        Today I discovered a new footer resize problem. When I delete the last row in the table (so now the table is empty), the table header stays the same size it was (all columns displayed), but the footer collapses to the left into a very small rectangle.

        Show
        Mistah Mikey added a comment - Today I discovered a new footer resize problem. When I delete the last row in the table (so now the table is empty), the table header stays the same size it was (all columns displayed), but the footer collapses to the left into a very small rectangle.
        Hide
        Arturo Zambrano added a comment -

        r45579,r45580: committed fix to adjust width when there are no rows in the body, to the 4.1 trunk and the 3.3 EE maintenance branch.

        Show
        Arturo Zambrano added a comment - r45579,r45580: committed fix to adjust width when there are no rows in the body, to the 4.1 trunk and the 3.3 EE maintenance branch.
        Hide
        Liana Munroe added a comment -

        Verified ICEfaces 4 trunk, ee-3.3.0 maintenance branch r45580, Tomcat 7, IE 11, 10, 9, 8, 7 FF 34, Chrome 43.

        Show
        Liana Munroe added a comment - Verified ICEfaces 4 trunk, ee-3.3.0 maintenance branch r45580, Tomcat 7, IE 11, 10, 9, 8, 7 FF 34, Chrome 43.

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Liana Munroe
          • Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: