-
Notifications
You must be signed in to change notification settings - Fork 5
Description
When the current results UI shows a list of 10,000 tests or more, scrolling using the scroll bar is janky and hangs the web client.
This is due to issue flutter/flutter#52207 . A long ListView in Flutter cannot have variable-sized items without creating all the items in the list to find the scroll length and scroll position. This is fixed by specifying the vertical size of all items with an itemExtent or prototypeItem argument to the ListView constructor, giving the common size for all items.
This bug was reported in #143 by a user.
Potential fixes are to move the expansion of a test item to show all the configurations and links for that test to a popup, side panel, top panel, or overlay. This UI might be less usable than the current expander button on an item.
If the expension is an overlay, it will cover over other test names, and perhaps two tests won't be expandable at the same time. Dismissing the popup will also require either a dismiss button, a click, or hitting the escape key.
Putting the details in a top panel has the advantage that the test list won't be covered over, just smaller, but the case that
the top panel needs scrolling will need to be handled too.
I propose having some UI discussions with other team members and Flutter UI team, then carrying out this fix.