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
F13321503: D12703.diff
Fri, Jun 14, 6:55 AM
F13290064: D12703.diff
Tue, Jun 4, 4:02 PM
F13274709: D12703.diff
Fri, May 31, 4:00 AM
F13260762: D12703.diff
May 27 2024, 12:21 AM
F13259396: D12703.id30501.diff
May 26 2024, 8:04 PM
F13247580: D12703.id30499.diff
May 23 2024, 8:41 PM
F13244864: D12703.diff
May 23 2024, 5:23 AM
F13213125: D12703.id.diff
May 17 2024, 7:24 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 30501

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