Closed
Bug 1482692
Opened 6 years ago
Closed 6 years ago
Hit testing on fixed items is wrong when zoomed in
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 1465616
Tracking | Status | |
---|---|---|
firefox61 | --- | unaffected |
firefox62 | --- | unaffected |
firefox63 | --- | fixed |
People
(Reporter: kats, Unassigned)
References
Details
(Keywords: regression)
[Tracking Requested - why for this release]: usability/correctness regression
Go to a page with a fixed position element, e.g. a searchfox page. Zoom in. Then tap on the search field in the fixed header.
Expected: search field gets focus
Actual: tap ends up hitting whatever is behind the fixed header at that point.
Comment 1•6 years ago
|
||
Just guessing, but might the position:fixed element changes from bug 1465616 (or more precisely one of its dependencies, since bug 1465616 has been backed out again) have caused this?
Reporter | ||
Comment 2•6 years ago
|
||
Quite likely, but it's not easy for me to verify at the moment because I need my laptop to run mozregression and doing that is incompatible with holding a baby that alternates between crying and sleeping :)
Updated•6 years ago
|
Comment 3•6 years ago
|
||
Bugs we can fix, babies we don't want to break. Can someone take this over for you?
Flags: needinfo?(kats)
Reporter | ||
Comment 4•6 years ago
|
||
If somebody (QA/softvision/anybody) can run mozregression on it we can probably back out whatever caused it. Or at least see if whoever landed the regressing patch can look into fixing it.
Flags: needinfo?(kats)
Comment 5•6 years ago
|
||
Sounds similar to bug 1478614, which would make bug 1423011 the regressor.
Comment 6•6 years ago
|
||
(In reply to Ian Moody [:Kwan] (UTC+0) from comment #5)
> Sounds similar to bug 1478614, which would make bug 1423011 the regressor.
Kashav, does that sound plausible?
Ioana, can your team confirm this with mozregression?
Flags: needinfo?(kshvmdn)
Flags: needinfo?(ioana.chiorean)
Comment 7•6 years ago
|
||
Yes, sure. Bogdan, can you look at it?
Flags: needinfo?(ioana.chiorean) → needinfo?(bogdan.surd)
Comment 8•6 years ago
|
||
Sorry for the spam, just realized after i pressed save changes that Bogdan is out. Oana?
Flags: needinfo?(bogdan.surd) → needinfo?(oana.horvath)
Comment 9•6 years ago
|
||
Hello,
Confirmed with mozregression that Bug 1423011 is the cause.
First bad build: 2018-07-12
Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=085cdfb90903d4985f0de1dc7786522d9fb45596&tochange=fe17acc6e291e54463db3ea82697c714ae5a4b27
DEBUG : Found commit message:
Bug 1423011 - Part 4: Fix for overflow-x:hidden test failures in the mochitest test_wheel_default_action.html. r=botond
Flags: needinfo?(oana.horvath)
Keywords: regressionwindow-wanted
Comment 10•6 years ago
|
||
I can't reproduce this in the latest Fennec nightly. I believe it was fixed by bug 1465616.
Kats (or anyone else), please feel free to reopen if you're still seeing this in a recent nightly.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(kshvmdn)
Resolution: --- → WORKSFORME
Comment 11•6 years ago
|
||
(I guess DUPLICATE is the more appropriate resolution.)
Resolution: WORKSFORME → DUPLICATE
Updated•6 years ago
|
tracking-firefox63:
+ → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•