Page MenuHomePhabricator
Differential D12703

Provide link to inlines in-context in $ghost['href']
ClosedPublic
Actions

Authored by epriestley on May 4 2015, 6:45 PM.
Tags
None
Referenced Files
F19054276: D12703.id.diff
Fri, Nov 28, 11:44 AM
F19049510: D12703.diff
Thu, Nov 27, 5:41 PM
F18864766: D12703.diff
Nov 3 2025, 8:35 AM
F18832011: D12703.diff
Oct 25 2025, 6:22 PM
F18823687: D12703.diff
Oct 23 2025, 12:58 PM
F18823135: D12703.diff
Oct 23 2025, 10:19 AM
F18673951: D12703.id30499.diff
Sep 25 2025, 12:41 PM
F18646968: D12703.id.diff
Sep 19 2025, 11:49 AM
View All Files
Subscribers
epriestley
yelirekim

Details

Reviewers
chad
Commits
Restricted Diffusion Commit
rP460d68ba27b6: Provide link to inlines in-context in $ghost['href']
Summary

See D12702.

Test Plan

Made something a link and clicked it, seemed to work OK.

Diff Detail

Repository
rP Phabricator
Branch
ghost1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5742
Build 5761: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 30499.May 4 2015, 6:45 PM
epriestley retitled this revision from to Provide link to inlines in-context in $ghost['href'].
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
Herald added subscribers: epriestley, yelirekim. · View Herald TranscriptMay 4 2015, 6:45 PM
epriestley mentioned this in D12702: Remove click to view on ghosties.May 4 2015, 6:45 PM
chad accepted this revision.May 4 2015, 6:48 PM
chad edited edge metadata.
This revision is now accepted and ready to land.May 4 2015, 6:48 PM
Closed by commit rP460d68ba27b6: Provide link to inlines in-context in $ghost['href'] (authored by epriestley, committed by epriestley). · Explain WhyMay 4 2015, 6:52 PM
This revision was automatically updated to reflect the committed changes.
chad added a comment.May 4 2015, 6:59 PM

It’d be nice if the link also changed the background color to yellow like we do transaction links.

epriestley added a comment.May 4 2015, 7:00 PM

Yeah, I think it's hypothetically supposed to but that code has been a bit finnicky in the past. I'll file something and look into it.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
6 lines
6 lines
query/
9 lines
storage/
3 lines

Diff 30499

View Options

src/applications/differential/controller/DifferentialChangesetViewController.php

Loading...
View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/applications/differential/query/DifferentialInlineCommentQuery.php

Loading...
View Options

src/applications/differential/storage/DifferentialRevision.php

Loading...
Phabricator · Built by Phacility