[GTK] The new web inspector can't be docked again once undocked

• Component: WebKitGTK
• Status: RESOLVED FIXED

[Carlos Garcia Campos]

The dock button in mac is in the window decoration, see:
https://developer.apple.com/technologies/safari/images/web-inspector.png

We should place the icon in the main toolbar to be able to dock the inspector again

[Gustavo Noronha (kov)]

How do you undock it, btw?

[Gustavo Noronha (kov)]

Looks like the only way to undock it atm is by having the webview that will be inspected be small enough that docking is not considered an option?

[Gustavo Noronha (kov)]

Actually, there is a little button, I was not seeing it because I had an 'attach to right' button instead, looks like pressing shift/alt withthe inspector view focused changes which button will be there.

[Gustavo Noronha (kov)]

Created attachment 216537 [details]
Patch

[Brent Fulgham]

Joe -- I think this patch would also help the Windows port (which also suffers from this problem).  Could you review please?

[Timothy Hatcher]

Comment on attachment 216537 [details]
Patch

This looks good as-is. Though it likely would be better to have one button that toggles when the alt/option key is held down (like undockButtonNavigationItem.) So the buttons do not take up more rooms.

[Joseph Pecoraro]

Comment on attachment 216537 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=216537&action=review

I agree with Timothy Hatcher's comment though. Is it conventional for the GTK port to toggle an alternate button when an "alt" key is down. Or would that be unconventional?

> Source/WebInspectorUI/UserInterface/Main.js:3
> + * Coprygith (C) 2013 Gustavo Noronha Silva <[e-mail]>

Typo: "Coprygith" => "Copyright"

> Source/WebInspectorUI/UserInterface/Main.js:224
> +    if (InspectorFrontendHost.platform() != "mac") {

Style: "==" => "==="

[Gustavo Noronha (kov)]

(In reply to comment #7)
> I agree with Timothy Hatcher's comment though. Is it conventional for the GTK port to toggle an alternate button when an "alt" key is down. Or would that be unconventional?

It is unconventional, it took me some time to figure out how to undock the inspector because I did not grasp how that worked at first.

In fact, one of the things I would like to do is to replace that behaviour in the docked state by showing both buttons, I can do the alt key thing work for the non-GTK case, though.

The other change I would like to do is to move the buttons (close in particular) to the right side of the toolbar where they make more sense for us. We'll be working at the WebKitGTK+ hackfest over the next few days, so I plan to take those on =)

[Carlos Garcia Campos]

Created attachment 239276 [details]
A different approach using GTK+

Use a header bar when building with a recent enough GTK+.

[Carlos Garcia Campos]

Created attachment 239277 [details]
Screenshot

This is how the inspector window looks with the header bar and dock buttons

[Carlos Garcia Campos]

Committed r174327: <http://trac.webkit.org/changeset/174327>

[Sergio Villar Senin]

Comment on attachment 239276 [details]
A different approach using GTK+

Clearing the r+ flag in order not to appear in the review queue.

[Sergio Villar Senin]

Comment on attachment 216537 [details]
Patch

Clearing the r? flag. This landed some time ago.