Page MenuHomePhabricator

Give typeahead browse dialogs sensible titles
ClosedPublic

Authored by epriestley on Apr 19 2015, 2:33 PM.
Tags
None
Referenced Files
F19554838: D12469.id29924.diff
Jan 30 2026, 3:59 AM
F19447636: D12469.id29924.diff
Dec 29 2025, 4:36 PM
F19095710: D12469.diff
Dec 4 2025, 5:53 AM
F18903781: D12469.id.diff
Nov 8 2025, 6:16 AM
F18877725: D12469.diff
Nov 6 2025, 11:38 AM
F18876584: D12469.diff
Nov 6 2025, 1:59 AM
F18780904: D12469.id29960.diff
Oct 12 2025, 8:31 PM
F18713909: D12469.id.diff
Sep 29 2025, 6:59 AM
Subscribers

Details

Summary

Ref T4100. Let datasources specify a more meaningful title than the class name.

Test Plan

Browsed some sources.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Give typeahead browse dialogs sensible titles.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
chad added a reviewer: chad.
chad added a subscriber: chad.
chad added inline comments.
src/applications/people/typeahead/PhabricatorViewerDatasource.php
7

Is this a browse interface of one item?

This revision is now accepted and ready to land.Apr 19 2015, 2:38 PM

Yeah, the viewer(), viewerprojects() and null() datasources all return only one item. Users can never normally hit these browse dialogs unless they're developing/debugging, though.

This revision was automatically updated to reflect the committed changes.