Clicking is broken in position:fixed elements in Fennec with MOZ_ENABLE_FIXED_POSITION_LAYERS enabled

VERIFIED FIXED in Firefox 8

Status

Fennec Graveyard
General
--
major
VERIFIED FIXED
6 years ago
3 years ago

People

(Reporter: Cork, Assigned: romaxa)

Tracking

(Depends on: 1 bug, {mobile, regression})

Trunk
Firefox 8
mobile, regression
Dependency tree / graph

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
With the current nightly i can't focus the password or email field. This worked fine last i tested (back in dec).

Steps to Reproduce:
1. Open the URL
2. Click "Account login" at the top right
3. Try to set focus in the password by tapping the field

Actual Results:
Nothing happens when the field is clicked.

Expected Results:
Focus should be moved to the input field,
tracking-fennec: --- → ?
Keywords: regression, regressionwindow-wanted
This is likely a regression from bug 607417.  This bug does not appear in Firefox 4, but it does appear on Aurora (Firefox 5) and Nightly (Firefox 6).
Blocks: 607417
status2.0: --- → unaffected
status-firefox5: --- → affected
tracking-firefox5: --- → ?
Severity: normal → major
Keywords: mobile
OS: Android → All
Hardware: ARM → All
Summary: Can't focus input elements in a form with position: fixed → Clicking is broken in position:fixed elements in Fennec
Created attachment 531497 [details]
test case

This affects more than just form fields.  Clicks are not dispatched correctly to position:fixed elements in Fennec.  Steps to reproduce:

1. Open the attached test case in Fennec.
2. Zoom in, then scroll down.
3. Click on "Click me."

Expected results: The link is activated, and navigates to example.com.
Actual results: Nothing happens.
Another good reason to turn off position: fixed layers for now.
(In reply to comment #3)
> Another good reason to turn off position: fixed layers for now.

Filed bug 656167.
(Assignee)

Comment 5

6 years ago
It looks like events are not properly redirected to content with fixed position...
it is possible to enter/activate input fields if your page is scrolled to (0, y) position, and if X!=0, then events are redirected somewhere else..., also page automatically centered if you click on input field (X!=0) and in order to switch to password field you need to scroll page back to X=0 position and then click on password field..
do we have any magic in events coordinates translation code when they are sent from Parent to Child? scroll offsets? 

also interesting that different Y- position does not break this behavior.
We'd like to get this behavior into platform. See bug 652240.
tracking-firefox5: ? → ---
tracking-fennec: ? → 6+
seems to have broken in 4/9, 4/8 works.

Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.2a1pre) gecko/20110409 Firefox/4.2a1pre Fennec/4.1a1pre
Mac OS X 10.6.7
Depends on: 652240
tracking-fennec: 6+ → 7+
fixed by bug 656167, but we hope to turn position:fixed back on, so leaving this open.

Comment 9

6 years ago
I also tried to reproduce this following the steps from Comment 2 and it's not happening. Should we mark this as WORKSFORME?

Mozilla /5.0 (Android;Linux armv7l;rv:8.0a1) Gecko/20110809 Firefox/8.0a1 Fennec/8.0a1
Device: LG Optimus 2X (Android 2.2)
This works for me as well using a current Nightly. Cork reopen if you can repro.
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → WORKSFORME
Keywords: regressionwindow-wanted
(In reply to Kevin Brosnan [:kbrosnan] from comment #10)
> This works for me as well using a current Nightly. Cork reopen if you can
> repro.

see #comment8
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Summary: Clicking is broken in position:fixed elements in Fennec → Clicking is broken in position:fixed elements in Fennec with MOZ_ENABLE_FIXED_POSITION_LAYERS enabled
Status: REOPENED → NEW
(Assignee)

Comment 12

6 years ago
Created attachment 552207 [details] [diff] [review]
Fix click for fixed layers after zoom

Ok, so link clicking broken after zoom, but works again after little scroll, which make it clear that problem is in missing updateCSSViewport after zoom animation finished...
Assignee: nobody → romaxa
Status: NEW → ASSIGNED
Attachment #552207 - Flags: review?(ben)
Attachment #552207 - Flags: review?(ben) → review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed
oleg, land on m-i?
status-firefox7: --- → fixed
http://hg.mozilla.org/integration/mozilla-inbound/rev/0324f71c6757

Updated

6 years ago
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/0324f71c6757
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 8
I wasn't able to reproduce this issue on Mozilla/5.0 (Android;Linux armv7l;rv:7.0)Gecko/20110830 Firefox/9.0a1 Fennec/7.0
Device: HTC Desire
OS: Android 2.2
Status: RESOLVED → VERIFIED
No longer depends on: 652240
Depends on: 652240
You need to log in before you can comment on or make changes to this bug.