ICEfaces
  1. ICEfaces
  2. ICE-1635

row selector performance does not scale well

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6DR#4
    • Fix Version/s: 1.6.1
    • Component/s: ICE-Components
    • Labels:
      None
    • Environment:
      ICEfaces

      Description

      Tables with a row selector show a linear curve of performance to # rows.

      When there are over 200 rows, the row selector mouse over effect becomes very delayed. I specifically mention this as that effect is purely java script; therefore making no server round trip. All the other components are equally affected; such as clicking a button to move to the next page; can take upwards of 3-5 seconds.

        Activity

        Hide
        Ted Goddard added a comment -

        We will need to look at the specific JavaScript algorithm for handling the mouse overs. In the case of drag and drop, we were able to make substantial improvements to the collision detection algorithm by including a level-of-detail strategy (initially checking for collision at a low level of detail). Likely a similar strategy can be employed here.

        Show
        Ted Goddard added a comment - We will need to look at the specific JavaScript algorithm for handling the mouse overs. In the case of drag and drop, we were able to make substantial improvements to the collision detection algorithm by including a level-of-detail strategy (initially checking for collision at a low level of detail). Likely a similar strategy can be employed here.
        Hide
        Adnan Durrani added a comment -

        There are significant improvement has been made regarding response time, Its fixed. However under IE hover effect still a bit sluggish but better then before.

        revision : 14673

        Show
        Adnan Durrani added a comment - There are significant improvement has been made regarding response time, Its fixed. However under IE hover effect still a bit sluggish but better then before. revision : 14673

          People

          • Assignee:
            Unassigned
            Reporter:
            Ted Goddard
          • Votes:
            2 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: