Can't dismiss instagram's "use the app" banner on Fennec / GeckoView

NEW
Unassigned

Status

()

defect
P3
normal
4 months ago
2 months ago

People

(Reporter: TYLin, Unassigned)

Tracking

unspecified
Unspecified
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 wontfix, firefox67 ?, firefox68 affected)

Details

(Whiteboard: [bcs:p2] [geckoview:fenix:p2][See comment 2 for workaround])

+++ This bug was initially created as a clone of Bug #1516963 +++

STR:

Expected:

You can dismiss it.

Actual:

You can't easily dismiss the banner unless clicking it multiple times, or force zoom-in the page to make the X larger enough to click.

I fixed bug 1516963 which makes AccessibleCaret doesn't show when clicking on the X, but per bug 1516963 comment 12 the banner still cannot be dismissed easily.

This might somewhat be related to touch event handling or pan zoom. CC :kats and :botond.

Component: Selection → Panning and Zooming
Priority: P2 → P3

Does it work better if you set ui.mouse.radius.enabled to true?

Flags: needinfo?(aethanyc)

(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #1)

Does it work better if you set ui.mouse.radius.enabled to true?

Yes! It does. I can dismiss the Instagram banner easily with ui.mouse.radius.enabled=true.

Flags: needinfo?(aethanyc)

:snorp, any thoughts on re-enabling this pref? It's still the first thing I enable on any Gecko android browser since without it i find clicking on things basically impossible.

Flags: needinfo?(snorp)

Yeah I think we should revisit this. IIRC, the reason we disabled it was that some hamburger menus were impossible to select. Did we fix that problem?

Flags: needinfo?(snorp) → needinfo?(kats)

Randall, do you recall the exact details of why we disabled the touch radius stuff?

Flags: needinfo?(rbarker)

It's because it makes elements that are made clickable with js libraries like jquery become unclickable if there is a regular link object next to it. The touch radius always finds the regular link since the jquery link just looks like a div or such.

Flags: needinfo?(rbarker)

(In reply to Randall Barker [:rbarker] from comment #6)

It's because it makes elements that are made clickable with js libraries like jquery become unclickable if there is a regular link object next to it. The touch radius always finds the regular link since the jquery link just looks like a div or such.

Ah, and I think some frameworks always connect listeners to the document or something so we can't immediately see that there is a click handler. It's coming back now.

Yes,all the clicks/touches were being processed by the root so there wasn't an obvious way to know if a element was clickable. We thought about doing a two pass method where if the point failed to be processed, use the radius to test for a clickable element. But I could never get it to work.

FWIW I never run into this problem with the mouse radius enabled. But I'm constantly frustrated by not being able to tap on things with the mouse radius disabled.

Flags: needinfo?(kats)
OS: Unspecified → Android
Whiteboard: [geckoview:p2] → [geckoview:fenix:p2]
Whiteboard: [geckoview:fenix:p2] → [geckoview:fenix:p2][See comment 2 for workaround]
Duplicate of this bug: 1544055
Whiteboard: [geckoview:fenix:p2][See comment 2 for workaround] → [geckoview:fenix:p2][See comment 2 for workaround] [bcs?]

(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #9)

FWIW I never run into this problem with the mouse radius enabled. But I'm constantly frustrated by not being able to tap on things with the mouse radius disabled.

I tested this in Chrome and the Instagram banner is also difficult to dismiss in Chrome. However, Fennec/Fenix makes it even more difficult because the "SELECT ALL" context menu opens when your tap misses the X button.

Bug 1437372 and bug 1465743 describe related difficult-to-click-button problems in Fennec.

Randall disabled ui.mouse.radius.enabled in bug 1247095 three years ago to unbreak multiple broken sites: bug 1192558, bug 1233272, and bug 1235339.

So it seems like this bug is at a standstill: fixing in makes some links unclickable, but not fixing it makes some links difficult-but-not-impossible to click.

Depends on: 1247095
See Also: → 1437372
Whiteboard: [geckoview:fenix:p2][See comment 2 for workaround] [bcs?] → [bcs:p2] [geckoview:fenix:p2][See comment 2 for workaround]

Hi, I've retested this issue using the latest Nightly build (68.0a1 from the 6th of May) on a Google Pixel 3XL(Android 9)and the problem can still be replicated.
As per the attached video and by judging the cursor, the selection of the 'X' button has to be quite precise to be able to dismiss the banner.

Wouldn't this be more of an accessibility issue on Instagram's part? And so filing a bug with them, when resolved would fix the issue on all browsers?

You need to log in before you can comment on or make changes to this bug.