Page MenuHomePhabricator

Introduce an edit tag that gets appended when ≥1 Tone Check is activated within an edit
Closed, ResolvedPublic

Description

This task involves the work of appending an edit tag to any published edit in which ≥1 Peacock Check is shown.

Requirements

  1. Introduce a new tag called TBD that it is applied to edits where the Tone Check is actually shown to people in the process of editing Wikipedia.
  2. The new editcheck-tone-shown tag is implemented in a way that does not limit the flexibility we have to [eventually] make it possible for people to see the number of Tone Checks that are activated within a given edit session
  3. This tag should be visible

Open questions

  • 1. What will we call this new tag?
    • editcheck-tone-shown

Done

  • All requirements are implemented
  • mw:Edit_check/Tags is updated to reflect this change
  • Editing QA confirms tag is being applied as expected

Event Timeline

Per what @DLynch shared offline, before moving forward with introducing a new Edit Tag for Peacock Check (as this task asks for), we'll need to consider how/if this one-tag/Check approach will scale to a future where ≥10 Check types could be activated within a single edit session.

In line with the above, I've added T342460 as a sub-task of this one.

Look for mw.editcheck.refCheckShown in init.js and controller.js to see what we're currently doing for the tag tracking the reference check. Note that Hooks.php contains a whitelist of tags that're allowed to be added this way.

Assigning this to myself to populate task description, using T342462 as a reference.

Update

Barring any blocking issues with renaming Peacock Check to Tone Check, I propose we name the tag that gets appended when ≥1 Peacock Check is activated within an edit editcheck-tone-shown.

The above follows the conventions we redefined in T373949.

Next step(s)

  • @ppelberg: ensure no blocking concerns with renaming Peacock Check to Tone Check
Aklapper renamed this task from Introduce an edit tag that gets appended when ≥1 Peacock Check is activated within an edit to Introduce an edit tag that gets appended when ≥1 Tone Check is activated within an edit.May 28 2025, 11:43 AM

Change #1152382 had a related patch set uploaded (by DLynch; author: DLynch):

[mediawiki/extensions/VisualEditor@master] Edit check: create editcheck-tone-shown tag

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

The tag is implemented as editcheck-tone-shown to mirror the reference check tag, and I made roughly-equivalent descriptions for translation purposes.

Change #1152382 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Edit check: create editcheck-tone-shown tag

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

The tag is implemented as editcheck-tone-shown to mirror the reference check tag, and I made roughly-equivalent descriptions for translation purposes.

Wonderful – this aligns with what I'd been thinking (T389897#10836525).

ppelberg added a project: Editing QA.
ppelberg moved this task from Inbox to High Priority on the Editing QA board.

Assumptions made while testing:

  1. TBD in No. 1 requirement is editcheck-tone-shown

✅ Tag is rightly attached

image.png (74×1 px, 38 KB)

⚠️ I don't see editcheck-tone-shown or Edit Check (tone) shown in the Tag Filter dropdown

image.png (1×1 px, 185 KB)

Filter rightly displayed

image.png (1×1 px, 224 KB)

Ryasmeen reopened this task as Open.EditedSep 8 2025, 8:43 PM
Ryasmeen subscribed.

I don't see the editcheck-tone-shown attached to these edits where the Tone Check card was presented during the edit sessions on this page:https://en.wikipedia.beta.wmcloud.org/w/index.php?title=Cats&action=history

Screenshot 2025-09-08 at 9.40.39 PM.png (1×2 px, 412 KB)

ppelberg triaged this task as High priority.Sep 8 2025, 8:54 PM

I think this was the known issue with the tag only being applied when you saw the check pre-save. That was fixed yesterday, though semi-amusingly it looks like the patch merged about 10 minutes after you tested it @Ryasmeen. Could you check again and make sure the tag is getting applied both mid-edit and pre-save?

My own tag from mid-edit:

CleanShot 2025-09-09 at 10.09.33@2x.png (144×1 px, 78 KB)

(You can also check this in the console to see what checks are remembered as having been shown in the current session: mw.editcheck.checksShown.)

I checked the fix for some scenarios that involves both mid-edit and pre-save trigger of tone check and verified that the tag editcheck-tone-shown is getting applied for both mid-edit and pre-save.