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
Unknown Object (File)
Thu, Dec 19, 8:21 PM
Unknown Object (File)
Tue, Dec 17, 1:43 PM
Unknown Object (File)
Sun, Dec 15, 8:18 AM
Unknown Object (File)
Mon, Dec 9, 4:36 PM
Unknown Object (File)
Mon, Dec 2, 11:19 PM
Unknown Object (File)
Mon, Dec 2, 10:45 AM
Unknown Object (File)
Nov 27 2024, 11:46 AM
Unknown Object (File)
Nov 20 2024, 8:19 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