HomePhabricator

In ProfileMenu, put more structure between "stored/configured items" and…

Description

In ProfileMenu, put more structure between "stored/configured items" and "display items"

Summary:
Depends on D20356. Ref T13275. See also T12871 and T12949.

Currently, the whole "ProfileMenu" API operates around stored items. However, stored items are allowed to produce zero or more display items, and we sometimes want to highlight display item X but render stored item Y (as is the case with "Link" items pointing at ?filter=xyz on Workboards).

For the most part, this either: doesn't work; or works by chance; or is kind of glued together with hope and prayer (as in D20353).

Put an actual structural layer in place between "stored/configured item" and "display item" that can link them together more clearly. Now:

  • The list of ItemConfiguration objects (stored/configured items) is used to build an ItemViewList.
  • This handles the selection/highlighting/default state, and knows which display items are related to which stored items.
  • When we're all done figuring out what we're going to select and what we're going to highlight, it pops out an actual View which can build the HTML.

This requires API changes which are not included in this change, see next change.

This doesn't really do anything on its own, but builds toward a more satisfying fix for T12871. I'd hoped to avoid doing this for now, but wasn't able to get a patch I felt good about for T12871 built without fixing this first.

Test Plan: See next change.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13275

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