Page MenuHomePhabricator

Make all comments quotable
AbandonedPublic

Authored by joshuaspence on Nov 5 2015, 8:09 PM.
Tags
None
Referenced Files
F15472201: D14416.id34837.diff
Sat, Apr 5, 11:21 AM
F15435620: D14416.diff
Tue, Mar 25, 9:21 AM
F15429457: D14416.id.diff
Mon, Mar 24, 2:26 AM
F15422167: D14416.id34837.diff
Sat, Mar 22, 6:12 AM
F15422052: D14416.id34826.diff
Sat, Mar 22, 5:02 AM
F15419099: D14416.id34826.diff
Fri, Mar 21, 3:28 AM
F15399076: D14416.id.diff
Mon, Mar 17, 2:46 AM
F15395286: D14416.diff
Sun, Mar 16, 4:48 AM
Subscribers

Details

Summary

Fixes T9077. This is a bit messy at the moment and I'm happy to do some tidying up / refactoring, but it seems to work.

Test Plan

Quoted comments in all affected applications.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8646
Build 10013: Run Core Tests
Build 10012: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Make all comments quotable.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.

For the refs, maybe just let the code render a neutral string if no ref is present? e.g.

(With Ref) In Txxx#qqqq, epriestley wrote:
(Without Ref) Previously, epriestley wrote:

We lose the link to the comment, but that seems like a small loss for badges/macros.

src/applications/paste/controller/PhabricatorPasteViewController.php
70

First half of this could be monogram, too.

joshuaspence edited edge metadata.

More applications

epriestley edited edge metadata.

WIP

This revision now requires changes to proceed.Nov 8 2015, 2:24 PM

Not actively working on this.