-
Notifications
You must be signed in to change notification settings - Fork 159
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(content-section): web-component version #4275
feat(content-section): web-component version #4275
Conversation
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 progress @IgnacioBecerra!
packages/web-components/src/components/content-section/content-section.ts
Outdated
Show resolved
Hide resolved
packages/web-components/src/components/content-section/content-section.ts
Outdated
Show resolved
Hide resolved
packages/web-components/src/components/content-section/content-section.ts
Outdated
Show resolved
Hide resolved
packages/web-components/src/components/content-section/content-section.ts
Outdated
Show resolved
Hide resolved
packages/web-components/src/components/content-section/content-section.ts
Outdated
Show resolved
Hide resolved
Deploy preview created for package Built with commit: ea0211aae114dfb5a8c517f41e086ca3e3b57fa2 |
Deploy preview created for package Built with commit: ea0211aae114dfb5a8c517f41e086ca3e3b57fa2 |
Content Section is only an internal pattern used for CardSection.
@asudoh Thank you for your comments, the changes are now present in the current commit. |
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.
LGTM 👍 - Thanks @IgnacioBecerra for the update!
Related Ticket(s)
#3736
Description
Created the internal component to use within the future Web Components version of
CardSection
.Changelog
New