Page MenuHomePhabricator

VisualDiffs: Add custom messages for reference changes
Closed, ResolvedPublic8 Estimated Story Points

Description

References

  • added/removed
  • group changed
  • contents changed

References list

  • added/removed
  • group changed
  • columns setting changed

Event Timeline

Change 343134 had a related patch set uploaded (by Jforrester):
[mediawiki/extensions/Cite] [WIP] Describe group changes for references and references lists

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

Jdforrester-WMF removed a project: Patch-For-Review.
Jdforrester-WMF updated the task description. (Show Details)
Jdforrester-WMF changed the point value for this task from 1 to 8.

Change 343134 merged by jenkins-bot:
[mediawiki/extensions/Cite@master] Describe group changes for references and references lists

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

Tchanders added a subscriber: Esanders.

@Esanders Do you have any ideas about "columns setting changed"?

For {{reflist}] there is probably a column attribute, but we don't yet support templates but that's a separate issue. For <references> list we recently introduced the responsive attribute, so maybe we could add a message if that is changed, but given we don't have a UI for changing it in VE yet - I'd say we're done here. @Jdforrester-WMF?

Deskana lowered the priority of this task from High to Medium.Aug 22 2017, 2:29 PM

Change 413643 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/Cite@master] VisualEditor: Describe reflist responsiveness changes too

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

Jdforrester-WMF moved this task from Incoming to Code review on the VisualEditor (Current work) board.
Jdforrester-WMF added a subscriber: Tchanders.

Only one, pretty trivial part of this left.

Change 413643 merged by jenkins-bot:
[mediawiki/extensions/Cite@master] VisualEditor: Describe reflist responsiveness changes too

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

I see the custom messages for the changes related with References list but not for adding/removing/changing group/changing content for references. Not sure if I am missing something.

Hmm, the task is phrased a bit strangely. We don't actually have custom messages for adding/removing/changing content for references - we just show the change.

As for changing the group of a reference did you mean this or something else?

Screenshot from 2018-02-28 21-21-16.png (83×888 px, 6 KB)

Screenshot from 2018-02-28 21-21-35.png (63×871 px, 13 KB)

Hmm, the task is phrased a bit strangely. We don't actually have custom messages for adding/removing/changing content for references - we just show the change.

Got it!

As for changing the group of a reference did you mean this or something else?

Screenshot from 2018-02-28 21-21-16.png (83×888 px, 6 KB)

Screenshot from 2018-02-28 21-21-35.png (63×871 px, 13 KB)

I actually meant the following by changing the group of a reference, which seems to be working fine now :)

Screen Shot 2018-02-28 at 2.06.43 PM.png (396×901 px, 50 KB)