-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
details shortcode: add example of both alternatives #783
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
base: main
Are you sure you want to change the base?
Conversation
readme: mention steps shortcode, move katex alpha order steps: use 1. instead of fixed numbers
|
no idea why the checks are failing , works fine locally with huge 0.151.1 |
| keyword or set `open=false`. | ||
|
|
||
| ## Example | ||
| ## Example 1 |
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 it would be better as ## Example with positional arguments
| Lorem markdownum insigne... | ||
| {{% /details %}} | ||
|
|
||
| ## Example 2 |
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 here, I think it would be better as ## Example with parameters
| ... | ||
| 2. ## Maecenas scelerisque sem. | ||
| 1. ## Maecenas scelerisque sem. |
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.
Please revert this file :)
| Suspendisse sed congue orci, eu congue metus. Nullam feugiat urna massa, et fringilla metus consectetur molestie. Curabitur pellentesque sodales ipsum, sed efficitur libero euismod ac. Donec sit amet erat nunc. Suspendisse porta nisl velit, quis auctor massa commodo nec. Donec sollicitudin tellus sit amet massa condimentum luctus. Etiam molestie at ante et convallis. | ||
|
|
||
| 2. ## Maecenas scelerisque sem. | ||
| 1. ## Maecenas scelerisque sem. |
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.
Please revert this file :)
|
Left few comments, otherwise looks good 👍 |
readme: mention steps shortcode, move katex alpha order
steps: use 1. instead of fixed numbers