-
Notifications
You must be signed in to change notification settings - Fork 0
Small update to UI on History page, phase 1 #11
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
base: main
Are you sure you want to change the base?
Conversation
kellenwiltshire
left a comment
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.
To avoid a massive PR with a lot of changes, I've done some ground work here for future work. Next up, I can work on the Fills History, Maintenance History, and Visual History component UI updates in separate PRs. Should allow faster iteration that way as well.
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.
Beginning stages of a clean sidebar for the History page
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.
Moved the components out of this file
| <body> | ||
| <Navbar /> | ||
| <main className='flex min-h-[90vh] items-center justify-center overflow-auto'> | ||
| <main className='font-montserrat-regular flex min-h-screen justify-center'> |
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.
Didn't mean to add this font as a class, it won't do anything, but will remove later
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.
Beginning stage of a slice for the History page
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.
These are jsx files for now so that the build completes. Plan is to have 3 separate PR's that addresses these files and converts them to updated UI with proper typescript
No description provided.