Page MenuHomePhabricator
Differential D19293

When a revision has more than 7 reviewers, render only the first 7 in the list view
ClosedPublic
Actions

Authored by epriestley on Apr 3 2018, 6:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 12:25 PM
Unknown Object (File)
Thu, Nov 21, 5:16 PM
Unknown Object (File)
Mon, Nov 18, 8:22 PM
Unknown Object (File)
Thu, Nov 14, 4:45 AM
Unknown Object (File)
Wed, Nov 13, 3:28 AM
Unknown Object (File)
Tue, Nov 12, 1:34 AM
Unknown Object (File)
Oct 24 2024, 5:45 AM
Unknown Object (File)
Oct 22 2024, 7:05 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13110: Plans: Hefty Differential revisions, draft state transitions, and exotic interactions
Commits
rPe40aec0210dc: When a revision has more than 7 reviewers, render only the first 7 in the list…
Summary

See PHI489. Ref T13110. At least for now, this just shows "..." at the end since you can click the revision to see the whole list anyway.

Also remove the older-style external Handle passing in favor of lazy construction via HandlePool.

Test Plan

Viewed revisions, fiddled with the 7 limit, got sensible-seeming "..." behavior.

Diff Detail

Repository
rP Phabricator
Branch
size1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19995
Build 27110: Run Core Tests
Build 27109: arc lint + arc unit

Event Timeline

epriestley created this revision.Apr 3 2018, 6:51 PM
Harbormaster completed remote builds in B19995: Diff 46174.Apr 3 2018, 6:52 PM
epriestley requested review of this revision.Apr 3 2018, 6:52 PM
This revision was not accepted when it landed; it landed in state Needs Review.Apr 3 2018, 7:47 PM
Closed by commit rPe40aec0210dc: When a revision has more than 7 reviewers, render only the first 7 in the list… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
12 lines
query/
18 lines
view/
84 lines
diffusion/
controller/
6 lines
people/
controller/
6 lines

Diff 46174

View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/applications/differential/query/DifferentialRevisionSearchEngine.php

Loading...
View Options

src/applications/differential/view/DifferentialRevisionListView.php

Loading...
View Options

src/applications/diffusion/controller/DiffusionBrowseController.php

Loading...
View Options

src/applications/people/controller/PhabricatorPeopleProfileRevisionsController.php

Loading...
Phabricator · Built by Phacility