Page MenuHomePhabricator

[Config Change] Make upstream mobile edit notice implementation available at en.wiki
Closed, DuplicatePublic

Description

This task involves the work with making the upstream mobile edit notice implementation we worked on in T312587 available at en.wiki.

NOTE: this config change should not be made until after we have consulted with volunteers at en.wiki and they have shared what mobile edit notice implementation they would like to be made available to people: A) the upstream implementation (T312587) or B) the EditNoticesOnMobile.js implementation.

Deployment Date

@ppelberg to document deployment date once finalized

Requirements

  • The upstream mobile edit notice implementation (T312587) is made available at en.wiki.

Open questions

  • 1. What code would need to be added to EditNoticesOnMobile.js in order to suppress the upstream mobile edit notice implementation, should volunteers at en.wiki prefer for people to see the mobile edit notices EditNoticesOnMobile.js make available?
  • 2. What would be involved with making it possible for the upstream solution to detect the state of the on-wiki gadget so that it could, potentially, disable itself without the gadget needing to make any changes?

Done

  • Volunteers at en.wiki communicate which mobile edit notice implementation they would like people to see
  • Deployment date is documented
  • Config change patch is written and backported

Event Timeline

"What code would need to be added to EditNoticesOnMobile.js suppress the upstream mobile edit notice implementation should volunteers at en.wiki to do this?"

There are some words missing in this line, I think. I'm not sure what you mean.

"What code would need to be added to EditNoticesOnMobile.js suppress the upstream mobile edit notice implementation should volunteers at en.wiki to do this?"

There are some words missing in this line, I think. I'm not sure what you mean.

Eeek, you're right. That language was confusing – thank you for saying something, @AlexisJazz. Does this revised language make more sense to you?

  1. What code would need to be added to EditNoticesOnMobile.js in order to suppress the upstream mobile edit notice implementation, should volunteers at en.wiki prefer for people to see the mobile edit notices EditNoticesOnMobile.js make available?

Right. How about the other way around? Have your solution read mw.user.options.get('gadget-EditNoticesOnMobile') and disable itself? (only works if/when/where EditNoticesOnMobile is available as a gadget though)

Right. How about the other way around? Have your solution read mw.user.options.get('gadget-EditNoticesOnMobile') and disable itself? (only works if/when/where EditNoticesOnMobile is available as a gadget though)

We'll look into whether doing the above is possible.

In the meantime, I've added this as an open question to the task description.

How about the other way around? Have your solution read mw.user.options.get('gadget-EditNoticesOnMobile') and disable itself? (only works if/when/where EditNoticesOnMobile is available as a gadget though)

If we're supporting gadget replacements at all for this feature, which we are, I'd rather let our version (which is inherently going to be distributed to all wikis because it's inside VE) be disabled by many gadgets rather than us needing to explicitly update it to support any other gadgets that come along. E.g. some wiki might want their own implementation rather than ours or yours, or someone who intensely dislikes edit notices could make a gadget that just disables the core implementation without replacing it...

How about the other way around? Have your solution read mw.user.options.get('gadget-EditNoticesOnMobile') and disable itself? (only works if/when/where EditNoticesOnMobile is available as a gadget though)

If we're supporting gadget replacements at all for this feature, which we are, I'd rather let our version (which is inherently going to be distributed to all wikis because it's inside VE) be disabled by many gadgets rather than us needing to explicitly update it to support any other gadgets that come along. E.g. some wiki might want their own implementation rather than ours or yours, or someone who intensely dislikes edit notices could make a gadget that just disables the core implementation without replacing it...

Either way some switch has to be provided. If I can set window.dontLoadVEEditNotices = true or something, that'll work.

https://phabricator.wikimedia.org/rMEXTd046bc1d44faf877e2e71cd7f46b20371ef04e32 : " Edit notices are not shown if the user has the EditNoticesOnMobile gadget enabled."

This is likely the right call as the upstream mobile edit notice implementation could probably load before any gadget. (when visiting a page edit link directly)

Providing a switch for other gadgets or "someone who intensely dislikes edit notices" could still be beneficial, but may not work in every single case.

I've merged this ticket into T316178 seeing as how T312587 makes it so the upstream mobile edit notice implementation will ONLY be used if a project has NOT already enabled Wikipedia:EditNoticesOnMobile thus obviating the need for deployments to happen in phases.