-
Notifications
You must be signed in to change notification settings - Fork 413
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
Render special list items #323
Conversation
def list(text, list_type) | ||
elided = text.gsub!(ELIDED_LI_TOKEN, '') | ||
return if text =~ /\A\s*\Z/ && elided | ||
return text if text =~ /aside-title/ |
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.
this will break anything that has the literal text aside-title
in it...
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.
Sure, I made it check for a bit more.
Awesome! Thanks @jpsim and @segiddins! 😄 Screenshot looks great. You are much appreciated! |
@@ -26,6 +26,56 @@ $code_font: 0.95em Menlo, monospace; | |||
$gray_border: 1px solid #e2e2e2; | |||
$declaration_swift_border: 5px solid #cde9f4; | |||
|
|||
// Asides | |||
$aside_attention_color: #aaa; |
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.
probably worth making #aaa
a var too
Great. Thanks @jpsim for the really really fast implementation :) Happy to use jazzy for our documentation. |
Fixes #317. This PR should make it clear that I don't know how to SASS. I welcome suggestions on how to improve this implementation. Thoughts @orta @segiddins?
But at least it looks nice!