feat(frontend): Add start page with quick actions to Admin Center #7995
Conversation
…pui-add-startpage-to-admin
✅ Deploy Preview for inventree-web-pui-preview ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #7995 +/- ##
==========================================
+ Coverage 87.80% 87.86% +0.05%
==========================================
Files 1277 1276 -1
Lines 57503 57461 -42
Branches 2023 2023
==========================================
- Hits 50493 50488 -5
+ Misses 6510 6473 -37
Partials 500 500
🚀 New features to boost your workflow:
|
…pui-add-startpage-to-admin
|
@SchrodingersGat ready for review and merge |
- Use responsive grid instead of carousel - Add icons - Translate text
|
@matmair looks great in general, I have made some minor tweaks to ensure that the text is translated, and to more closely match the visual style of the rest of the admin center: |
| const items = [ | ||
| { | ||
| id: '0', | ||
| title: t`Open an Issue`, |
There was a problem hiding this comment.
Are you sure it is a good idea to have this here forever? In my original intend this message was just part of the tipp to direct first impressions
| )); | ||
|
|
||
| return ( | ||
| <SimpleGrid |
There was a problem hiding this comment.
Will this approach scale when we have 20-30 quick actions? Just based on the amount of quick things we have in the various panels right now.
The idea with the carousel was that it will always take up the same amount of space
There was a problem hiding this comment.
The issue with the carousel is that you have to navigate left / right to find the action you want - not exactly a "quick" action
| <Title order={5}> | ||
| <Trans>System status</Trans> | ||
| </Title> | ||
| <StylishText size='lg'> |
There was a problem hiding this comment.
I find this heavy use of accent colors fatiguing to the eye compared to a normal bold header
There was a problem hiding this comment.
I'd be happy to move to normal bold header for accordion headings - this was simply to bring in line with other rendering
This PR adds a "start page" to the Admin Center.
Closes invenhost#108
Closes invenhost#112
Closes invenhost#113
To-do:
Later?