Bug 32453

Summary: Web Inspector: Replace split pane with tabbed pane in resource contents view.
Product: WebKit Reporter: Pavel Feldman <pfeldman>
Component: Web Inspector (Deprecated)Assignee: Pavel Feldman <pfeldman>
Severity: Normal CC: bweinstein, joepeck, rik, timothy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Description Flags
[IMAGE] Mock
[IMAGE] Patch renders it like this.
[PATCH] Proposed change. timothy: review+

Description Pavel Feldman 2009-12-11 14:59:41 PST
Created attachment 44711 [details]
[IMAGE] Mock

When resource is selected in the Resources Panel, it should show tabbed pane to the right with following tabs:
- HTTP Info
- Contents
(anything else?).

I'll need tabs design for this.
Comment 1 Pavel Feldman 2010-01-24 14:25:55 PST
Created attachment 47300 [details]
[IMAGE] Patch renders it like this.

I am doing this since it simplifies migration to SourceFrame2 (which only works well with absolute positioning). I'll submit a patch shortly that renders tabs like in this image. Good design for tabs is appreciated. Btw, at some point we should optimize it a way that we only load source when content tab is switched on.
Comment 2 Pavel Feldman 2010-01-24 14:28:54 PST
Created attachment 47301 [details]
[PATCH] Proposed change.
Comment 3 Timothy Hatcher 2010-01-24 14:45:34 PST
Comment on attachment 47301 [details]
[PATCH] Proposed change.

> +    this.tabsElement.className = "scope-bar";

I don't think reusing scope-bar is good. I'm not fond of the recessed tab look. But it is fine for now.
Comment 4 Pavel Feldman 2010-01-25 02:38:34 PST
Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/English.lproj/localizedStrings.js
	M	WebCore/inspector/front-end/ResourceView.js
	M	WebCore/inspector/front-end/ResourcesPanel.js
	M	WebCore/inspector/front-end/ScriptsPanel.js
	M	WebCore/inspector/front-end/Settings.js
	M	WebCore/inspector/front-end/SourceFrame.js
	M	WebCore/inspector/front-end/SourceView.js
	M	WebCore/inspector/front-end/inspector.css
Committed r53794