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 Markdown syntax #663
Fix Markdown syntax #663
Conversation
Thanks for finding this bug, @yochem. The fix would be to change: *[Normalize.css v1 provides legacy browser support]
(https://github.com/necolas/normalize.css/tree/v1) to *[Normalize.css v1 provides legacy browser support](https://github.com/necolas/normalize.css/tree/v1) If you make that change, I’d happily accept this PR. Thanks again for spotting this and submitting a fix. |
Done! It should be fine to merge it. |
Is this intentional? |
Yeah, as @KIPdeKIP mentions, I think the correction should be:
or even
so the output looks like
|
Thank you for adding the italicization. Note that it's still included in the browser support list. It's currently:
But both the original and the current state on
I'd like the opinion of one of the normalize.css people on how it's supposed to be formatted though. |
Thanks @yochem! |
* 'master' of https://github.com/necolas/normalize.css: (35 commits) 8.0.1 Render the main element correctly in IE Remove stylelint Fix typo in CHANGELOG v8.0.0 Update CHANGELOG Don't remove search input cancel button in Chrome/Safari Add package-lock.json Remove bower Update browser support Point to yarnpkg's list of CDNs Remove the font-size test on html tag Inherit font family for inputs Restore gaps in links underline in iOS 8+ and Safari 8+ 7.0.0 Revert changes to 'body' and form elements Non-functional maintenance Fix Markdown syntax (necolas#663) Correct the padding in firefox (necolas#671) Update CHANGELOG.md (necolas#670) ...
I think the list misses one.
Changed
into
(weird, the preview doesn't show the italic properly? It looks normal on the markdown preview in Atom.)