Summary: | Move graphs for resources under #resources-container-content | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Anthony Ricaud <rik> | ||||||||||
Component: | Web Inspector (Deprecated) | Assignee: | Nobody <webkit-unassigned> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | aroben, timothy | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | All | ||||||||||||
OS: | All | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 14378 | ||||||||||||
Attachments: |
|
Description
Anthony Ricaud
2008-08-27 18:49:27 PDT
Created attachment 23048 [details]
Work in progress patch
This seems to behave exactly the same. Except one big problem : sorting the graphs like the resources...
Is updateGraphSideWidth needed any more? Can it just be in the CSS now with a 100% width? That would eliminate a loop through all resources when resizing. Created attachment 23381 [details]
Working patch
This patch is doing the job. But I have to look for obsolete CSS rules, JS code and better variable names.
Created attachment 23385 [details]
Working better
Forgot to clean the graphs after a refresh.
Created attachment 23419 [details]
Proposed patch
Removed a CSS rule, renamed WebInspector.ResourceGraphElement to WebInspector.ResourceGraph and renamed _graphSideElement to _graphElement.
Added my copyright too.
Comment on attachment 23419 [details]
Proposed patch
Nice work!
|