Page MenuHomePhabricator

Provide convenience method addTextCrumb() to PhabricatorCrumbsView
ClosedPublic

Authored by epriestley on Dec 18 2013, 7:34 PM.
Tags
None
Referenced Files
F13086233: D7787.diff
Thu, Apr 25, 12:07 AM
Unknown Object (File)
Thu, Apr 18, 5:37 PM
Unknown Object (File)
Mon, Apr 8, 3:36 AM
Unknown Object (File)
Fri, Apr 5, 9:07 PM
Unknown Object (File)
Fri, Apr 5, 7:29 AM
Unknown Object (File)
Mar 17 2024, 8:56 PM
Unknown Object (File)
Mar 10 2024, 1:17 PM
Unknown Object (File)
Jan 28 2024, 3:01 AM
Subscribers

Details

Summary

We currently have a lot of calls to addCrumb(id(new PhabricatorCrumbView())->...) which can be expressed much more simply with a convenience method. Nearly all crumbs are only textual.

Test Plan
  • This was mostly automated, then I cleaned up a few unusual sites manually.
  • Bunch of grep / randomly clicking around.

Diff Detail

Branch
crumb
Lint
Lint Passed
Unit
Tests Passed