Page MenuHomePhabricator

Gadgets now leak into mobile site
Closed, ResolvedPublic


I have a gadget enabled - mw:Reference Tooltips and this now seems to be running on the mobile site. There's no guarantee gadgets will work on mobile so we should probably disable these for the time being.

Luckily this won't effect the average visitor who will not have gadgets installed.

Version: unspecified
Severity: normal



Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 1:22 AM
bzimport set Reference to bz44918.

Can't seem to replicate this on a local version.. not sure why this is happening.

(In reply to comment #0)

Luckily this won't effect the average visitor who will not have gadgets

Some gadgets are enabled by default.

Max lies (according to Maryana)
On Maryana's account I'm seeing the following gadgets load:




Several mobile exceptions thrown breaking her mobile experience.

Uncaught ReferenceError: hookEvent is not defined index.php:7671
Uncaught ReferenceError: importStylesheet is not defined index.php:19
Uncaught ReferenceError: getElementsByClassName is not defined

mgrover wrote:

Change on 02/28 were abandoned (Gerrit Change Ibc43852486041b940464310201654e4fa6c53c6e) | change APPROVED and MERGED [by jenkins-bot]

Did the core change get applied? gadgets still seem to be leaking..

The Gadgets change is not yet live:

maxsem@fenari:/h/w/c/php-1.22wmf3/extensions/Gadgets$ grep wfRunHooks Gadgets_body.php

Interesting - the changeset is present (via git log) in the Gadgets extension submodule of the php-1.22wmf3 branch; I guess it just hasn't been updated/deployed in production.