Feature #21651
closed
Use virtual lists for panels with large amounts of text
Added by Peter Amstutz about 2 months ago.
Updated about 1 month ago.
Release relationship:
Auto
Description
From discussion:
Virtual lists significantly improves performance of the I/O panels, but it turns out the "JSON" I/O tab and "Command" process panel are still very slow when they start to become very large.
These are "CodeSnippet" controls.
These are good candidates to be virtualized because the main prop is the "lines" array and they are monospace text of fixed height.
Create a "VirtualCodeSnippet" that has the same interface but uses a virtual list, and use that for the JSON I/O tab and Command panel.
- Description updated (diff)
- Description updated (diff)
- Related to Bug #21508: Browser struggles with very large number of input or output parameters added
- Status changed from New to In Progress
Changes at arvados|4d892bd66c5d11d8cbde42348099e92da0c21a99 branch 21651-cmd-card-virtual-list
Tests developer-run-tests-services-workbench2: #692 
- All agreed upon points are implemented / addressed.
- Added virtual code snippet, implemented in cmd card
- Added on demand line formatting to virtual code snippet
- Added custom copy to clipboard component to use callback to format and fetch copied text on demand using callback
- Implemented virtual list in json tab
- Updated unit tests, browserdb, removed warnings
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- Code is tested and passing, both automated and manual, what manual testing was done is described
- Passes automated tests, manually tested with many rows
- Documentation has been updated.
- Behaves appropriately at the intended scale (describe intended scale).
- Considered backwards and forwards compatibility issues between client and server.
- Follows our coding standards and GUI style guidelines.
Stephen Smith wrote in #note-6:
Changes at arvados|4d892bd66c5d11d8cbde42348099e92da0c21a99 branch 21651-cmd-card-virtual-list
Tests developer-run-tests-services-workbench2: #692
- All agreed upon points are implemented / addressed.
- Added virtual code snippet, implemented in cmd card
- Added on demand line formatting to virtual code snippet
- Added custom copy to clipboard component to use callback to format and fetch copied text on demand using callback
- Implemented virtual list in json tab
- Updated unit tests, browserdb, removed warnings
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- Code is tested and passing, both automated and manual, what manual testing was done is described
- Passes automated tests, manually tested with many rows
- Documentation has been updated.
- Behaves appropriately at the intended scale (describe intended scale).
- Considered backwards and forwards compatibility issues between client and server.
- Follows our coding standards and GUI style guidelines.
This is great, I tried out stress test container request and it works very well, loads almost immediately and is perfectly performant instead of causing the browser to struggle and stutter. Huge, huge improvement, thank you. Please merge.
- Status changed from In Progress to Resolved
Also available in: Atom
PDF
Updated by 
Updated by