Bug 9778 – [Drosera] Organize the file resource pull-down by tab/window

• Status: CLOSED
• Resolution: FIXED
• Component: New Bugs

[Alice Liu]

the file resource popup menu is just a flat list, which can get confusing when you have multiple tabs.  It would be nice to organize this list with sublists for each window/tab.  

[Timothy Hatcher]

Great idea! I have been considering a sidebar with a tree of all the files, like Xcode.

[Vladimir Olexa (vladinecko)]

Can this bug be merged with Bug 9596? http://bugs.webkit.org/show_bug.cgi?id=9596
I'm currently working on both features at the same time and will be releasing them in one patch.

[Vladimir Olexa (vladinecko)]

Created attachment 11016 [details]
Patch

Ok, I added this functionality to Drosera. I only commented out (as opposed to removing) most of the code that was handling the files pop-up menu as I will be placing the function pop-up list there as mentioned in Bug 9596.

[Timothy Hatcher]

Comment on attachment 11016 [details]
Patch

Please attach a screenshot of what Drosera looks like with this change.

We will want to keep the file popup menu to be consistent with what Xcode does, it shows all open files in that menu. Right now Drosera has no way to close files, but it should (similar to bug 9895). Please make a new patch that keeps the file popup menu.

What was your change to Drosera/English.lproj/Debugger.nib?

[Timothy Hatcher]

I am very curious how this looks! Thanks for tackling it.

[Vladimir Olexa (vladinecko)]

Created attachment 11017 [details]
Screenshot with FileBrowser in place

[Timothy Hatcher]

Comment on attachment 11017 [details]
Screenshot with FileBrowser in place

Cool! I don't think we should wrap the long site addresses, they should have tooltips using the title attribute.

We should also use overflow-y: auto so the scrollbar gutter hides when it isn't needed.

[Vladimir Olexa (vladinecko)]

Created attachment 11018 [details]
Patch Revised

- changed overflow from scroll to auto to hide the scrollbar when it's not needed
- changed site text from bold to normal
- site name no longer wraps 
- put the file pop-up button back

[Vladimir Olexa (vladinecko)]

Created attachment 11019 [details]
Screenshot with changes from the revised patch

[Timothy Hatcher]

Landed in r16984.

[Timothy Hatcher]

Closing since Drosera has been replaced by the new Web Inspector debugger. Moving to the New Bugs component so the Drosera component can be closed and removed.