-
Notifications
You must be signed in to change notification settings - Fork 117
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
frontend: Refactor workflows to use new layout and spacing units #3156
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looking good, lets see if we can break up this PR into 2 so that we can reduce the scope that this one impacts.
layoutProps: { | ||
variant: "standard", | ||
title: "Project Catalog", | ||
subtitle: "A catalog of all projects.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this just utilize the description versus a secondary subtitle?
Description
Screenshots
Standard Layout
Before
After
Note
New props were added to the layout to display only the breadcrumbs and preserve previous route component header
Wizard Layout
Before
After
Custom Layout
Before
After
Testing Performed
Manual
TODO