Homec4science

Move some PhabricatorSearchField subclasses

Authored by Joshua Spence <josh@joshuaspence.com> on Jul 6 2015, 14:52.

Description

Move some PhabricatorSearchField subclasses

Summary: Move some PhabricatorSearchField subclasses to be adjacent to the application to which they belong. This seems generally better to me than lumping them all together in the src/applications/search/field/ directory. I was also wondering if it makes sense to rename these subclasses as PhabricatorXSearchField rather than PhabricatorSearchXField (as per T5655), but wasn't really sure if these objects are meant to be search-fields, or just fields belonging to the #search application.

Test Plan: N/A.

Reviewers: #blessed_reviewers, epriestley

Reviewed By: #blessed_reviewers, epriestley

Subscribers: epriestley, Korvin

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

Details

Committed
Joshua Spence <josh@joshuaspence.com>Jul 6 2015, 14:52
Pushed
aubortJan 31 2017, 17:16
Parents
rPHab7dc13c67fd: Rebuild Lato fonts for Firefox
Branches
Unknown
Tags
Unknown

Event Timeline

Joshua Spence <josh@joshuaspence.com> committed rPHacb1eb81ccf4: Move some PhabricatorSearchField subclasses (authored by Joshua Spence <josh@joshuaspence.com>).Jul 6 2015, 14:52