Created attachment 153170 [details] Patch

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

Created attachment 153224 [details] Patch

(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.

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.

Created attachment 153438 [details] Screenshot

(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.

> 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 on attachment 153224 [details] Patch Clearing r? for now per Pavel's comments.

Created attachment 158997 [details] Whiteboard prototype, ResourcesPanel to FileSystemView

Created attachment 158998 [details] Whiteboard prototype, FileSystemView to {File,Directory}ContentView

Created attachment 159000 [details] Whiteboard prototype, FileSystemVie

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?

Created attachment 159008 [details] Screenshot +another screenshot of local build. Adding crumbs like this.

(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.

(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.

This feature is no longer in WebKit.