-
Notifications
You must be signed in to change notification settings - Fork 26
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
docs(accordion): implement and document vars and attrs #790
Conversation
Deploy preview for helix-ui ready! Built with commit 1a7b19c |
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.
DEV LGTM
@@ -7,10 +7,10 @@ hx-accordion.hxStepper { | |||
counter-reset: step; | |||
|
|||
hx-accordion-panel { | |||
border-top: 1px solid $gray-300; | |||
border-top: 1px solid var(--hxAccordian-panel-borderColor, $gray-300); |
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.
Please change to:
border-top: 1px solid var(--hxAccordian-stepper-panel-border-top, $gray-300);
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.
Changes have been done, please review.
|
||
&:last-child { | ||
border-bottom: 1px solid $gray-300; | ||
border-bottom: 1px solid var(--hxAccordian-child-borderColor, $gray-300); |
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.
Please change to:
border-bottom: 1px solid var(--hxAccordian-stepper-panel-border-bottom, $gray-300);
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.
Changes have been done, please review.
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.
Dev LGTM
Description
Implement and Document variables and attributes
What are the relevant story cards/tickets? Any additional PRs or other references?
Jira: SURF 2094
Before you request a review for this PR:
yarn test
to ensure all tests passed?