Page MenuHomePhabricator

Disable ReferencePreviews when gadget is detected
Closed, ResolvedPublic8 Estimated Story Points

Description

Background
Based on investigation in T234204: Investigation: how many wikis have the ReferenceTooltip Gadget enabled by default?, we know the experience of encountering both the ReferencePreview extension and the Navigation-Popups or ReferenceTooltip gadget, which is set as default on several top 10 wikis, is a broken one. To ensure that doesn't occur, we have decided to auto-disable ReferencePreviews if the gadgets are detected.

Open questions:

  • How should the auto-disable detect the gadget? Based on name or another solution? Same as already done for Navigation-Popups, see T234204#5604893
  • Under which names does the ReferenceTooltip gadget exist on the wikis? If solution is to be based on it's name, gather list of names used on wikis that have this gadget enabled in the configuration.

Requirements:

  • Make our Reference Previews auto-disable when the gadget is detected. This should work both for logged-in users that have the gadget enabled, and for anynomous users on wikis in which the gadget is enabled by default.

Event Timeline

Lena_WMDE updated the task description. (Show Details)

We'll read the user's preferences to check for the gadget being enabled.

... which won't work for anonymous users.

This ticket should only deal with logged-in users. For anons, we just have to be careful to never configure both preview features by default.

One more comment about this. We do need to handle the case where a wiki is configured with both Reference Tooltips and Reference Previews enabled by default. Always suppress ours in this case, as discussed here.

Lena_WMDE renamed this task from [Draft] Disable ReferencePreviews when gadget is detected to Disable ReferencePreviews when gadget is detected.Oct 19 2020, 9:42 AM
Lena_WMDE updated the task description. (Show Details)
Lena_WMDE updated the task description. (Show Details)
Lena_WMDE set the point value for this task to 8.

Change 637531 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/BetaFeatures@master] Allow Beta preference checkboxes to be disabled

https://gerrit.wikimedia.org/r/637531

Change 637531 merged by jenkins-bot:
[mediawiki/extensions/BetaFeatures@master] Allow Beta preference checkboxes to be disabled

https://gerrit.wikimedia.org/r/637531

Analogous to conflicts with navigation pop-ups we adapted the code, so that only one pop-up at a time is being displayed. This is the current state (click to start):

Peek_reference-previews_ 2020-11-02.gif (770×1 px, 1 MB)

Change 638490 had a related patch set uploaded (by Svantje Lilienthal; owner: Svantje Lilienthal):
[mediawiki/extensions/Popups@master] hide reference previews when reference tooltip gadget is active

https://gerrit.wikimedia.org/r/638490