Arvados

Changes at arvados|b220283a4a532ca37a9f1f74ddcab93dffe828cc branch 21893-input-collections-virtual-list
Tests developer-run-tests-services-workbench2: #1140

• All agreed upon points are implemented / addressed.
  • Implement virtual list for process io card input collections tab
  • Add separate custom column width styles for collection input table
• Anything not implemented (discovered or discussed during work) has a follow-up story.
  • n/a
• Code is tested and passing, both automated and manual, what manual testing was done is described
  • tested manually by injecting 400,000 rows
• Documentation has been updated.
  • n/a
• Behaves appropriately at the intended scale (describe intended scale).
  • Up to ~400 rows or until virtual list scrolling breaks
• Considered backwards and forwards compatibility issues between client and server.
  • none
• Follows our coding standards and GUI style guidelines.
  • no significant design changes

Code updates LGTM, my only suggestion (unrelated to this ticket) is to keep the linter happy:

WARNING in [eslint]
src/store/processes/processes-actions.ts
  Line 9:10:   'Process' is defined but never used                   @typescript-eslint/no-unused-vars
  Line 24:10:  'ContainerResource' is defined but never used         @typescript-eslint/no-unused-vars
  Line 25:10:  'ContainerRequestResource' is defined but never used  @typescript-eslint/no-unused-vars

src/store/workbench/workbench-actions.ts
  Line 50:13:  'processesActions' is defined but never used  @typescript-eslint/no-unused-vars

src/views-components/context-menu/actions/helpers.ts
  Line 5:10:   'REDIRECT_TO_DOWNLOAD_KEY' is defined but never used  @typescript-eslint/no-unused-vars
  Line 5:36:   'REDIRECT_TO_PREVIEW_KEY' is defined but never used   @typescript-eslint/no-unused-vars
  Line 19:12:  'uuid' is assigned a value but never used             @typescript-eslint/no-unused-vars

src/views/process-panel/process-details-attributes.tsx
  Line 13:10:   'Process' is defined but never used  @typescript-eslint/no-unused-vars
  Line 137:40:  Expected '===' and instead saw '=='  eqeqeq

src/views/run-process-panel/run-process-advanced-form.tsx
  Line 14:10:  'PROCESS_NAME_VALIDATION' is defined but never used  @typescript-eslint/no-unused-vars