ICEfaces
  1. ICEfaces
  2. ICE-9492

ace:dataTable > Pinning & TableConfig features cause page to scroll to top

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.3.0.GA_P01, 4.0.BETA
          • Fix Version/s: 4.0.BETA, EE-3.3.0.GA_P02, 4.0
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            ICEfaces3 trunk revision# 37274
          • Assignee Priority:
            P2

            Description

            The page scrolls to the top when
            1) Pinning/Unpinning
            2) Changing dataTable state via the tableConfigPanel

            Both issues can be easily reproduced in Showcase if making the window smaller so that the page has a scroll bar, and then performing one of the actions.

              Activity

              Ascending order - Click to sort in descending order
              Cruz Miraback created issue -
              Cruz Miraback made changes -
              Field Original Value New Value
              Assignee Nils Lundquist [ nils.lundquist ]
              Ken Fyten made changes -
              Fix Version/s 3.4 [ 10770 ]
              Assignee Priority P2 [ 10011 ]
              Cruz Miraback made changes -
              Summary ace:dataTable > Pinning/Unpinning causes page to scroll to top ace:dataTable > Pinning & TableConfig features cause page to scroll to top
              Description When pinning/unpinning a dataTable column the page scrolls up to the top.

              To reproduce:
              1) Build / deploy test app located at: http://server.ice:8888/svn/repo/qa/trunk/Regression-Icefaces2/Sparkle/Nightly/dataTable
              2) Navigate to DataTable Ajax test
              3) Scroll down to the pin/unpin event test
              4) Pin a column (Page scrolls to top)               		The page scrolls to the top when
              1) Pinning/Unpinning
              2) Changing values via the tableConfigPanel

              Both issues can be easily reproduced in Showcase if making the window smaller so that the page has a scroll bar, and then performing one of the actions.
              Cruz Miraback made changes -
              Comment [ A comment with security level 'icesoft-internal-developers' was removed. ]
              Cruz Miraback made changes -
              Description The page scrolls to the top when
              1) Pinning/Unpinning
              2) Changing values via the tableConfigPanel

              Both issues can be easily reproduced in Showcase if making the window smaller so that the page has a scroll bar, and then performing one of the actions.               		The page scrolls to the top when
              1) Pinning/Unpinning
              2) Changing dataTable state via the tableConfigPanel

              Both issues can be easily reproduced in Showcase if making the window smaller so that the page has a scroll bar, and then performing one of the actions.
              Ken Fyten made changes -
              Assignee Nils Lundquist [ nils.lundquist ] Arturo Zambrano [ artzambrano ]
              Assignee Priority P2 [ 10011 ] P1 [ 10010 ]
              Hide
              Permalink
              Arturo Zambrano added a comment - - edited

              r38726:
              Added 'onclick="return false;"' to pinning and table config buttons (<a> elements) to prevent scrolling to top.

              Show
              Arturo Zambrano added a comment - - edited r38726: Added 'onclick="return false;"' to pinning and table config buttons (<a> elements) to prevent scrolling to top.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #38726 Tue Nov 05 10:47:00 MST 2013 art.zambrano ICE-9492 added 'return false;' to pinning and table config buttons (<a> elements) to prevent scrolling to top
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/tableconfigpanel/TableConfigPanelRenderer.java
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableHeadRenderer.java
              Ken Fyten made changes -
              Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
              Ken Fyten made changes -
              Affects Version/s EE-3.3.0.GA_P01 [ 11174 ]
              Ken Fyten made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Assignee Priority P1 [ 10010 ] P2 [ 10011 ]
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Backported fix to 3.3 EE maintenance branch at revision 38734.

              Show
              Arturo Zambrano added a comment - Backported fix to 3.3 EE maintenance branch at revision 38734.
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Carmen Cristurean added a comment -

              Fix confirmed with icefaces-ee-3.3.0_P01-Williams-tableConfigEnh-Build rev# 38805 (Firefox25, IE10, Chrome30).

              Show
              Carmen Cristurean added a comment - Fix confirmed with icefaces-ee-3.3.0_P01-Williams-tableConfigEnh-Build rev# 38805 (Firefox25, IE10, Chrome30).
              Hide
              Permalink
              Liana Munroe added a comment - - edited

              Icefaces 4, Tomcat 7. I can still reproduce the pinning issue in the ajax tests "pin Event Test" (execute=@all render=@all) mojarra 2.1 and 2.2 with FF 3.6, not in any other browser)

              Show
              Liana Munroe added a comment - - edited Icefaces 4, Tomcat 7. I can still reproduce the pinning issue in the ajax tests "pin Event Test" (execute=@all render=@all) mojarra 2.1 and 2.2 with FF 3.6, not in any other browser)
              Liana Munroe made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Hide
              Permalink
              Liana Munroe added a comment -

              FF 3.6 only problem with scrolling will be considered a known issue.

              Show
              Liana Munroe added a comment - FF 3.6 only problem with scrolling will be considered a known issue.
              Liana Munroe made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 4.0 [ 11382 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Cruz Miraback
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  5 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: