Page MenuHomePhabricator

"Display title" broken in visual editor
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

  1. Edit page that includes localized equivalent of {{DISPLAYTITLE:...}} using VisualEditor.
  2. Observe the diff.

What happens?:
String DISPLAYTITLE: is prepended to localized equivalent of {{DISPLAYTITLE:...}} in the article's wikitext.

What should have happened instead?:
The {{DISPLAYTITLE:…}} tag in the article's wikitext shouldn't be changed.

Other information (browser name/version, screenshots, etc.):
Example: https://de.wikipedia.org/w/index.php?diff=259967738
This bug was observed after T402322 was resolved.

Details

Related Changes in Gerrit:

Event Timeline

Change #1190785 had a related patch set uploaded (by Medelius; author: Medelius):

[mediawiki/extensions/VisualEditor@master] WIP: Display title fix

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

Change #1190785 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Display title fix

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

DLynch added a project: Editing QA.
DLynch subscribed.

This was my fault via poor advice -- I thought that Parsoid was still handling localizing this for us, as it was before the switch to treating it as parserfunction markup, and I didn't actually check that I was right about that. (With the caveat this is still less broken than it was before T402322, because it does break the displaytitle, but at least does it without jamming the word DISPLAYTITLE onto the front of the article.)

QA: patch makes it so that whatever alias was used already will be preserved, or the localized variant will be used if you're adding a new displaytitle to a page that didn't already have one. E.g. on dewiki that'd be SEITENTITEL. So, please double-check that we're not accidentally overwriting that with DISPLAYTITLE when editing an existing one.

EAkinloose edited projects, added: Verified; removed: Editing QA.