Page MenuHomePhabricator

Ensure discussiontools markup is hidden by CSS when parser split is removed
Open, Needs TriagePublic

Description

This task about making sure that once the parser cache split is removed, the DiscussionTools features visible to people align with the settings the wiki they are using has set as well as the settings they, as individuals, have set within Special:Preferences.

Requirements

Feature visibility

  • People who have the Enable quick replying setting ENABLED within #mw-prefsection-editing should see [ reply ] links on the talk pages they visit
  • People who have the Enable topic subscription setting ENABLED within #mw-prefsection-editing should see [ subscribe ] links on the talk pages they visit
  • People who have the yet-to-be-created Enable readability enhancements setting [i] ENABLED within #mw-prefsection-editing should see topic containers (T269950), new reply and topic affordances (T255560 + T267444) etc.
  • People who have the Enable quick replying setting DISABLED within #mw-prefsection-editing should NOT see [ reply ] links on the talk pages they visit
  • People who have the Enable topic subscription setting DISABLED within #mw-prefsection-editing should NOT see [ subscribe ] links on the talk pages they visit
  • People who have the yet-to-be-created Enable readability enhancements setting [i] DISABLED within #mw-prefsection-editing should NOT see topic containers (T269950), new reply and topic affordances (T255560 + T267444) etc.

Meta
The "Feature visibility" requirements above should not be affected by whether an individual manually configured a particular setting or whether the wiki as a whole configured the setting in a particular way for all users.

Note: I've intentionally omitted the New Discussion Tool from the lists above considering the guidance @DLynch
shared in T285995#7195218: "Guiding principle: are there any visible changes inside the text of the talk page? The comment formatter will run."

Deployment

Deployment of this patch does NOT depend on the Performance and Data Persistence Teams considering it will not have any impact on the parser cache.

Done

  • Functionality is implemented that enables us to have the comment formatter run while ensuring the DiscussionTools features visible to people on talk pages align with the state of the settings they have set within Special:Preferences

i. The Enable readability enhancements setting will be implemented in T270316

Event Timeline

Background: we're going to be moving DiscussionTools out of beta on some wikis. When that happens, the parser-cache split will go away and the DT markup is going to be served to everyone, regardless of whether they have the features enabled. For people who have it disabled, we want to make sure that the reply/subscribe links are hidden.

I've updated the task description with requirements for this task.

@DLynch: do you see anything within the task description that is unexpected, missing, etc.?

@ppelberg the "deployment" section is sort-of irrelevant to this ticket -- it's fine to deploy the CSS-hiding that this references. It's not fine to deploy the removal of the parser cache split, but that's not covered by this ticket anyway.

@ppelberg the "deployment" section is sort-of irrelevant to this ticket -- it's fine to deploy the CSS-hiding that this references.

Noted. Task description updated.

It's not fine to deploy the removal of the parser cache split, but that's not covered by this ticket anyway.

I see. @DLynch, are you aware of a ticket that removes the parser cache split? If not, are you able to file a ticket for this work so I can mark it as a sub-task of T280599?

I ask the above thinking: we have T279864 and it seems like another ticket might be needed to disentangle the parser cache split from tools being offered by default.