Skip to content

[Issue #8863] Improve Application breadcrumbs#9027

Draft
andycochran wants to merge 1 commit intomainfrom
andycochran/8863--application-breadcrumbs
Draft

[Issue #8863] Improve Application breadcrumbs#9027
andycochran wants to merge 1 commit intomainfrom
andycochran/8863--application-breadcrumbs

Conversation

@andycochran
Copy link
Collaborator

@andycochran andycochran commented Mar 12, 2026

Summary

Fixes #8863

Changes proposed

  • Add breadcrumbs to the application list
  • Add breadcrumbs to the application view
  • Remove "Home" and add "Workspace > Applications" to the application form breadcrumbs
image image image

Context for reviewers

Per new guidance, all breadcrumbs are to be managed consistently, i.e. on their page (not in the constant file). This is part of a larger effort to refactor breadcrumbs. In a subsequent ticket, all Applications pages & subpages will be moved under /workspace. But that'll happen along with other route changes, when /dashboard will also change to /workspace.

Validation steps

  • Sign in
  • View /applications
    • The breadcrumbs should be "Workspace > Applications"
  • View a specific application
    • The breadcrumbs should be "Workspace > Applications > [applicationName]"
  • View a form in that application
    • The breadcrumbs should be "Workspace > Applications > [applicationName] > [formName]"
  • Make sure all the links for each page's breadcrumbs take you where you're supposed to go

@andycochran
Copy link
Collaborator Author

I might need help with this one. IDK why this has broken the applications tests and can't figure out how to resolve it. cc @ErinPattisonNava @doug-s-nava @mdragon @btabaska

@ErinPattisonNava
Copy link
Collaborator

ErinPattisonNava commented Mar 13, 2026

I might need help with this one. IDK why this has broken the applications tests and can't figure out how to resolve it. cc @ErinPattisonNava @doug-s-nava @mdragon @btabaska

@andycochran The Breadcrumb component relies on ReturnToGrantsNotification component, which assumes that searchParams exists in next's navigation. I wouldn't cut that deep though, I'd recommend creating a mock for the Breadcrumb component in the applications test file and adding a test that you are getting the parameters you expect. You can see an example of how to do the mock here and here is where the test of breadcrumb values for that page exists

@doug-s-nava Since breadcrumbs are such a core component, it might be worth building some utilities around these mocks so the code doesn't have to be copied everywhere.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve breadcrumbs on Application pages

2 participants