Page MenuHomePhabricator

Order inline comments in Diffusion consistently with Differential
ClosedPublic

Authored by epriestley on Jan 11 2017, 10:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 19, 3:29 AM
Unknown Object (File)
Tue, Mar 5, 11:33 AM
Unknown Object (File)
Feb 14 2024, 3:41 PM
Unknown Object (File)
Feb 11 2024, 6:41 PM
Unknown Object (File)
Jan 27 2024, 1:58 PM
Unknown Object (File)
Jan 24 2024, 8:22 AM
Unknown Object (File)
Jan 20 2024, 4:32 PM
Unknown Object (File)
Jan 16 2024, 5:11 PM
Subscribers
None

Details

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