153756 2016-02-01 13:09:25 -0800 Web Inspector: DataGridNode should support adding/removing a status element to any cell 2026-01-12 09:04:25 -0800 1 1 1 Unclassified WebKit Web Inspector WebKit Nightly Build All All NEW InRadar P2 Normal --- 153032 1 mattbaker mattbaker graouts inspector-bugzilla-changes webkit-bug-importer oldest_to_newest 1160985 0 mattbaker 2016-02-01 13:09:25 -0800 * SUMMARY DataGridNode should support adding/removing a status element to any cell. As part of the Timelines UI redesign, grids will need cells with status elements, so the Network and Overview grids can display an IndeterminateProgressSpinner for loading resources. 1160986 1 webkit-bug-importer 2016-02-01 13:09:52 -0800 <rdar://problem/24444108> 1160988 2 joepeck 2016-02-01 13:11:50 -0800 Again, this sounds to me like something that should go inside a cell itself, instead of DataGridNode. We already have TimelineDataGridNode, perhaps these should be going in there? 1160999 3 mattbaker 2016-02-01 13:25:43 -0800 (In reply to comment #2) > Again, this sounds to me like something that should go inside a cell itself, > instead of DataGridNode. We already have TimelineDataGridNode, perhaps these > should be going in there? Not sure what you mean by placing it "in the cell itself", since we deal in grid nodes/rows, not cells. I like placing it in DataGridNode, since it feels like basic functionality which could have utility outside the timeline grids (even though TimelineDataGridNode is the only use case as of now).