Page MenuHomePhabricator
Differential D18465

Prevent Differential changeset HTML anchors from colliding with comment anchors
ClosedPublic
Actions

Authored by epriestley on Aug 24 2017, 8:40 PM.
Tags
None
Referenced Files
F18840788: D18465.id.diff
Tue, Oct 28, 4:02 AM
F18838164: D18465.diff
Mon, Oct 27, 9:31 AM
F18814602: D18465.diff
Mon, Oct 20, 11:35 PM
F18786862: D18465.id44360.diff
Tue, Oct 14, 8:52 PM
F18775441: D18465.diff
Fri, Oct 10, 7:05 PM
F18733731: D18465.id.diff
Sep 30 2025, 10:40 PM
F18639146: D18465.id.diff
Sep 17 2025, 12:51 PM
F18170292: D18465.id44368.diff
Aug 15 2025, 5:57 PM
View All Files
Subscribers
cspeckmim

Details

Reviewers
chad
Maniphest Tasks
T12970: Differential HTML document anchors for changesets and comments may collide
Commits
rPba1925b15515: Prevent Differential changeset HTML anchors from colliding with comment anchors
Summary

Fixes T12970. This is easier than I expected, and appears to occur in only one place.

This prevents a change from ever generating with an anchor like #12345678, which is ambiguous because it may be a comment anchor.

Test Plan

Viewed a revision, saw new change-xxxyyyzzz anchors, clicked one, got jumped to the right place.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 24 2017, 8:40 PM
Harbormaster completed remote builds in B18147: Diff 44360.Aug 24 2017, 8:41 PM
chad accepted this revision.Aug 24 2017, 8:52 PM
This revision is now accepted and ready to land.Aug 24 2017, 8:52 PM
Closed by commit rPba1925b15515: Prevent Differential changeset HTML anchors from colliding with comment anchors (authored by epriestley). · Explain WhyAug 24 2017, 10:25 PM
This revision was automatically updated to reflect the committed changes.
cspeckmim added a subscriber: cspeckmim.Aug 25 2017, 12:05 AM

Would this break anyone's bookmarks to file locations in a diff~

epriestley added a comment.Aug 25 2017, 12:46 AM

Yes, but surely no one has those?

cspeckmim added a comment.Aug 25 2017, 12:53 AM

I don't know anyone that has those

tbh it sounds like something only weirdo internet experts would do

epriestley mentioned this in D18609: Fix an outdated HTML anchor link in Diffusion table of contents.Sep 15 2017, 12:34 PM
epriestley mentioned this in rP5ae3af6691ca: Fix an outdated HTML anchor link in Diffusion table of contents.Sep 15 2017, 3:37 PM
cspeckmim mentioned this in T12970: Differential HTML document anchors for changesets and comments may collide.Jan 22 2018, 3:01 PM
cspeckmim mentioned this in T13045: Differential file anchors may contain characters which prevent remarkup linking properly.Jan 22 2018, 3:51 PM

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
storage/
2 lines

Diff 44368

View Options

src/applications/differential/storage/DifferentialChangeset.php

Loading...
Phabricator · Built by Phacility