Summary: | [Qt][EFL][WK2] Resizing the window doesn't always result in right content position | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kenneth Rohde Christiansen <kenneth> | ||||||
Component: | WebKit EFL | Assignee: | Kenneth Rohde Christiansen <kenneth> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | dongseong.hwang, joone, lucas.de.marchi, noam, yael | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 103105 | ||||||||
Attachments: |
|
Description
Kenneth Rohde Christiansen
2012-12-07 16:34:02 PST
Adding Qt to the title. I see the same problem in Qt's MiniBrowser. Swapping the order of translate and scale in EwkViewImpl::transformFromScene, EwkViewImpl::displayTimerFired and not scaling EwkViewImpl::m_pagePosition in PageViewportControllerClientEfl::setViewportPosition is fixing the issue for EFL. The order of translate and scale was swapped in r136000. I will upload a proper patch tomorrow, after better understanding the implications of this swap. Resetting assignee. Kenneth is working on this. Created attachment 178565 [details]
Patch
This fixes it, but it would be nice to have it tested with Qt to make sure it doesn't affect pinch zooming etc.
Great! Qt runs well with this patch. I tested in your scrolling test: http://www.dorothybrowser.com/test/webkitTest/css3/fixedelement.html However, I can not test pinch zoom. I just resized qt window. Created attachment 178771 [details]
Patch
Landed in 137304 |