Page MenuHomePhabricator
Differential D16632

Group similar transaction comments in Conpherence
ClosedPublic
Actions

Authored by chad on Sep 29 2016, 7:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 13, 2:26 PM
Unknown Object (File)
Thu, Dec 5, 8:38 PM
Unknown Object (File)
Wed, Dec 4, 9:16 AM
Unknown Object (File)
Mon, Dec 2, 10:32 PM
Unknown Object (File)
Thu, Nov 28, 10:20 AM
Unknown Object (File)
Sat, Nov 23, 4:57 PM
Unknown Object (File)
Thu, Nov 21, 12:52 AM
Unknown Object (File)
Tue, Nov 19, 9:23 PM
View All Files
Subscribers
Korvin
Tokens
"Party Time" token, awarded by scp.

Details

Reviewers
scp
epriestley
Commits
rP8af29f2df1a1: Group similar transaction comments in Conpherence
Summary

Adds a CSS class if comments come in from the same user in the past 2 minutes for cleaner UI. Note will have to find some better display UI when comment editing comes.

Test Plan

Test lots of random Conpherence messages with different transactions, different people, and quick commenting.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 40039.Sep 29 2016, 7:20 PM
chad retitled this revision from to Group similar transaction comments in Conpherence.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, scp.
Herald added a subscriber: Korvin. · View Herald TranscriptSep 29 2016, 7:20 PM
epriestley accepted this revision.Sep 29 2016, 7:21 PM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Sep 29 2016, 7:21 PM
Closed by commit rP8af29f2df1a1: Group similar transaction comments in Conpherence (authored by chad, committed by chad). · Explain WhySep 29 2016, 7:23 PM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Sep 29 2016, 7:25 PM

@scp you still owe us one. I didn't use your "hax"

scp awarded a token.Sep 30 2016, 3:41 AM

Revision Contents
Changeset List

PathSize
resources/
celerity/
6 lines
src/
applications/
conpherence/
20 lines
webroot/
rsrc/
css/
application/
conpherence/
10 lines

Diff 40040

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/conpherence/ConpherenceTransactionRenderer.php

Loading...
View Options

webroot/rsrc/css/application/conpherence/message-pane.css

Loading...
Phabricator · Built by Phacility