Bug 207228 - Web Inspector: the height of the undocked title area shouldn't change when zoomed
Summary: Web Inspector: the height of the undocked title area shouldn't change when zo...
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: WebKit Local Build
Hardware: All All
: P2 Normal
Assignee: Devin Rousso
Keywords: InRadar
Depends on: 204627
  Show dependency treegraph
Reported: 2020-02-04 14:25 PST by Devin Rousso
Modified: 2020-03-02 16:54 PST (History)
7 users (show)

See Also:

Patch (2.81 KB, patch)
2020-02-04 14:28 PST, Devin Rousso
no flags Details | Formatted Diff | Diff
Patch (3.17 KB, patch)
2020-02-05 20:10 PST, Devin Rousso
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Devin Rousso 2020-02-04 14:25:56 PST
1. inspect any page
2. detach web inspector
3. zoom in/out (⌘+ or ⌘-)
 => the height of the title area increases
Comment 1 Devin Rousso 2020-02-04 14:28:36 PST
Created attachment 389709 [details]
Comment 2 Timothy Hatcher 2020-02-05 17:30:11 PST
Comment on attachment 389709 [details]

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

> Source/WebInspectorUI/UserInterface/Base/Main.js:548
> +    }
> +    WI.settings.zoomFactor.addEventListener(WI.Setting.Event.Changed, updateZoomFactorCSSVariable);

Blank line between. Or move the function inline as a arrow function in addEventListener.
Comment 3 Devin Rousso 2020-02-05 20:10:24 PST
Created attachment 389933 [details]
Comment 4 WebKit Commit Bot 2020-02-05 21:06:39 PST
The commit-queue encountered the following flaky tests while processing attachment 389933 [details]:

editing/spelling/spellcheck-async-remove-frame.html bug 158401 (authors: morrita@google.com, rniwa@webkit.org, and tony@chromium.org)
imported/w3c/web-platform-tests/IndexedDB/fire-success-event-exception.html bug 206554 (authors: shvaikalesh@gmail.com and youennf@gmail.com)
The commit-queue is continuing to process your patch.
Comment 5 WebKit Commit Bot 2020-02-05 21:07:17 PST
Comment on attachment 389933 [details]

Clearing flags on attachment: 389933

Committed r255901: <https://trac.webkit.org/changeset/255901>
Comment 6 WebKit Commit Bot 2020-02-05 21:07:19 PST
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2020-02-05 21:08:14 PST
Comment 8 Jon Lee 2020-02-07 20:23:27 PST
Reopening. This was reverted by https://trac.webkit.org/changeset/256086/webkit in b207422.
Comment 9 Devin Rousso 2020-03-02 16:54:45 PST
This will be resolved by <https://webkit.org/b/207317>.