Page MenuHomePhabricator
Differential D18129

Move "List Inline Comments" to the inline header dropdown menu
ClosedPublic
Actions

Authored by epriestley on Jun 15 2017, 1:35 PM.
Tags
None
Referenced Files
F18094746: D18129.id.diff
Thu, Aug 7, 10:19 PM
F18090836: D18129.id43623.diff
Wed, Aug 6, 5:56 PM
F18086837: D18129.id43624.diff
Wed, Aug 6, 12:31 AM
F18086755: D18129.id43624.diff
Tue, Aug 5, 11:55 PM
F18052500: D18129.id.diff
Mon, Aug 4, 2:43 AM
F18048976: D18129.diff
Sun, Aug 3, 2:52 PM
Unknown Object (File)
Jun 2 2025, 2:27 AM
Unknown Object (File)
May 31 2025, 10:58 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T8250: Would like to see all comments and their status in a list
T12733: (2017 Week 20) Inline Comments Errata / Feedback
Commits
rP9b93697d52f4: Move "List Inline Comments" to the inline header dropdown menu
Summary

See D18128. Ref T12733. Ref T8250.

Test Plan

Screen Shot 2017-06-15 at 5.31.34 AM.png (1×1 px, 516 KB)

Diff Detail

Repository
rP Phabricator
Branch
menu4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 17515
Build 23492: Run Core Tests
Build 23491: arc lint + arc unit

Event Timeline

epriestley created this revision.Jun 15 2017, 1:35 PM
Harbormaster completed remote builds in B17515: Diff 43623.Jun 15 2017, 1:36 PM
chad accepted this revision.Jun 15 2017, 1:55 PM
This revision is now accepted and ready to land.Jun 15 2017, 1:55 PM
Closed by commit rP9b93697d52f4: Move "List Inline Comments" to the inline header dropdown menu (authored by epriestley, committed by epriestley). · Explain WhyJun 15 2017, 2:11 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
50 lines
src/
applications/
differential/
controller/
12 lines
view/
13 lines
webroot/
rsrc/
js/
application/
diff/
17 lines
differential/
3 lines
phuix/
10 lines

Diff 43623

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/applications/differential/view/DifferentialChangesetListView.php

Loading...
View Options

webroot/rsrc/js/application/diff/DiffChangesetList.js

Loading...
View Options

webroot/rsrc/js/application/differential/behavior-populate.js

Loading...
View Options

webroot/rsrc/js/phuix/PHUIXActionView.js

Loading...
Phabricator · Built by Phacility