Paths

Table of Contentst

Differential D12490

Distinguish between ported-forward and ported-backward comments
ClosedPublic
Actions

Authored by epriestley on Apr 21 2015, 1:19 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Dec 17, 4:26 PM

	Unknown Object (File)
	Thu, Dec 5, 1:16 PM

	Unknown Object (File)
	Tue, Dec 3, 5:28 PM

	Unknown Object (File)
	Mon, Dec 2, 10:44 PM

	Unknown Object (File)
	Thu, Nov 28, 1:01 AM

	Unknown Object (File)
	Tue, Nov 26, 10:45 PM

	Unknown Object (File)
	Wed, Nov 20, 9:22 PM

	Unknown Object (File)
	Wed, Nov 20, 9:22 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T7447: Bring inline comments forward across revision updates

Commits

Restricted Diffusion Commit
rPfe774e68e255: Distinguish between ported-forward and ported-backward comments

Summary

Ref T7447. This might be overkill, but I want to over-explain things until we have more confidence that this is rarely confusing.

NOTE: I'm playing it a bit fast and loose with setIsGhost() (passing a dictionary) because making API changes requires changing the interface and Diffusion, which is a pain. I'll clean this up at the end once the interface is more final. This is well-contained for now.

Test Plan

Viewed "base vs 2" in a diff with 3 diffs, saw some "older comments" and some "newer comments".
Hovered the tags for an explanation of comment spookiness.

Screen_Shot_2015-04-21_at_6.18.54_AM.png (413×809 px, 51 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 29985.Apr 21 2015, 1:19 PM

epriestley retitled this revision from to Distinguish between ported-forward and ported-backward comments.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T7447: Bring inline comments forward across revision updates.

Herald added a subscriber: epriestley. · View Herald TranscriptApr 21 2015, 1:19 PM

epriestley updated this object.Apr 21 2015, 1:19 PM

btrahan accepted this revision.Apr 21 2015, 5:33 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Apr 21 2015, 5:33 PM

Closed by commit rPfe774e68e255: Distinguish between ported-forward and ported-backward comments (authored by epriestley, committed by epriestley). · Explain WhyApr 21 2015, 6:12 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

controller/

DifferentialChangesetViewController.php

17 lines

infrastructure/

diff/

view/

PHUIDiffInlineCommentDetailView.php

19 lines

Diff 30000

src/applications/differential/controller/DifferentialChangesetViewController.php

Loading...

src/infrastructure/diff/view/PHUIDiffInlineCommentDetailView.php

Loading...