ICEfaces
  1. ICEfaces
  2. ICE-10337

ace:dataTable - multiple row selection styling issue

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Minor Minor
          • Resolution: Fixed
          • Affects Version/s: 4.0
          • Fix Version/s: EE-4.0.0.GA, 4.1
          • Component/s: ACE-Components, QA
          • Labels:
            None
          • Environment:
            ICEfaces4 trunk rev. 43051/ 43075
            Browsers: IE10/IE11
            Server: Tomcat 7.0.42
          • Assignee Priority:
            P2

            Description

            In ace:dataTable test application > Multi Row Selection Test: when selecting multiple rows using the "shift" key, all rows are selected including the table headers, until user clicks outside the table (IE only).

            QA test app for ace:dataTable:
            http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Nightly/dataTable

            Test file: /multiSelectionTest.jsf
            1. multiIE8.PNG
              31 kB
            2. multiIE9.PNG
              27 kB
            3. multiRowChrome.png
              80 kB
            4. multiRowFF.png
              82 kB
            5. multiRowIE.png
              121 kB

              Activity

              Ascending order - Click to sort in descending order
              Carmen Cristurean created issue -
              Ken Fyten made changes -
              Field Original Value New Value
              Workaround Description Need to disable the underlying DOM event so the browser doesn't also select the text under the mouse when selecting the rows via Shift-click.
              Assignee Arturo Zambrano [ artzambrano ]
              Fix Version/s EE-4.0.0.GA [ 11171 ]
              Priority Major [ 3 ] Minor [ 4 ]
              Assignee Priority P3 [ 10012 ]
              Hide
              Permalink
              Ken Fyten added a comment -

              Need to disable the underlying DOM event so the browser doesn't also select the text under the mouse when selecting the rows via Shift-click.

              Show
              Ken Fyten added a comment - Need to disable the underlying DOM event so the browser doesn't also select the text under the mouse when selecting the rows via Shift-click.
              Ken Fyten made changes -
              Workaround Description Need to disable the underlying DOM event so the browser doesn't also select the text under the mouse when selecting the rows via Shift-click.
              Ken Fyten made changes -
              Assignee Priority P3 [ 10012 ] P2 [ 10011 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43911 Thu Jan 15 11:33:08 MST 2015 art.zambrano ICE-10337 disabled ability to select text in the table when pressing the shift key and clicking on a row
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Arturo Zambrano added a comment -

              r43911: disabled ability to select text in the table when pressing the shift key and clicking on a row, for all browsers.

              Testing notes: I could reproduce this issue on other browsers as well, so please verify this fix in all browsers.

              Show
              Arturo Zambrano added a comment - r43911: disabled ability to select text in the table when pressing the shift key and clicking on a row, for all browsers. Testing notes: I could reproduce this issue on other browsers as well, so please verify this fix in all browsers.
              Arturo Zambrano made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Summary ace:dataTable - multiple row selection styling issue (IE10/11) ace:dataTable - multiple row selection styling issue
              Ken Fyten made changes -
              Security Private [ 10001 ]
              Hide
              Permalink
              Liana Munroe added a comment -

              Tested with ICEfaces 4 trunk r43917. Tomcat 7, IE 11, FF 34, Chrome 39. Issue still remains when selecting rows using the shift key. Please see attached screen shots.

              Show
              Liana Munroe added a comment - Tested with ICEfaces 4 trunk r43917. Tomcat 7, IE 11, FF 34, Chrome 39. Issue still remains when selecting rows using the shift key. Please see attached screen shots.
              Liana Munroe made changes -
              Attachment multiRowChrome.png [ 17894 ]
              Attachment multiRowFF.png [ 17895 ]
              Attachment multiRowIE.png [ 17896 ]
              Liana Munroe made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43942 Thu Jan 22 11:18:14 MST 2015 art.zambrano ICE-10337 disabled ability to select text in the table when pressing the shift key and clicking on a row (re-commit)
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Arturo Zambrano added a comment -

              Committed fix again at revision 43942. It had been removed when committing a fix for ICE-10445.

              Show
              Arturo Zambrano added a comment - Committed fix again at revision 43942. It had been removed when committing a fix for ICE-10445 .
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Liana Munroe added a comment -

              Tested with ICEfaces4 trunk r43942. Verified using single and double click selections with Chrome and FF. Single click selection is verified fixed in IE browsers but the double click selection of rows still causes the highlighting issue as seen in screenshot multiRowIE.png

              Show
              Liana Munroe added a comment - Tested with ICEfaces4 trunk r43942. Verified using single and double click selections with Chrome and FF. Single click selection is verified fixed in IE browsers but the double click selection of rows still causes the highlighting issue as seen in screenshot multiRowIE.png
              Liana Munroe made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43950 Thu Jan 22 15:21:11 MST 2015 art.zambrano ICE-10337 modified fix to prevent element/page selection on IE when pressing shift and doing double click
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Arturo Zambrano added a comment -

              r43950: modified fix to prevent element/page selection on IE when pressing shift and doing double click.

              Show
              Arturo Zambrano added a comment - r43950: modified fix to prevent element/page selection on IE when pressing shift and doing double click.
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              Liana Munroe added a comment -

              Tested with ICEfaces 4 trunk r43950. Verified double-click + shift selection issue fixed in IE 10 and 11. IE 8 and 9 still highlight text when using double-click + shift selection. See screenshots multiIE8.png and multiIE9.png

              Show
              Liana Munroe added a comment - Tested with ICEfaces 4 trunk r43950. Verified double-click + shift selection issue fixed in IE 10 and 11. IE 8 and 9 still highlight text when using double-click + shift selection. See screenshots multiIE8.png and multiIE9.png
              Liana Munroe made changes -
              Resolution Fixed [ 1 ]
              Status Resolved [ 5 ] Reopened [ 4 ]
              Liana Munroe made changes -
              Attachment multiIE8.PNG [ 17997 ]
              Attachment multiIE9.PNG [ 17998 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43963 Tue Jan 27 15:23:49 MST 2015 art.zambrano ICE-10337 additional fix for IE8 and IE9
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Arturo Zambrano added a comment -

              r43963: committed additional fix for IE8 and IE9.

              Show
              Arturo Zambrano added a comment - r43963: committed additional fix for IE8 and IE9.
              Arturo Zambrano made changes -
              Status Reopened [ 4 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #43964 Tue Jan 27 15:40:01 MST 2015 art.zambrano ICE-10337 additional fix for IE8 and IE9
              Files Changed
              Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
              Hide
              Permalink
              Liana Munroe added a comment -

              Verified ICEfaces 4 trunk r43964. IE8, IE9, Tomcat 7

              Show
              Liana Munroe added a comment - Verified ICEfaces 4 trunk r43964. IE8, IE9, Tomcat 7
              Ken Fyten made changes -
              Fix Version/s 4.1 [ 11375 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Arturo Zambrano
                  Reporter:
                  Carmen Cristurean
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: