-
Notifications
You must be signed in to change notification settings - Fork 30
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
feat: refactor side panels and enabled overlay panels #698
Conversation
Demo starting at https://lxd-ui-698.demos.haus |
06bea0b
to
04bd460
Compare
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.
The composable SidePanel
is super nice. Some ideas on structure and naming things below.
04bd460
to
a3c38bd
Compare
2e9ce86
to
b812857
Compare
Signed-off-by: Mason Hu <mason.hu@canonical.com>
b812857
to
6cef4fd
Compare
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.
Now it's perfect, thanks :)
Done
DetailPanel
andAside
components and refactored them into a compound componentSidePanel
InstanceDetailPanel
andProfileDetailPanel
to useSidePanel
QA