ICEfaces
  1. ICEfaces
  2. ICE-5911

RowSelector hover styling disappears after selection+deselection of a row

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Cannot Reproduce
    • Affects Version/s: 1.8.2-EE-GA_P01
    • Fix Version/s: 1.8.2-EE-GA_P02
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      All

      Description

      Using the rowSelection demo in ICEfaces Comp. Showcase:

      1. Hover over a row, row styles to indicate "hovered" state.
      2. Click mouse to select the row, row styles as "selected" (correct).
      3. Click mouse again to de-select row, row styles to "blank", instead of "hovered" styling which is expected. Moving the mouse returns the hover styling over the row. This doesn't help keyboard users though, unless they arrow-down to the next row and then arrow-up to the previous one again.

      The above occur using all selection modes (single, multiple, enh-multiple).

      4. In addition, when in enh-mulitple, after completing the steps above, if you hover over a different row(s), then return the mouse to the original row, it still does not style as hovered (moving the mouse does not return the "hovered" styling). If you subsequently select another row, then the previous row will function normally again.

      Note that the above issues are identical if keyboard navigation is used instead of the mouse to hover and select the rows.

        Activity

        Hide
        yip.ng added a comment - - edited

        Can't reproduce 3. (Tried many times with very steady clicks.) Both the highlight color changes on the row and the class name changes in Firebug show that it is working. See video at http://screencast.com/t/KgjXU8K9DJEm.

        Show
        yip.ng added a comment - - edited Can't reproduce 3. (Tried many times with very steady clicks.) Both the highlight color changes on the row and the class name changes in Firebug show that it is working. See video at http://screencast.com/t/KgjXU8K9DJEm .
        Hide
        Ken Fyten added a comment -

        Try using the keyboard exclusively to select the rows, you'll see the issue. I believe it's a logic problem in how we're triggering the client-side styling for highlighting. Right now it depends on a mouse event to work.

        Show
        Ken Fyten added a comment - Try using the keyboard exclusively to select the rows, you'll see the issue. I believe it's a logic problem in how we're triggering the client-side styling for highlighting. Right now it depends on a mouse event to work.
        Hide
        yip.ng added a comment - - edited

        Tried using enter and space bar to do select and de-select. Result same as using mouse.

        Show
        yip.ng added a comment - - edited Tried using enter and space bar to do select and de-select. Result same as using mouse.
        Hide
        yip.ng added a comment -

        Can't reproduce 4 either. Tried using (click and Ctrl-click) and (space bar and Ctrl-space bar).

        Show
        yip.ng added a comment - Can't reproduce 4 either. Tried using (click and Ctrl-click) and (space bar and Ctrl-space bar).
        Hide
        yip.ng added a comment -

        From Ken:

        Can you research changes to this component in JIRA to see if there are any recent commits that would explain this being resolved already?

        Show
        yip.ng added a comment - From Ken: Can you research changes to this component in JIRA to see if there are any recent commits that would explain this being resolved already?
        Hide
        yip.ng added a comment -

        Inspected svn log entries and changed files all the way back to June 30, 2010. Nothing looked like fix for this.

        Show
        yip.ng added a comment - Inspected svn log entries and changed files all the way back to June 30, 2010. Nothing looked like fix for this.
        Hide
        Ken Fyten added a comment -

        This issue cannot be reproduced using FF 3.6, IE 7, or Safari 5.0.2. Both the ICEfaces EE 1.8.2.GA_P01 release and current development trunk for P02 were tested.

        Show
        Ken Fyten added a comment - This issue cannot be reproduced using FF 3.6, IE 7, or Safari 5.0.2. Both the ICEfaces EE 1.8.2.GA_P01 release and current development trunk for P02 were tested.

          People

          • Assignee:
            yip.ng
            Reporter:
            Ken Fyten
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: