Page MenuHomePhabricator
Create Task
Maniphest T233813

Separate user preferences for Page previews and Reference previews for Beta
Closed, ResolvedPublic5 Estimated Story Points
Actions

Description

Implement what came out of the investigation documented in T233109: Spike: How should users enable the reference preview feature?:

Separate user preferences for Page Previews and Reference Previews. This will make it possible for users to have both features disabled, both enabled, or one enabled and the other disabled.

Acceptance criteria:

  • The existing setting on the "Appearance" tab should continue to talk about "Page previews" only, not about "popups".
  • The new setting should say something like "enable Reference Previews".
  • The new setting should be directly after the existing one.
  • As long as we are in Beta, the new setting needs to stay on the Beta tab. The new setting on the Appearance tab should already be implemented, but hidden then.
  • The text on the Beta tab needs to be changed: Remove the sentence about "Page Previews must be enabled as well".
  • The existing "Page previews" setting can not disable the extension any more, but only one of the popup types. To achieve this, the check must be moved from the backend (PopupsContext.php) to the frontend (probably in src/index.js).
  • The module should not be loaded when all popup types are disabled.

To do:

  • T235737 The workflow for anonymous users is entirely undefined as of now! It might be that reference previews are always on for anonymous users, or disabled together with page previews.

[] Talk to UX and/or the WMF team formerly working on Page-Previews and ask if it is ok to turn the existing "Enable/Disable" radio buttons into a single checkbox. (Note from @thiemowmde: I remember asking the devs about this. They said this was intentional to give the user something with much more surface area, so it feels much more like a deliberate decision made by the user.)

  • Check if we have a ticket to remove all BetaFeature code when it's not needed any more.
  • Check if we have a ticket to remove the PopupsReferencePreviews feature flag when it's not needed any more. This is especially relevant because this is exposed in JavaScript, and does waste a tiny little bit of our users resources.

Details

SubjectRepoBranchLines +/-
Refactor PopupsHooks for easier removal and/or expansionmediawiki/extensions/Popupswmf/1.35.0-wmf.2+31 -20
Rename PHP constant for ReferencePreview preferencemediawiki/extensions/Popupsmaster+8 -8
Make use of User::getBoolOption() where appropriatemediawiki/extensions/Popupsmaster+3 -4
Put reference previews back into beta mode on beta clusteroperations/mediawiki-configmaster+0 -4
Refactor PopupsHooks for easier removal and/or expansionmediawiki/extensions/Popupsmaster+31 -20
Split user preferences for Page and Reference previewsmediawiki/extensions/Popupsmaster+76 -54
Customize query in gerrit

Related Objects
Search...

Event Timeline

thiemowmde created this task.Sep 25 2019, 1:39 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptSep 25 2019, 1:39 PM
thiemowmde added a subtask: T233109: Spike: How should users enable the reference preview feature?.Sep 25 2019, 1:39 PM
thiemowmde set the point value for this task to 5.
thiemowmde moved this task from Proposed to Currently in sprint on the WMDE-TechWish board.
thiemowmde closed subtask T233109: Spike: How should users enable the reference preview feature? as Resolved.Sep 25 2019, 1:41 PM
thiemowmde mentioned this in T233109: Spike: How should users enable the reference preview feature?.
JStrodt_WMDE moved this task from Backlog to Doing on the Reference Previews board.Sep 30 2019, 8:21 AM
thiemowmde updated the task description. (Show Details)Sep 30 2019, 11:27 AM
WMDE-Fisch claimed this task.Oct 2 2019, 11:17 AM
WMDE-Fisch moved this task from Sprint Backlog to Doing on the WMDE-QWERTY-Sprint-2019-09-25 board.
awight moved this task from Doing to Sprint Backlog on the WMDE-QWERTY-Sprint-2019-09-25 board.Oct 4 2019, 7:52 AM
awight moved this task from Sprint Backlog to Doing on the WMDE-QWERTY-Sprint-2019-09-25 board.
thiemowmde removed WMDE-Fisch as the assignee of this task.Oct 9 2019, 2:15 PM
thiemowmde added a project: WMDE-QWERTY-Sprint-2019-10-09.
thiemowmde added a subscriber: WMDE-Fisch.
thiemowmde claimed this task.Oct 10 2019, 12:31 PM
thiemowmde moved this task from Sprint Backlog to Doing on the WMDE-QWERTY-Sprint-2019-10-09 board.
awight mentioned this in T233037: [Regression] Not possible for logged in users to disable page previews.Oct 10 2019, 1:02 PM
awight added a parent task: T233037: [Regression] Not possible for logged in users to disable page previews.Oct 10 2019, 1:44 PM
gerritbot added a comment.Oct 10 2019, 2:25 PM

Change 542117 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/Popups@master] [WIP] Split user preferences for Page and Reference previews

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

gerritbot added a project: Patch-For-Review.Oct 10 2019, 2:25 PM
gerritbot added a comment.Oct 10 2019, 2:45 PM

Change 542127 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/Popups@master] Make use of User::getBoolOption() where appropriate

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

gerritbot added a comment.Oct 10 2019, 3:24 PM

Change 542140 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/Popups@master] Refactor PopupsHooks for easier removal and/or expansion

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

gerritbot added a comment.Oct 10 2019, 3:37 PM

