Page MenuHomePhabricator
Differential D18609

Fix an outdated HTML anchor link in Diffusion table of contents
ClosedPublic
Actions

Authored by epriestley on Sep 15 2017, 12:34 PM.
Tags
None
Referenced Files
F18380701: D18609.id44679.diff
Thu, Aug 28, 6:10 PM
F18348293: D18609.id44679.diff
Tue, Aug 26, 4:31 PM
F18281713: D18609.id.diff
Fri, Aug 22, 11:09 PM
F18236641: D18609.id.diff
Wed, Aug 20, 8:25 PM
F18223553: D18609.id.diff
Tue, Aug 19, 10:53 PM
F18215729: D18609.id.diff
Tue, Aug 19, 7:04 AM
F18180079: D18609.id44679.diff
Sat, Aug 16, 5:22 AM
F18112277: D18609.diff
Aug 12 2025, 4:18 PM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Commits
rP5ae3af6691ca: Fix an outdated HTML anchor link in Diffusion table of contents
Summary

See https://discourse.phabricator-community.org/t/navigating-to-changed-files-in-diffusion-does-not-work-anymore/433.

In D18465, I updated these but this hard-coded the anchor for some reason (???) and I missed it while grep-ing.

Test Plan

Viewed a commit (/rXYZaaaa) and clicked a file link in the table of contents. Got modern #change-... anchor and navigation into the document.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
src/
applications/
diffusion/
controller/
2 lines

Diff 44685

View Options

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...
Phabricator · Built by Phacility