Ref T4100. This is still a bit rough around the edges, but mostly does what we're after.
- Implements viewer() and members(...) functions.
- The new browse workflow makes these discoverable.
Differential D12444
Implement viewer() and members(project) typeahead functions epriestley on Apr 17 2015, 12:37 AM. Authored by Tags None Referenced Files
Subscribers Tokens
Details
Ref T4100. This is still a bit rough around the edges, but mostly does what we're after.
Diff Detail
Event TimelineComment Actions I think I'd ideally like to do this:
I don't know if we can figure out a silhouette approach that doesn't look junky, though. If we can't and/or you don't like that, a less aggressive approach might be:
That's a little less crazy than trying to do silhouette changes, and maybe better? I'm not happy with them right now; I'd like them to be more distinct than they are. Comment Actions Either of those color approaches would also let us vary the icons freely. I think it might be more natural to give "members(x)" a "users" icon than a "magic wand" icon. Comment Actions As I think about it I'm less into the silhouette idea. Let's just try the colors first. I'll do that in a followup. Comment Actions
Comment Actions I'm looking for "(" because I want this to count as a "function token" as the user types it out: members(x Specifically, that's sufficient for the "members" datasource to know that the user is calling the "members" function, and it should query for projects matching "x...". That said, I can replace that call with PhabricatorTypeaheadDatasource::isFunctionToken(). |