window.innerHeight changes with the dynamic toolbar on Firefox
Categories
(Core :: Layout, enhancement, P3)
Tracking
()
People
(Reporter: karlcow, Assigned: hiro)
References
(Blocks 1 open bug, )
Details
(Whiteboard: [webcompat] [geckoview:p3])
Attachments
(2 files)
spin off of https://webcompat.com/issues/20655
- with Firefox Android
- Go to http://mattdavella.com/
- scroll
Actual:
when the URL toolbar disappears, the background image stutters.
The reason is that the parallax JavaScript on the site is dependent on the window.innerHeight value.
see the details in https://webcompat.com/issues/20655
On Chrome the value of window.innerHeight doesn't change if the URL bar appears/disappears.
Opening this bug to track if there are other issues of this type for webcompat.
We may decide that it's not a big enough issue.
Assignee | ||
Comment 2•6 years ago
|
||
(In reply to Karl Dubostπ‘ :karlcow from comment #0)
spin off of https://webcompat.com/issues/20655
- with Firefox Android
- Go to http://mattdavella.com/
- scroll
Actual:
when the URL toolbar disappears, the background image stutters.The reason is that the parallax JavaScript on the site is dependent on the window.innerHeight value.
see the details in https://webcompat.com/issues/20655
On Chrome the value of window.innerHeight doesn't change if the URL bar appears/disappears.
That's probably not because of window.innerHeight. As far as I can tell both of Chrome and Fennec change window.innerHeight when the URL bar is going to be hidden. I've confirmed the value on http://bokand.github.io/demo/urlbarsize.html
I think the issue is caused by the difference of documentElement.clientHeight.
Comment 3•5 years ago
|
||
Migrating Webcompat whiteboard priorities to project flags. See bug 1547409.
Updated•5 years ago
|
Assignee | ||
Comment 5•5 years ago
|
||
Karl, do you have an archived version of the site in question? The site has been changed since this bug was filed.
![]() |
Reporter | |
Comment 6•5 years ago
|
||
Hiroyuki,
unfortunately, one of these cases, where keeping an archived version would have been good. sorry.
Assignee | ||
Comment 7•5 years ago
|
||
Never mind, then. (It's not your fault at all.) Hope that this will be fixed by bug 1514429, if it's not, will be fixed by bug 1515980.
Assignee | ||
Comment 8•5 years ago
|
||
Copying from bug 1552608 comment 57:
Indeed, I may be wrong here. If Chrome's layout view height matches window.innerHeight (I am also saying scale=1 case),
the layout viewport height is excluding the toolbar height in the case where the toolbar is completely shown
the layout viewport height is excluding the toolbar height as if the toolbar is completely shown in the case of the toolbar transition
the layout viewport height is whole visible area's height in the case where the toolbar is hidden
At least for window.innerHeight this is what Chrome does, so it looks the visual viewport height exeeds the layout viewport height during the toolbar transition, but I haven't actually checked.
Updated•5 years ago
|
Assignee | ||
Comment 9•5 years ago
|
||
I am going to reuse this bug to change window.innerHeight when the dynamic toolbar is completely hidden or starts to be visible, and fire a window resize event at that time.
window.innerHeight on Chrome
- is not including the dynamic toolbar height when the dynamic toolbar is visible (regardless of its height)
- is including the area which was covered by the dynamic toolbar when the dynamic toolbar is completely hidden
Assignee | ||
Comment 10•5 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=79d0f1921372ff0f6eb677402362327996079899
DynamicToolbarTest.kt failed in the try, but it should pass once after bug 1514429 is fixed.
Assignee | ||
Comment 11•5 years ago
|
||
The fix in the try is totally wrong. window.innerHeight should be the layout viewport height.
Assignee | ||
Comment 12•5 years ago
|
||
(In reply to Hiroyuki Ikezoe (:hiro) from comment #11)
The fix in the try is totally wrong. window.innerHeight should be the layout viewport height.
I am going to defer this issue into bug 1598487.
Assignee | ||
Comment 13•5 years ago
|
||
Comment 15•5 years ago
|
||
Pushed by hikezoe.birchill@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/87bbf1af61ac Fire a window resize event at the same time we change position:fixed %-units style values. r=botond https://hg.mozilla.org/integration/autoland/rev/a10194fbd502 window.innerHeight returns the size for vh units when the dynamic toolbar is completely hidden. r=botond
Comment 16•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/87bbf1af61ac
https://hg.mozilla.org/mozilla-central/rev/a10194fbd502
Updated•5 years ago
|
Description
•