Bug 207455

Summary: [GTK][WebInspector] Do not make Web Inspector windows transient
Product: WebKit Reporter: Adrian Perez <aperez>
Component: WebKitGTKAssignee: Adrian Perez <aperez>
Status: RESOLVED FIXED    
Severity: Normal CC: bugs-noreply, cgarcia, commit-queue, hi, jonlee, timothy, zan
Priority: P2    
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Adrian Perez
Reported 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).
Attachments
Patch (5.39 KB, patch)
2020-02-10 02:18 PST, Adrian Perez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adrian Perez
Comment 1 2020-02-10 02:18:22 PST
Created attachment 390237 [details] Patch
WebKit Commit Bot
Comment 2 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.
WebKit Commit Bot
Comment 3 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>
WebKit Commit Bot
Comment 4 2020-02-10 03:21:44 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.