Page MenuHomePhabricator

Implement Diff view for Jade Entity UI
Closed, ResolvedPublic

Description

We need to create a separate widget for embedded context on a Jade Entity page.

This would be something like DiffWidget and then we add it to ext.Jade.entityView inside of a Panel widget / Stack layout.

The widget will need to call the compare api module and dynamically add a diff view above the Facets list.

Design mockups are here: https://phabricator.wikimedia.org/T212370

Event Timeline

Change 572362 had a related patch set uploaded (by Accraze; owner: Accraze):
[mediawiki/extensions/Jade@master] [WIP] Implement Diff view for Jade Entity UI

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

Moving this to the review column for now. There might be some edge cases I haven't caught yet, however we now have a functional diff and header.

Here's a quick screenshot:

Change 572362 merged by jenkins-bot:
[mediawiki/extensions/Jade@master] Implement Diff view for Jade Entity UI

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

Change 576937 had a related patch set uploaded (by Accraze; owner: Accraze):
[mediawiki/extensions/Jade@master] Use revid for diff widget

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

Change 576937 merged by jenkins-bot:
[mediawiki/extensions/Jade@master] Use revid for diff widget

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

Change 577009 had a related patch set uploaded (by Accraze; owner: Accraze):
[mediawiki/extensions/Jade@master] Handle initial revision diff

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

Change 577009 merged by jenkins-bot:
[mediawiki/extensions/Jade@master] Handle initial revision diff

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