Page MenuHomephabricator

Parse the Phabricator links on other relevant pages as well
ClosedPublic

Authored by kerberizer on Nov 15 2018, 11:59 AM.

Details

Summary

D5 implemented parsing the links to Phabricator in JSS 9's summaries only when viewing the history of pages in the MediaWiki and Module namespaces. However, these links appear also in other places, where it is still desirable to convert them to proper hyperlinks:

  • Special:Contributions/JSS 9,
  • Special:Recentchanges,
  • Special:Watchlist,
  • diff links of pages in NS MediaWiki and Module.
Test Plan

The code behaves as expected in the browser's JS console.

Diff Detail

Repository
rUI JS/CSS User Interface
Branch
enh/jss-links-all
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 18
Build 18: arc lint + arc unit

Event Timeline

kerberizer requested review of this revision.Nov 15 2018, 11:59 AM
kerberizer created this revision.
kerberizer retitled this revision from Parse the Phabricator links on other relevant pages as well. to Parse the Phabricator links on other relevant pages as well.Nov 15 2018, 12:07 PM
kerberizer planned changes to this revision.Nov 15 2018, 3:46 PM

Открих си абсурдно глупава грешка. След малко ще кача обновена версия на пача.

kerberizer updated this revision to Diff 21.Nov 15 2018, 4:29 PM

D'oh! Selecting on Special:Contributions needs a different method.

kerberizer updated this revision to Diff 22.Nov 15 2018, 4:49 PM

$jss9Comments needs not be global, use "var".

Borislav accepted this revision.Nov 17 2018, 1:25 PM
This revision is now accepted and ready to land.Nov 17 2018, 1:25 PM
This revision was automatically updated to reflect the committed changes.