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
F13191616: D18609.id44685.diff
Sat, May 11, 10:59 PM
F13185248: D18609.diff
Sat, May 11, 2:43 AM
Unknown Object (File)
Tue, May 7, 5:10 AM
Unknown Object (File)
Fri, May 3, 5:24 PM
Unknown Object (File)
Wed, May 1, 1:28 AM
Unknown Object (File)
Sat, Apr 27, 10:18 AM
Unknown Object (File)
Mon, Apr 22, 10:34 PM
Unknown Object (File)
Fri, Apr 19, 6:49 PM
View All 86 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