-
Notifications
You must be signed in to change notification settings - Fork 902
feat: add audit page title, subtitle, and CLI snippet #3419
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
Conversation
body: "An audit log is a record of events and changes made throughout a system.", | ||
} | ||
|
||
export const AuditHelpTooltip: FC = () => { |
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.
Good use of component nesting here (tooltips)
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.
Nice work!
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.
Nice!
The small viewport story doesn't have a small viewport for me, do I need to configure anything? |
@presleyp hmmm, are you seeing something weird in Chromatic CI (if so, can you post a screenshot?) or do you mean you're having trouble when running locally? The Chromatic CI snapshot looks right to me. In a smaller viewport, the code block containing the CI command can be found under the page title and subtitle as opposed to across from it. |
I meant Chromatic CI but actually now it looks fine! |
resolves #3357
We've begun to scaffold out the Audit Page. The CLI command widget may fluctuate in design down the road; right now, I just wanted to get something in there for MVP, so I'm using our existing
CodeExample
component. The CLI command is a placeholder rn as we figure out what we actually want.Large Screens
Small Screens