Page MenuHomePhabricator

Trigger diff-specific functions using the wikipage.diff hook in Mobile Diffs
Closed, ResolvedPublicBUG REPORT

Description

I propose that the relevant portion of code — line 234 in initMobile.js — should be triggered via mw.hook( 'wikipage.diff' ).

This would improve extensibility and allow reuse in scripts that dynamically load diffs, such as my tool: Instant Diffs.

As you can see, the timestamp marker is added by the initModifiedInfo function:

image.png (392×846 px, 47 KB)
image.png (390×844 px, 32 KB)

Details

Event Timeline

Something to consider - tagging @ovasileva for prioritization

KSarabia-WMF lowered the priority of this task from Medium to Low.
KSarabia-WMF moved this task from Incoming to Freezer on the Web-Team board.
KSarabia-WMF added a project: patch-welcome.
Jdlrobson-WMF subscribed.

Diffs on mobile are styled using skinStyles/mediawiki.diff.styles.less in Minerva. Untagging MobileFrontend.

Change #1336902 had a related patch set uploaded (by Osps7; author: Osps7):

[mediawiki/skins/MinervaNeue@master] Trigger modified info on mobile diffs

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

Implemented a patch for this task by triggering initModifiedInfo through the wikipage.diff hook. This allows the modified-info enhancements to be triggered when diffs are dynamically loaded, improving extensibility and reuse for Mobile Diffs.

Change #1336902 had a related patch set uploaded (by Osps7; author: Osps7):

[mediawiki/skins/MinervaNeue@master] Trigger modified info on mobile diffs

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

Change #1336902 had a related patch set uploaded (by Osps7; author: Osps7):

[mediawiki/skins/MinervaNeue@master] Trigger modified info on mobile diffs

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

Change #1336902 merged by jenkins-bot:

[mediawiki/skins/MinervaNeue@master] Trigger modified info on mobile diffs

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