Bug 75421

Summary: set height miscalculation when zoomed
Product: WebKit Reporter: Micah Bucy <eternalsword>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: UNCONFIRMED    
Severity: Normal CC: markus.magnuson, simon.fraser
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
URL: http://eternalsword.com/webkitbug/zoom-set-height-test.html
Attachments:
Description Flags
Patch none

Micah Bucy
Reported 2012-01-01 14:39:33 PST
This may be related to bug #47611 or could be a jquery bug as it is using jquery's .height(newHeight) method. Tested on: chromium 15.0.874.121 on gentoo. chrome 16.0.912.63 on mac os x 10.7 lion. safari 5.1.2 (7534.52.7) on mac os x 10.7 lion. will have access to windows machine on Jan 3, 2012 for further testing. When zoomed in (not pinch zoomed in Safari) like via Command++/Ctrl++, jquery set height does not set the height correctly. See http://eternalsword.com/webkitbug/zoom-set-height-test.html for testing and how the test works. Problem does not seem to be exhibited when zoom factor is < 100%, only > 100%. Has not been tested on any version of msie, but the test works properly on these tested browsers: firefox 10beta2 on mac os x 10.7 lion opera 9.5 on mac os x 10.7 lion opera 11.60 on mac os x 10.7 lion opera 11.52 on gentoo firefox 9 on gentoo
Attachments
Patch (1.96 KB, patch)
2012-01-17 06:58 PST, Marc-André Decoste 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Micah Bucy
Comment 1 2012-01-01 20:50:36 PST
tested on webkit nightly as well on mac os x 10.7 lion and the issue is still exhibited. I have a workaround method for any developers frustrated by this that can be located at http://eternalsword.com/webkitbug/zoom-set-height-workaround.html
Marc-André Decoste
Comment 2 2012-01-17 06:58:11 PST
Created attachment 122762 [details] Patch
Marc-André Decoste
Comment 3 2012-01-17 07:18:58 PST
Oops, typo in bug number, I was aiming at 76421, sorry... :-/
Simon Fraser (smfr)
Comment 4 2017-05-29 08:50:40 PDT
Testcase is no longer available.
Note You need to log in before you can comment on or make changes to this bug.