Bug 153756
| Summary: | Web Inspector: DataGridNode should support adding/removing a status element to any cell | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Matt Baker <mattbaker> |
| Component: | Web Inspector | Assignee: | Matt Baker <mattbaker> |
| Status: | NEW | ||
| Severity: | Normal | CC: | graouts, inspector-bugzilla-changes, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | |||
| Bug Blocks: | 153032 | ||
Matt Baker
* 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.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/24444108>
Joseph Pecoraro
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?
Matt Baker
(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).