HomePhorge

When a revision has more than 7 reviewers, render only the first 7 in the list…

Description

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

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.

Maniphest Tasks: T13110

Differential Revision: https://secure.phabricator.com/D19293

Details

Provenance
epriestleyAuthored on Apr 3 2018, 11:48 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP592d72e00688: Move PhabricatorModularTransaction slightly closer to having "final" methods…
Branches
Unknown
Tags
Unknown

Event Timeline