Created admin login UI component#15
Conversation
|
Graphite Automations"Request reviewers once CI passes" took an action on this PR • (05/28/26)2 reviewers were added to this PR based on Henry Chen's automation. |
| logo | ||
| </div> | ||
| <h1 style={{ fontSize: "36px", fontWeight: 700, color: "black" }}>PatChats</h1> | ||
| </header> |
There was a problem hiding this comment.
The header should be broken out into its own component. It should also likely be reused across pages.
You can look at js/src/components/ui/page/PageShell.tsx in codebloom to see how it's done there.
We don't have to go to that extent right now, but yeah.
| width: "250px", | ||
| }} | ||
| > | ||
| <h2 |
There was a problem hiding this comment.
Use the Title component in mantine for html headers.
| }; | ||
|
|
||
| return ( | ||
| <div |
There was a problem hiding this comment.
Use Mantine Box and Flex, which wrap the basic HTML divs.
| style={{ | ||
| minHeight: "100vh", | ||
| width: "100vw", | ||
| backgroundColor: "white", |
There was a problem hiding this comment.
You shouldn't need to set the color of the entire page here. We can just use the theming to do it.
| size="sm" | ||
| styles={{ | ||
| root: { | ||
| backgroundColor: "#a94700", |
There was a problem hiding this comment.
We should probably just lean into the provided styles from the mantine theme.
There was a problem hiding this comment.
The AdminLogin should be its own page. Admin.page.tsx should be the Admin home page.
There was a problem hiding this comment.
The admin stuff shouldn't be under js/src/app/user/admin, but should be under js/src/app/admin
2 participants
Created the admin login UI for the
/adminroute.Testing
Verified
/adminrenders the admin login UI:Verified empty submit shows validation errors:
Verified valid submit reaches the placeholder handler: