-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Modified Kitchen Sink Page #10500
Modified Kitchen Sink Page #10500
Conversation
Added data-closable attribute to callout div
Thanks for the PR! This makes the callout closable, but to get it to close, we need to make the close button also have a |
Added data-close to button
Thanks, done that now and it works! |
docs/pages/kitchen-sink.md
Outdated
<div class="callout"> | ||
<button class="close-button" aria-label="Close alert" type="button"> | ||
<div class="callout" data-closable> | ||
<button class="close-button" aria-label="Close alert" type="button" data-close> | ||
<span aria-hidden="true">×</span> | ||
</button> | ||
<p>This is a static close button example.</p> |
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.
I think you need to remove the word static
? No ?
Removed static
Removed the word static. |
Perfect! Merged, thanks @danclayton50! |
Added data-closable attribute to closable callout div so it closes properly
That's all!