Page MenuHomePhabricator

Expand typeahead document copy
ClosedPublic

Authored by chad on Apr 27 2015, 4:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 9:38 AM
Unknown Object (File)
Tue, Apr 9, 11:10 PM
Unknown Object (File)
Sat, Apr 6, 3:01 AM
Unknown Object (File)
Tue, Apr 2, 7:51 AM
Unknown Object (File)
Mon, Apr 1, 3:47 PM
Unknown Object (File)
Mon, Apr 1, 11:31 AM
Unknown Object (File)
Mar 6 2024, 7:02 PM
Unknown Object (File)
Mar 6 2024, 7:01 PM
Subscribers

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rP12dc865a00f9: Expand typeahead document copy
Summary

Seems reasonable? At least, it always matches however a user might think about documents (app or document). Unclear if "Diffusion" for example, are actually needed.

Test Plan

tested searching for "phriction", "wiki", "document", etc.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Expand typeahead document copy.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

These are used in a handful of other places so it's possible some of them will look weird and we'll want to add getShortTypeName() + getFullTypeName() or something, but the other cases all seem like they're probably fine to me from a quick search. This is definitely a lot clearer in global search.

This revision is now accepted and ready to land.Apr 27 2015, 5:04 PM
This revision was automatically updated to reflect the committed changes.