-
Notifications
You must be signed in to change notification settings - Fork 310
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
Add Panel component #6809
Add Panel component #6809
Conversation
387e270
to
38192d8
Compare
81a5616
to
79ec796
Compare
pkg/webui/components/shortcut-panel/shortcut-item/shortcut-item.styl
Outdated
Show resolved
Hide resolved
pkg/webui/components/shortcut-panel/shortcut-item/shortcut-item.styl
Outdated
Show resolved
Hide resolved
pkg/webui/components/shortcut-panel/shortcut-item/shortcut-item.styl
Outdated
Show resolved
Hide resolved
@ryaplots I have added here a review on the UI of the panel: https://www.figma.com/file/7pBLWK4tsjoAbyJq2viMAQ/console-redesign?type=design&node-id=2973%3A8689&mode=design&t=xOwdF36i5TRHqifD-1 |
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.
Code LGTM but please rerequest review after addressing @pierrephz's comments.
Co-authored-by: Kevin Schiffer <[email protected]>
…/lorawan-stack into feature/panel-component
I've pushed a commit that refactors these replacement icons as part of the |
36356bc
to
a9c609b
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.
Thank you! Please note the remaining comments. I've pushed the fixes so we can merge this now.
|
||
&-active | ||
color: var(--c-text-neutral-min) | ||
box-shadow: 0px 1px 2px 0px rgba(0, 0, 0, 0.05) |
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.
This shadow is virtually invisible (even with my design-sensitive eye). It maps to the button-shadow
on Figma, which is equally invisible. Is this intentional @pierrephz ?
message={label} | ||
value={value} | ||
onClick={onToggleChange} | ||
unstyled |
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.
unstyled |
The unstyled prop has been removed and is now the default when no primary/secondary/naked/etc is set.
Summary
This PR adds new components to build the various panels.
Changes
Testing
Run storybook
Notes for Reviewers
Checklist
README.md
for the chosen target branch.CHANGELOG.md
.CONTRIBUTING.md
, there are no fixup commits left.