-
Notifications
You must be signed in to change notification settings - Fork 19
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
fix: use span instead of div for icon #22
Conversation
@elviswolcott could you please help me with the tests? I can not figure out why they do not pass. |
@@ -9,35 +9,31 @@ | |||
<h1>This is a test</h1> | |||
<div class="admonition admonition-note alert alert--secondary"> | |||
<div class="admonition-heading"> | |||
<h5> | |||
<div class="admonition-icon">ℹ️</div>note</h5> | |||
<h5><span class="admonition-icon">ℹ️</span>note</h5> |
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.
Because the tests are really flaky the difference in whitespace causes them to fail.
<h5> | ||
<div class="admonition-icon"> | ||
<svg xmlns="http://www.w3.org/2000/svg" width="14" height="16" viewBox="0 0 14 16"> | ||
<h5><span class="admonition-icon"><svg xmlns="http://www.w3.org/2000/svg" width="14" height="16" viewBox="0 0 14 16"> |
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.
Same deal here with whitespace
Travis is refusing to even boot a VM for the branch build, but the PR is passing so I'll merge and release. |
1.2.1 is published 🎉 Thanks for the PR! |
From this audit facebook/docusaurus#2403
cc @elviswolcott