Bug 207455 - [GTK][WebInspector] Do not make Web Inspector windows transient
Summary: [GTK][WebInspector] Do not make Web Inspector windows transient
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adrian Perez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-02-10 02:04 PST by Adrian Perez
Modified: 2020-02-10 03:21 PST (History)
7 users (show)

See Also:


Attachments
Patch (5.39 KB, patch)
2020-02-10 02:18 PST, Adrian Perez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian Perez 2020-02-10 02:04:51 PST
Currently when popping out the Web Inspector panel into its own top level
window, its “transient-for“ property is set to the window that contains
the WebKitWebView. This forces the Inspector window to be stacked always
on top of the window that contains the web view, making it impossible
to bring the window with the web view to the front. This can be an issue
in small screens where there is not enough space to comfortable arrange
both windows in some way that both can be seen at the same time (e.g.
side-by-side).
Comment 1 Adrian Perez 2020-02-10 02:18:22 PST
Created attachment 390237 [details]
Patch
Comment 2 WebKit Commit Bot 2020-02-10 03:21:10 PST
The commit-queue encountered the following flaky tests while processing attachment 390237 [details]:

editing/spelling/spellcheck-async-remove-frame.html bug 158401 (authors: morrita@google.com, rniwa@webkit.org, and tony@chromium.org)
The commit-queue is continuing to process your patch.
Comment 3 WebKit Commit Bot 2020-02-10 03:21:42 PST
Comment on attachment 390237 [details]
Patch

Clearing flags on attachment: 390237

Committed r256114: <https://trac.webkit.org/changeset/256114>
Comment 4 WebKit Commit Bot 2020-02-10 03:21:44 PST
All reviewed patches have been landed.  Closing bug.