845902 - Unable to look at events on calendar

Status: RESOLVED INCOMPLETE

(Firefox for Android Graveyard :: General, defect)

Description

[Wesley Johnston (:wesj)]

Clicking on links in the calendar at doesn't seem to do anything:

http://dayonecenter.com/classes/calendar/san_francisco

On desktop it works fine and shows a detail for the event.

I see a few errors in the console like:

E/GeckoConsole(29798): [JavaScript Error: "TypeError: $(...).getElements is not a function" {file: "https://dayonecenter.com/js/mega/global.js" line: 308}]

E/GeckoConsole(29798): [JavaScript Error: "TypeError: $(...).getElements is not a function" {file: "https://dayonecenter.com/js/mega/global.js" line: 308}]

Neither occur when I'm clicking, but happen during page load. They also both show up on desktop.

Comment 1

[:Margaret Leibovic]

(In reply to Wesley Johnston (:wesj) from comment #0)
> Clicking on links in the calendar at doesn't seem to do anything:
> 
> http://dayonecenter.com/classes/calendar/san_francisco

I get a php error when I try to load this url.

Comment 2

[Wesley Johnston (:wesj)]

I do too now. Worked an hour ago...

Comment 3

[Wesley Johnston (:wesj)]

Site is back up. Looks like this is an iframe with a css transform set on it. Essentially getBoundingClientRect on elements inside that iframe doesn't take the transform into account (which makes sense to me, if you're drawing in that document, you don't want to know about your transform...), but we use getBoundingClientRect to try and wiggle touches onto the correct target. Working on a fix...

Comment 4

[Wesley Johnston (:wesj)]

Attachment: WIP Patch 1

This was my first take. Basically to get these transform and take them into account when fixing the event coordinates. It works. It just feels like way to much work. I'm also still not doing things like handling frames where the transform origin is something other than top-left.

I think what I'd rather do is try and look for a better fix to bug 710704 that doesn't use window.top and hence can ignore these transform problems? Or maybe just bail on touch redirection if the element is in an iframe at all...

Comment 5

[Wesley Johnston (:wesj)]

Attachment: Patch

This just doesn't do this redirection if we're inside an iframe that has a transform. This is a thousand times simpler and is less likely to break. But its turning off this feature for a (small) subset of pages (i.e. clicks will not work correctly all the time).

I think the simplicity is worth the tradeoff.

Comment 6

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 719262 [details] [diff] [review]
WIP Patch 1

Review of attachment 719262 [details] [diff] [review]:
-----------------------------------------------------------------

Yeah, I don't want to be implementing all of CSS transforms in browser.js

Comment 7

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 719276 [details] [diff] [review]
Patch

Review of attachment 719276 [details] [diff] [review]:
-----------------------------------------------------------------

We use getBoundingClientRect in a lot of places, though. This only addresses one possible code path. And unless I'm misunderstanding the problem, this will only "fix" it for small transforms, right? I mean we use getBoundingClientRect in elementFromPoint, which is used to determine the _highlightElement so even without the "wiggle" code the click will be sent to the wrong element. Minusing for now, but let me know if I'm off base here.

Comment 8

[Wesley Johnston (:wesj)]

I agree it would be good to fix this everywhere, but I don't think its trivial. Maybe we can get bug 788073 up and running instead.

Comment 9

[Wesley Johnston (:wesj)]

Not actively working on this, but I am working on the other bug. I'll leave this open and recheck once that lands.

Comment 10

[BMO Automation]

We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.