Change 542150 had a related patch set uploaded (by Thiemo Kreuz (WMDE); owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/Popups@master] Rename PHP constant for ReferencePreview preference

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

thiemowmde removed thiemowmde as the assignee of this task.Oct 11 2019, 8:09 AM
thiemowmde updated the task description. (Show Details)
thiemowmde moved this task from Doing to Review on the WMDE-QWERTY-Sprint-2019-10-09 board.
thiemowmde updated the task description. (Show Details)Oct 11 2019, 8:28 AM
thiemowmde added a comment.Oct 16 2019, 11:19 AM

@Andrew-WMDE, it would be pretty critical if we (the WMDE-TechWish) could merge the patch https://gerrit.wikimedia.org/r/542117 (along with it's requirements) this week, latest next Monday (21 October). We need it to be in the wmf.3 branch cut that will happen on 22 October, see https://www.mediawiki.org/wiki/MediaWiki_1.35/Roadmap.

thiemowmde mentioned this in T235737: Discuss and decide on underspecified behavior for anonymous users.Oct 17 2019, 9:35 AM
gerritbot added a comment.Oct 17 2019, 9:46 AM

Change 542140 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Refactor PopupsHooks for easier removal and/or expansion

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

gerritbot added a comment.Oct 17 2019, 9:46 AM

Change 542117 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Split user preferences for Page and Reference previews

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

thiemowmde removed a project: Patch-For-Review.Oct 17 2019, 12:02 PM
thiemowmde updated the task description. (Show Details)
thiemowmde moved this task from Review to Demo on the WMDE-QWERTY-Sprint-2019-10-09 board.
ReleaseTaggerBot added a project: MW-1.35-notes (1.35.0-wmf.3; 2019-10-22).Oct 17 2019, 2:00 PM
gerritbot added a comment.Oct 21 2019, 8:13 AM

Change 544768 had a related patch set uploaded (by Awight; owner: Thiemo Kreuz (WMDE)):
[mediawiki/extensions/Popups@wmf/1.35.0-wmf.2] Refactor PopupsHooks for easier removal and/or expansion

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

gerritbot added a project: Patch-For-Review.Oct 21 2019, 8:13 AM
awight added a comment.Oct 21 2019, 8:32 AM

We'll need this message translated in order for the new feature to make sense:
https://translatewiki.net/w/i.php?title=Special:Translations&message=MediaWiki%3APopups-refpreview-beta-feature-description%2Fde

Once the translations are done and the code is merged, we need to refresh the localization cache.

gerritbot added a comment.Oct 21 2019, 10:12 AM

Change 544855 had a related patch set uploaded (by Awight; owner: Awight):
[operations/mediawiki-config@master] Put reference previews back into beta mode on beta cluster

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

gerritbot added a comment.Oct 21 2019, 11:40 AM

Change 544855 merged by jenkins-bot:
[operations/mediawiki-config@master] Put reference previews back into beta mode on beta cluster

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

Stashbot added a comment.Oct 21 2019, 11:42 AM

Mentioned in SAL (#wikimedia-operations) [2019-10-21T11:42:23Z] <awight@deploy1001> Synchronized wmf-config/InitialiseSettings-labs.php: SWAT: [[gerrit:544855|Put reference previews back into beta mode on beta cluster (T233813)]] (duration: 01m 00s)

awight added a comment.Oct 21 2019, 12:00 PM

Smoke testing looks good on the beta cluster, in all 4 combinations of Page Previews / Reference Previews enablement!

gerritbot added a comment.Oct 22 2019, 7:31 AM

Change 542150 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Rename PHP constant for ReferencePreview preference

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

gerritbot added a comment.Oct 22 2019, 8:09 AM

Change 542127 merged by jenkins-bot:
[mediawiki/extensions/Popups@master] Make use of User::getBoolOption() where appropriate

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

awight moved this task from Demo to Done on the WMDE-QWERTY-Sprint-2019-10-09 board.Oct 23 2019, 10:03 AM
gerritbot added a comment.Oct 23 2019, 12:52 PM

Change 544768 abandoned by Awight:
Refactor PopupsHooks for easier removal and/or expansion

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

Maintenance_bot removed a project: Patch-For-Review.Oct 23 2019, 1:10 PM
Lena_WMDE moved this task from Doing to In preparation on the Reference Previews board.Oct 14 2020, 4:07 PM
Lena_WMDE updated the task description. (Show Details)Oct 19 2020, 8:32 AM
• MichaelSchoenitzer_WMDE subscribed.Oct 28 2020, 1:27 PM
Lena_WMDE added a parent task: T271206: Enable RefPreviews on first wikis.Jan 5 2021, 3:57 PM
Lena_WMDE added a subtask: T235737: Discuss and decide on underspecified behavior for anonymous users.Jan 6 2021, 8:44 AM
Lena_WMDE renamed this task from Separate user preferences for Page previews and Reference previews to Separate user preferences for Page previews and Reference previews for Beta.Mar 17 2021, 9:18 AM
WMDE-Fisch closed this task as Resolved.Sep 29 2023, 6:29 PM
WMDE-Fisch mentioned this in T282999: Enable Reference Previews on all wikis using the Popups extension, on Nov 21.
WMDE-Fisch updated the task description. (Show Details)

Remaining work is documented in T282999: Enable Reference Previews on all wikis using the Popups extension, on Nov 21

Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL · Credits