Page MenuHomePhabricator

Provide convenience method addTextCrumb() to PhabricatorCrumbsView
ClosedPublic

Authored by epriestley on Dec 18 2013, 7:34 PM.
Tags
None
Referenced Files
F19913691: D7787.id17617.diff
Fri, Apr 3, 4:50 AM
F19898262: D7787.id17617.diff
Sun, Mar 29, 11:31 AM
F19836698: D7787.id.largetrue.diff
Mar 11 2026, 1:14 PM
F19802650: D7787.id17617.diff
Mar 1 2026, 5:57 PM
F19777016: D7787.id17644.largetrue.diff
Feb 22 2026, 6:49 PM
F19672324: D7787.id.diff
Feb 8 2026, 3:13 AM
F19559091: D7787.id.diff
Jan 31 2026, 12:39 AM
F19559082: D7787.id.diff
Jan 31 2026, 12:35 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