Bug 168522

Summary: Web Inspector: Zoom level UI out of sync with setting value
Product: WebKit Reporter: Matt Baker <mattbaker>
Component: Web InspectorAssignee: Devin Rousso <hi>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, hi, inspector-bugzilla-changes
Priority: P2    
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
Attachments:
Description Flags
[Image] bug
none
Patch none

Matt Baker
Reported 2017-02-17 09:58:34 PST
Created attachment 301949 [details] [Image] bug Summary: Zoom level UI out of sync with setting value. Steps to Reproduce: 1. Open Inspector > Settings 2. Change Zoom to 200% => Zoom changes, but UI still shows 100% Note: Correct value is shown after refreshing the page
Attachments
[Image] bug (151.69 KB, image/png)
2017-02-17 09:58 PST, Matt Baker 		no flags
Details
Patch (2.02 KB, patch)
2017-02-17 13:49 PST, Devin Rousso 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Devin Rousso
Comment 1 2017-02-17 13:44:58 PST
I think this is partially working after <https://webkit.org/b/168222>, but one aspect that is still missing is when the user uses ⌘+ or ⌘- when on the Settings tab.
Devin Rousso
Comment 2 2017-02-17 13:49:20 PST
Created attachment 301982 [details] Patch
Blaze Burg
Comment 3 2017-02-17 13:56:34 PST
Comment on attachment 301982 [details] Patch r=me
WebKit Commit Bot
Comment 4 2017-02-17 14:49:05 PST
Comment on attachment 301982 [details] Patch Clearing flags on attachment: 301982 Committed r212580: <http://trac.webkit.org/changeset/212580>
WebKit Commit Bot
Comment 5 2017-02-17 14:49:09 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.