Page MenuHomePhabricator

Provide convenience method addTextCrumb() to PhabricatorCrumbsView
ClosedPublic

Authored by epriestley on Dec 18 2013, 7:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 3:26 PM
Unknown Object (File)
Fri, Dec 13, 9:02 PM
Unknown Object (File)
Tue, Dec 10, 7:11 PM
Unknown Object (File)
Tue, Dec 10, 2:31 AM
Unknown Object (File)
Wed, Dec 4, 1:16 AM
Unknown Object (File)
Tue, Dec 3, 6:18 PM
Unknown Object (File)
Nov 26 2024, 6:46 PM
Unknown Object (File)
Nov 23 2024, 6:04 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

Lint
Lint Skipped
Unit
Tests Skipped