Page MenuHomePhabricator

VisualEditor: Title is not updated on save if DISPLAYTITLE changed
Closed, ResolvedPublic

Description

  1. Edit [[testwiki:Displaytitle]] using VE
  2. Using the transclusion dialog box, add the following "template": DISPLAYTITLE:''displaytitle''
  3. Save the page

The page title remains "Displaytitle" in roman type.

Expected to see "displaytitle" in italics. This is what you see when you refresh the page.


Version: unspecified
Severity: minor
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=49667

Details

Reference
bz50341

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 1:44 AM
bzimport set Reference to bz50341.

Created attachment 12689
Wrong title displayed in "correct title" template.

I think it's related, so I'm sticking this here.

After http://it.wikipedia.org/w/index.php?title=Utente:Dry_Martini/Cricca&diff=prev&oldid=59761793, the page, containing a "Correct title" template, would display, after being saved, the message usually generated by that template, although it disappears if you refresh.

I tried it myself and this time I actually changed the title in the template.
The screenshot shows what happened after http://it.wikipedia.org/w/index.php?title=Utente%3ADry_Martini%2FCricca&diff=59761896&oldid=59761793 my edit.
Basically, the page retains its formerly given title, while displaying in the message the one I chose. Again, everything is ok if you refresh.
(For the record, the usual title is "There is no Cabal", while mine reads "There's a Cabal, and it's using VisualEditor".)

Attached:

wrong_title.jpg (631×1 px, 338 KB)

I think this includes my problem with {{tl|italic title}}, which is at two levels:

  1. When I add it, I expect to see it take effect immediately - it doesn't (so I wonder whether I remembered to add it, whether I formatted it correctly, ... etc)
  2. Even when I save the page, it doesn't take effect until I refresh the page (same set of worries).

Confirmed that adding italics to an existing page doesnt display italics on save - it does display the italic title after save + refresh.

However it does display italics on save, if the DISPLAYTITLE is set for a new page.

Tested with {{italics title}} and {{DISPLAYTITLE:''ItalicsTitle''}} on testwiki

https://test.wikipedia.org/wiki/ItalicsTitle

So now that the dependency is merged, Parsoid now recognises {{DISPLAYTITLE:stuff}} as a meta property (instead of a template), and VE provides a box on the options page to change the display title. I think what we should do is update the title (almost) as soon as it's changed.

Yeah it's actually kind of a pain to do that. Let's update it on save.

Change 139205 had a related patch set uploaded by Alex Monk:
Update displaytitle on save

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

Change 139205 merged by jenkins-bot:
Update displaytitle on save

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

Change 140591 had a related patch set uploaded by Alex Monk:
Update displaytitle on save (try #2)

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

Change 140591 merged by jenkins-bot:
Update displaytitle on save (try #2)

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

Change 141098 had a related patch set uploaded by Alex Monk:
Update displaytitle on save (try #3)

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

Change 141098 merged by jenkins-bot:
Update displaytitle on save (try #3)

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

So this appears to actually be working now.

(In reply to Alex Monk from comment #13)

So this appears to actually be working now.

Woo-hoo. :-)