Project

General

Profile

Actions
Copy link

Bug #22379

open

Refactor Virtualized lists

Added by Lisa Knox over 1 year ago. Updated 6 days ago.

Status:
New
Priority:
Normal
Assigned To:
Lisa Knox
Category:
Workbench2
Target version:
Development 2026-03-18
Story points:
-
Release:
Arvados Frontend Bugs
Release relationship:
Auto

Description

The primary goal of this issue is to remove the MUI errors that appear in the browser console when viewing a virtualized list in Workbench. The problem is that the Autosizer and FixedSizeList elements each return a div wrapping the child elements, the last of which is a TableRow. MUI throws a validateDOMNesting warning because the only legal child of a TableBody is a TableRow, and the current implementation adds two divs between them.

A possible solution is to bump MUI to 6^ to take advantage of the virtualized table support:
https://mui.com/material-ui/migration/upgrade-to-v6/
https://mui.com/material-ui/react-table/#virtualized-table

It is also possible that the warning could be eliminated by manipulating the innerElement/outerElement props on those two components, but I tried and could not come up with a solution that both removed the warning and rendered the table properly.

Subtasks 1 (1 open — 0 closed)

Task #23341: ReviewNewStephen SmithActions

Related issues 3 (2 open — 1 closed)

Related to Arvados - Bug #22231: Clean up MUI and react warnings/errors that appear on the browser consoleResolvedLisa KnoxActions
Related to Arvados - Bug #22856: <tr> cannot appear as a child of <div>NewActions
Related to Arvados - Bug #22381: Replace redux-formIn ProgressLisa Knox03/18/2026Actions
Actions
Copy link
 #1

Updated by Lisa Knox over 1 year ago

  • Related to Bug #22231: Clean up MUI and react warnings/errors that appear on the browser console added
Actions
Copy link
 #2

Updated by Brett Smith 7 months ago

  • Related to Bug #22856: <tr> cannot appear as a child of <div> added
Actions
Copy link
 #3

Updated by Brett Smith 4 months ago

  • Target version set to Development 2025-12-10
  • Assigned To set to Lisa Knox
Actions
Copy link
 #4

Updated by Brett Smith 4 months ago

  • Subtask #23341 added
Actions
Copy link
 #5

Updated by Lisa Knox 4 months ago

  • Status changed from New to In Progress
Actions
Copy link
 #6

Updated by Brett Smith 3 months ago

  • Release set to 83
  • Target version deleted (Development 2025-12-10)

We will defer this until after #22381.

Actions
Copy link
 #7

Updated by Brett Smith 3 months ago

Actions
Copy link
 #8

Updated by Brett Smith 2 months ago

  • Target version set to Development 2026-02-04
Actions
Copy link
 #9

Updated by Brett Smith about 2 months ago

  • Target version changed from Development 2026-02-04 to Development 2026-02-18
Actions
Copy link
 #10

Updated by Brett Smith about 1 month ago

  • Target version changed from Development 2026-02-18 to Development 2026-03-04
Actions
Copy link
 #11

Updated by Brett Smith 20 days ago

  • Target version changed from Development 2026-03-04 to Development 2026-03-18
Actions
Copy link
 #12

Updated by Lisa Knox 6 days ago

Proposed solution is to upgrade MUI to take advantage of the virtualized list support in later MUI versions. The Redux-form dependency is a hard blocker for this, and is being addressed in #22381. The CircularSuspense component created bin the course of #22381 should be updated/replaced once the MUI upgrade happens.

Actions
Copy link
 #13

Updated by Lisa Knox 6 days ago

  • Status changed from In Progress to New
Actions
Copy link

Also available in: Atom PDF