Order inline comments in Diffusion consistently with Differential
ClosedPublic
Actions

Authored by epriestley on Jan 11 2017, 10:46 PM.

Tags

None

Referenced Files

	F19523799: D17184.id.diff
	Sat, Jan 17, 7:36 PM

	F19264080: D17184.diff
	Mon, Dec 22, 6:42 PM

	F18826717: D17184.id.diff
	Oct 24 2025, 5:22 AM

	F18825617: D17184.diff
	Oct 23 2025, 11:02 PM

	F18821259: D17184.id.diff
	Oct 22 2025, 9:14 PM

	F18817792: D17184.id41327.diff
	Oct 21 2025, 6:29 PM

	F18789442: D17184.diff
	Oct 15 2025, 9:43 AM

	F18784679: D17184.id41327.diff
	Oct 13 2025, 2:23 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T8739: Inline comments not sorted

Commits

rPb471f6c07aef: Order inline comments in Diffusion consistently with Differential

Summary

Fixes T8739. Currently, Diffusion inline comments in the timeline are sorted arbitrarily, mostly by creation order.

Instead, sort them by line number, like Differential.

Test Plan

Made comments in "C", "B", "A" order, saw them in line order after submit:

Screen Shot 2017-01-11 at 2.44.08 PM.png (271×504 px, 36 KB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 11 2017, 10:46 PM

Harbormaster completed remote builds in B15211: Diff 41324.Jan 11 2017, 10:47 PM

chad accepted this revision.Jan 11 2017, 10:56 PM

This revision is now accepted and ready to land.Jan 11 2017, 10:56 PM

Closed by commit rPb471f6c07aef: Order inline comments in Diffusion consistently with Differential (authored by epriestley, committed by epriestley). · Explain WhyJan 11 2017, 10:57 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

audit/

view/

PhabricatorAuditTransactionView.php

82 lines

Diff 41327

View Options

src/applications/audit/view/PhabricatorAuditTransactionView.php

Order inline comments in Diffusion consistently with DifferentialClosedPublicActions