renderer: Apply initial viewport scale to page_zoom.#39067
renderer: Apply initial viewport scale to page_zoom.#39067shubhamg13 wants to merge 2 commits intoservo:mainfrom
page_zoom.#39067Conversation
Signed-off-by: Shubham Gupta <[email protected]>
shubhamg13
force-pushed
the
fix_apply_viewport
branch
from
180332d to
8bfc3f8
Compare
|
🤖 Opened new upstream WPT pull request (web-platform-tests/wpt#54619) with upstreamable changes. |
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#54619). |
shubhamg13
force-pushed
the
fix_apply_viewport
branch
from
8bfc3f8 to
67d62d3
Compare
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#54619). |
|
✍ Updated existing upstream WPT pull request (web-platform-tests/wpt#54619) title and body. |
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#54619). |
shubhamg13
force-pushed
the
fix_apply_viewport
branch
from
5e60354 to
41a9da6
Compare
|
📝 Transplanted new upstreamable changes to existing upstream WPT pull request (web-platform-tests/wpt#54619). |
|
cc: @xiaochengh |
…lying. Signed-off-by: Shubham Gupta <[email protected]>
shubhamg13
force-pushed
the
fix_apply_viewport
branch
from
41a9da6 to
96f932c
Compare
|
🤖 This change no longer contains upstreamable changes to WPT; closed existing upstream pull request (web-platform-tests/wpt#54619). |
|
Closing this as the issue that it was meant to fix is fixed now by another PR. @shubhamg13 feel to reopen this if you still think it is applicable. Thanks! |
3 participants
Attached scenario in issue highlights that applying viewport scale needs reflow.
Hence applying it to
page_zoombecause we reflow forpage_zoom.Follow-Up: Ignore applying of viewport meta tag by default for desktop browsers.
Chromium and Firefox ignore in desktop browser, only enable in devtools. #39133
Testing: Manual Test Case using mobile size window dimensions.
Fixes: #39002