Bug 161015 - Web Inspector: DataGrid: Remove cells of hidden columns from DOM
Summary: Web Inspector: DataGrid: Remove cells of hidden columns from DOM
Status: ASSIGNED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Nikita Vasilyev
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2016-08-19 14:27 PDT by Nikita Vasilyev
Modified: 2016-12-13 15:40 PST (History)
2 users (show)

See Also:

Attachments
WIP (15.54 KB, patch)
2016-08-19 14:31 PDT, Nikita Vasilyev 		nvasilyev: review-
nvasilyev: commit-queue-
Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Nikita Vasilyev 2016-08-19 14:27:12 PDT 
Currently, we hide columns by setting <col style="width: 0">.

Because of that we had issues with double borders, which was addressed by a CSS hack (bug 160723).

A better solution would be to remove columns from DOM, not just hide them. It would be better for performance, too. Currently, we update <td>-s even when they're part of a hidden column.
Comment 1 Radar WebKit Bug Importer 2016-08-19 14:27:28 PDT 
<rdar://problem/27930252>
Comment 2 Nikita Vasilyev 2016-08-19 14:31:14 PDT 
Created attachment 286480 [details]
WIP

I'm currently not working on this, as it isn't a high priority, but I'll keep my WIP here for future.