Page MenuHomePhabricator
Differential D17981

Show "reply" inlines as replies in the objective list
ClosedPublic
Actions

Authored by epriestley on May 20 2017, 12:51 PM.
Tags
None
Referenced Files
F15573702: D17981.id.diff
Mon, May 5, 9:34 PM
F15568551: D17981.id43251.diff
Sat, May 3, 5:08 AM
F15566420: D17981.diff
Thu, May 1, 8:12 PM
F15539161: D17981.id43251.diff
Thu, Apr 24, 11:43 PM
F15509174: D17981.id43238.diff
Wed, Apr 16, 10:25 AM
F15466392: D17981.id43238.diff
Apr 3 2025, 5:02 AM
F15464014: D17981.id43251.diff
Apr 2 2025, 5:26 AM
F15460435: D17981.diff
Mar 31 2025, 11:05 PM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12733: (2017 Week 20) Inline Comments Errata / Feedback
Commits
rPe6843b1c00f3: (stable) Show "reply" inlines as replies in the objective list
rPe15009b76e1b: Show "reply" inlines as replies in the objective list
Summary

Ref T12733. Show a "reply" icon for replies, and make them stack directly under their parent.

Test Plan

Screen Shot 2017-05-20 at 5.49.02 AM.png (109×134 px, 5 KB)

Diff Detail

Repository
rP Phabricator
Branch
map8
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17146
Build 22946: Run Core Tests
Build 22945: arc lint + arc unit

Event Timeline

epriestley created this revision.May 20 2017, 12:51 PM
Harbormaster completed remote builds in B17146: Diff 43238.May 20 2017, 12:52 PM
chad accepted this revision.May 20 2017, 2:34 PM
This revision is now accepted and ready to land.May 20 2017, 2:34 PM
Closed by commit rPe15009b76e1b: Show "reply" inlines as replies in the objective list (authored by epriestley, committed by epriestley). · Explain WhyMay 20 2017, 3:00 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
52 lines
webroot/
rsrc/
js/
application/
diff/
5 lines
18 lines
9 lines

Diff 43238

View Options

resources/celerity/map.php

Loading...
View Options

webroot/rsrc/js/application/diff/DiffInline.js

Loading...
View Options

webroot/rsrc/js/application/diff/ScrollObjective.js

Loading...
View Options

webroot/rsrc/js/application/diff/ScrollObjectiveList.js

Loading...
Phabricator · Built by Phacility