Bug 91709 - Web Inspector: Enable FileSystem support by default
Summary: Web Inspector: Enable FileSystem support by default
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Taiju Tsuiki
URL:
Keywords:
Depends on: 91831 94301 94908 97524
Blocks: 68203
  Show dependency treegraph
 
Reported: 2012-07-18 20:59 PDT by Taiju Tsuiki
Modified: 2014-08-03 18:31 PDT (History)
12 users (show)

See Also:


Attachments
Patch (2.74 KB, patch)
2012-07-18 21:01 PDT, Taiju Tsuiki
no flags Details | Formatted Diff | Diff
Patch (4.77 KB, patch)
2012-07-19 04:24 PDT, Taiju Tsuiki
no flags Details | Formatted Diff | Diff
Screenshot (241.00 KB, image/png)
2012-07-20 00:53 PDT, Taiju Tsuiki
no flags Details
Whiteboard prototype, ResourcesPanel to FileSystemView (244.62 KB, image/jpeg)
2012-08-16 22:26 PDT, Taiju Tsuiki
no flags Details
Whiteboard prototype, FileSystemView to {File,Directory}ContentView (261.71 KB, image/jpeg)
2012-08-16 22:27 PDT, Taiju Tsuiki
no flags Details
Whiteboard prototype, FileSystemVie (246.81 KB, image/jpeg)
2012-08-16 22:28 PDT, Taiju Tsuiki
no flags Details
Screenshot (49.23 KB, image/png)
2012-08-16 23:05 PDT, Taiju Tsuiki
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Taiju Tsuiki 2012-07-18 20:59:24 PDT
All patch for adding basic support of FileSystem has landed, and it seems to work well for me.
So, may I drop its experiments flag and enable FileSystem support by default?
Comment 1 Taiju Tsuiki 2012-07-18 21:01:40 PDT
Created attachment 153170 [details]
Patch
Comment 2 Vsevolod Vlasov 2012-07-19 02:46:44 PDT
Comment on attachment 153170 [details]
Patch

We should remove this from Preferences, there is no need to disable file system inspection when it is supported. We should use Capabilities instead on platforms where file system is not supported yet.
Please refer to the earlier Pavel's comment https://bugs.webkit.org/show_bug.cgi?id=72691#c19
Comment 3 Taiju Tsuiki 2012-07-19 04:24:39 PDT
Created attachment 153224 [details]
Patch
Comment 4 Taiju Tsuiki 2012-07-19 04:39:17 PDT
(In reply to comment #2)
> (From update of attachment 153170 [details])
> We should remove this from Preferences, there is no need to disable file system inspection when it is supported. We should use Capabilities instead on platforms where file system is not supported yet.
> Please refer to the earlier Pavel's comment https://bugs.webkit.org/show_bug.cgi?id=72691#c19

I see. I erased Preference.exposeFileSystemInspection and added Capabilities.fileSystemAPISupport.
Comment 5 Pavel Feldman 2012-07-19 05:03:46 PDT
There are changes I'd like to see applied before it leaves experimental:

1) The tree as of today does not scale: it has no scrollers, it would not scale for large trees even with scrollers

I'd suggest that the tree is embedded into the table (a-la Finder) or is converted into a list and embedded into the table (a-la Windows Explorer)

2) We can make preview show to the right from the tree-table for docked to bottom mode and in the bottom part of the screen for undocked / docked-to-right mode.

3) I think it is important to be able to format the file system and remove particular folder for the debugging purposes.
Comment 6 Taiju Tsuiki 2012-07-20 00:53:41 PDT
Created attachment 153438 [details]
Screenshot
Comment 7 Taiju Tsuiki 2012-07-20 01:03:14 PDT
(In reply to comment #5)
> There are changes I'd like to see applied before it leaves experimental:
> 
> 1) The tree as of today does not scale: it has no scrollers, it would not scale for large trees even with scrollers
> 
> I'd suggest that the tree is embedded into the table (a-la Finder) or is converted into a list and embedded into the table (a-la Windows Explorer)
> 

Does adding some navigator help to walk on large tree?
Such as "Navigate to parent/ancestor" button on status bar and navigation by double-clicking on DirectoryContentView.
It is like my favorite Windows Explorer configuration.

