Page MenuHomePhabricator

Rename the "Visual edit" tab to "Edit with form"
Closed, ResolvedPublic

Description

As I started working on it, I realized what we're building isn't really so much a "visual" mode as it is just adding a form. "Visual" implies WYSIWYG, as in you could click on the chart itself to add a label, etc.

Looking at T425608 may illustrate the problem better. Even in source editing, you can get the same preview. So really, our work is just supplying a form in place of a JSON editor.

Acceptance criteria

Derived Requirement

  1. The editing interface must replace the "Visual edit" tab label with "Edit with form".
  2. The updated tab label must accurately communicate that the interface provides a form-based editing experience rather than a WYSIWYG visual editor.
  3. The renamed tab must remain accessible and functional in the same location as the previous "Visual edit" tab.
  4. Selecting the "Edit with form" tab must continue to open the existing form-based editing workflow without behavioral regressions.
  5. References to the previous "Visual edit" label within the related editing interface must be updated for consistency where applicable.

Test Steps

Test Case 1: Verify the "Visual edit" tab is renamed to "Edit with form"

  1. Open a page that supports the chart editing workflow.
  2. Navigate to the editing interface containing the editing tabs.
  3. Observe the tab labels.
  4. ✅❓❌⬜ AC1: The tab label displays "Edit with form" instead of "Visual edit".

Test Case 2: Verify the renamed tab remains functional

  1. Open a page that supports the chart editing workflow.
  2. Locate the "Edit with form" tab.
  3. Select the tab.
  4. Observe the editing interface that loads.
  5. ✅❓❌⬜ AC2: Selecting the "Edit with form" tab opens the form-based editing interface successfully.

Test Case 3: Verify tab placement remains consistent after renaming

  1. Open a page that supports the chart editing workflow.
  2. Locate the editing tabs.
  3. Compare the placement of the "Edit with form" tab with expected tab ordering.
  4. ✅❓❌⬜ AC3: The "Edit with form" tab appears in the same interface location previously used by "Visual edit".

Test Case 4: Verify the editing experience is form-based and not WYSIWYG

  1. Open the "Edit with form" tab on a supported page.
  2. Observe the editing controls presented to the user.
  3. Attempt to directly manipulate chart content visually.
  4. ✅❓❌⬜ AC4: The interface presents structured form inputs rather than direct WYSIWYG editing controls.

QA Results - Local

Event Timeline

MusikAnimal changed the task status from Open to In Progress.May 9 2026, 2:08 AM
MusikAnimal claimed this task.

Change #1285471 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/Chart@master] ext.chart.wizard: Rename to Chart Wizard; conform to designs

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

Change #1285471 merged by jenkins-bot:

[mediawiki/extensions/Chart@master] ext.chart.wizard: Rename to Chart Wizard; conform to designs

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

HMonroy changed the task status from In Progress to Open.May 12 2026, 1:10 AM

@MusikAnimal Can you please review the potential issues section below if it's anything worth fixing?

Test Result - Local

Status: ✅ PASS / ❓ Need More Info
Environment: Local: 1.47.0-alpha (d260740) 15:33, 12 May 2026
OS: macOS Tahoe 26.4.1
Browser: Chrome 147
Device: MBA
Emulated Device: NA

Test Artifact(s):

Test Steps

Test Case 1: Verify the "Visual edit" tab is renamed to "Edit with form"

  1. Open a page that supports the chart editing workflow.
  2. Navigate to the editing interface containing the editing tabs.
  3. Observe the tab labels.
  4. ✅❓ AC1: The tab label displays "Edit with form" instead of "Visual edit".

See ❓Issues below

Test Case 2: Verify the renamed tab remains functional

  1. Open a page that supports the chart editing workflow.
  2. Locate the "Edit with form" tab.
  3. Select the tab.
  4. Observe the editing interface that loads.
  5. AC2: Selecting the "Edit with form" tab opens the form-based editing interface successfully.

See AC1

Test Case 3: Verify tab placement remains consistent after renaming

  1. Open a page that supports the chart editing workflow.
  2. Locate the editing tabs.
  3. Compare the placement of the "Edit with form" tab with expected tab ordering.
  4. AC3: The "Edit with form" tab appears in the same interface location previously used by "Visual edit".

See AC1

Test Case 4: Verify the editing experience is form-based and not WYSIWYG

  1. Open the "Edit with form" tab on a supported page.
  2. Observe the editing controls presented to the user.
  3. Attempt to directly manipulate chart content visually.
  4. AC4: The interface presents structured form inputs rather than direct WYSIWYG editing controls.

See AC1

❓Issues

  1. Flickers to Edit for a second when you click on "Edit with form"

  1. Options disappear when you click "Edit with form" in Minerva

  1. Ar language, "Edit with form" does not translate

2026-05-13_15-56-12.png (1,407×1,002 px, 110 KB)

GMikesell-WMF changed the task status from Open to In Progress.May 13 2026, 11:05 PM
GMikesell-WMF updated the task description. (Show Details)
GMikesell-WMF moved this task from QA to In Development on the Community-Tech (Sea Lion Squad) board.

❓Issues

  1. Flickers to Edit for a second when you click on "Edit with form"

F81279753

This will be fixed after T425610. Until then, you can ignore it.

  1. Options disappear when you click "Edit with form" in Minerva

F81280013

This will fall under T426014 I guess. We haven't done anything at all for mobile treatment yet.

  1. Ar language, "Edit with form" does not translate

2026-05-13_15-56-12.png (1,407×1,002 px, 110 KB)

It hasn't been translated yet. Generally speaking, I think for the purposes of QA you only really need to test the LTR/RTL behavour. Translations are more or less guaranteed to work for system messages, so if you see something untranslated, it's because no one has translated it yet. (By contrast, for MediaWiki-extensions-CommunityRequests we needed to test user-submitted translations, but that is not the case here.)

It's rather tedious but one thing you can do is browse to the i18n files directly to see what has been translated:

Note here we are browsing the master branch, so whatever is there should be on your local (assuming you're on the master branch and have pulled in the latest changes), and this also will be what's on Beta. But in production, translations can be delayed by up to a week.

MusikAnimal changed the task status from In Progress to Open.Thu, May 14, 2:50 PM

Got it, I will ignore the other 2 issues since they will be taken care of in other tasks. Noted with Translation. No behavior issue with RTL. I will mark this as Resolved. Thanks for all your work!

GMikesell-WMF updated the task description. (Show Details)
GMikesell-WMF updated the task description. (Show Details)
GMikesell-WMF updated Other Assignee, removed: GMikesell-WMF.

Change #1287513 had a related patch set uploaded (by MusikAnimal; author: MusikAnimal):

[mediawiki/extensions/Chart@master] wizard: rename ChartVisualModeEnabled feature flag to ChartWizardEnabled

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

Change #1287513 merged by jenkins-bot:

[mediawiki/extensions/Chart@master] wizard: rename ChartVisualModeEnabled feature flag to ChartWizardEnabled

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