Bug 161015

Summary: Web Inspector: DataGrid: Remove cells of hidden columns from DOM
Product: WebKit Reporter: Nikita Vasilyev <nvasilyev>
Component: Web InspectorAssignee: Nikita Vasilyev <nvasilyev>
Status: ASSIGNED    
Severity: Normal CC: inspector-bugzilla-changes, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
See Also: https://bugs.webkit.org/show_bug.cgi?id=160723
Attachments:
Description Flags
WIP nvasilyev: review-, nvasilyev: commit-queue-

Nikita Vasilyev
Reported 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.
Attachments
WIP (15.54 KB, patch)
2016-08-19 14:31 PDT, Nikita Vasilyev 		nvasilyev: review-
nvasilyev: commit-queue-
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2016-08-19 14:27:28 PDT
<rdar://problem/27930252>
Nikita Vasilyev
Comment 2 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.
Note You need to log in before you can comment on or make changes to this bug.