> 2) We can make preview show to the right from the tree-table for docked to bottom mode and in the bottom part of the screen for undocked / docked-to-right mode.
> 

Is FileContentView too narrow horizontally for undocked mode?
How about adding a button to hide directory tree?
It will widen {File,Directory}ContentView as same width as other resource preview.

> 3) I think it is important to be able to format the file system and remove particular folder for the debugging purposes.

I agree it is important for debugging, but I thought it can be done incrementally.
OK for rising its priority.
Comment 8 Pavel Feldman 2012-07-20 01:52:30 PDT
> Does adding some navigator help to walk on large tree?
> Such as "Navigate to parent/ancestor" button on status bar and navigation by double-clicking on DirectoryContentView.
> It is like my favorite Windows Explorer configuration.

It would be great to see the crumbs (a-la Windows Explorer or bottom of Elements panel).

> Is FileContentView too narrow horizontally for undocked mode?
> How about adding a button to hide directory tree?
> It will widen {File,Directory}ContentView as same width as other resource preview.
> 

I don't think we need a tree. Windows Explorer does not have it.

> I agree it is important for debugging, but I thought it can be done incrementally.
> OK for rising its priority.

Incrementally sounds good, but we should not go out of experimental until basic features are there. I think these two are the most important.
Comment 9 Vsevolod Vlasov 2012-08-03 00:29:08 PDT
Comment on attachment 153224 [details]
Patch

Clearing r? for now per Pavel's comments.
Comment 10 Taiju Tsuiki 2012-08-16 22:26:15 PDT
Created attachment 158997 [details]
Whiteboard prototype, ResourcesPanel to FileSystemView
Comment 11 Taiju Tsuiki 2012-08-16 22:27:12 PDT
Created attachment 158998 [details]
Whiteboard prototype, FileSystemView to {File,Directory}ContentView
Comment 12 Taiju Tsuiki 2012-08-16 22:28:16 PDT
Created attachment 159000 [details]
Whiteboard prototype, FileSystemVie
Comment 13 Taiju Tsuiki 2012-08-16 23:00:06 PDT
I'm refining UI for FileSystemView as attached whiteboard prototype.
Diffs to current version are:
 - Adding "Delete" button,
 - Removing directory tree, so that {File,Directory}ContentView is goint to be main view of FileSystemView,
 - Adding Breadcrumb list to status bar.

Any suggestion at this stage?
Comment 14 Taiju Tsuiki 2012-08-16 23:05:43 PDT
Created attachment 159008 [details]
Screenshot

+another screenshot of local build.
Adding crumbs like this.
Comment 15 Yury Semikhatsky 2012-08-21 23:11:17 PDT
(In reply to comment #14)
> Created an attachment (id=159008) [details]
> Screenshot
> 
> +another screenshot of local build.
> Adding crumbs like this.

I'm a bit confused as the screenshot contains separate directory tree which is missing on the whiteboard. I'm personally for removing separate directory tree and making the directory view more like in Finder on Mac OS X or Nautilus in Gnome. You can always step up using the crumbs and step down by either expanding a directory or stepping into it via double-click. The separate tree just clutters the UI with duplicate data.
Comment 16 Taiju Tsuiki 2012-08-22 00:20:00 PDT
(In reply to comment #15)
> (In reply to comment #14)
> > Created an attachment (id=159008) [details] [details]
> > Screenshot
> > 
> > +another screenshot of local build.
> > Adding crumbs like this.
> 
> I'm a bit confused as the screenshot contains separate directory tree which is missing on the whiteboard. I'm personally for removing separate directory tree and making the directory view more like in Finder on Mac OS X or Nautilus in Gnome. You can always step up using the crumbs and step down by either expanding a directory or stepping into it via double-click. The separate tree just clutters the UI with duplicate data.

Ah, sorry for confusing screenshot.
I'm implementing it like whiteboard sketch without directory tree but not yet removed directory tree. It'll be removed soon later.
I meant to show how crumbs looks like, by the screenshot.
Comment 17 Brian Burg 2014-08-03 18:31:46 PDT
This feature is no longer in WebKit.