Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Sub-list items which are multiples of 3 are wrapped with pre/code blocks in version 4.0.6 #2314

Closed
udara28 opened this issue Dec 8, 2021 · 3 comments · Fixed by #2318 · May be fixed by Omrisnyk/goof-jenkins#21
Closed

Sub-list items which are multiples of 3 are wrapped with pre/code blocks in version 4.0.6 #2314

udara28 opened this issue Dec 8, 2021 · 3 comments · Fixed by #2318 · May be fixed by Omrisnyk/goof-jenkins#21
Labels
category: lists L1 - broken Valid usage causes incorrect output OR a crash AND there is no known workaround for the issue released

Comments

@udara28
Copy link

udara28 commented Dec 8, 2021

Marked version:
4.0.6

Describe the bug
Every sublist item which is a multiple of 3 is wrapped with a <pre><code></code></pre> block. This issue is present in the 4.0.6(latest) version but not in 4.0.5.

To Reproduce
A hosted test case to reproduce this issue can be found in this CodePen Demo

Example markdown text to reproduce the bug:

1. list item one
    1. sublist item one
    2. sublist item two

2. list item two
    1. sublist item one
    2. sublist item two

3. list item three
    1. sublist item one
    2. sublist item two

4. list item four
    1. sublist item one
    2. sublist item two

Expected behavior
<pre><code></code></pre> should not appear in the sub list items.
@UziTech UziTech added category: lists L1 - broken Valid usage causes incorrect output OR a crash AND there is no known workaround for the issue labels Dec 9, 2021
@UziTech
Copy link
Member

UziTech commented Dec 9, 2021 •
edited

confirmed demo

@calculuschild
Copy link
Contributor

I think I know where the error is.

calculuschild added a commit to calculuschild/marked that referenced this issue Dec 9, 2021
@calculuschild
Fix markedjs#2314
08b74d5
@calculuschild calculuschild mentioned this issue Dec 9, 2021
Merged
5 tasks
styfle pushed a commit that referenced this issue Dec 9, 2021
@calculuschild
fix: Fix every third list item broken (#2318)
346b162 
* Fix #2314
github-actions bot pushed a commit that referenced this issue Dec 9, 2021
@semantic-release-bot
chore(release): 4.0.7 [skip ci]
9b5e264 
## [4.0.7](v4.0.6...v4.0.7) (2021-12-09)

### Bug Fixes

* Fix every third list item broken ([#2318](#2318)) ([346b162](346b162)), closes [#2314](#2314)
@github-actions
Copy link

🎉 This issue has been resolved in version 4.0.7 🎉

The release is available on:

Your semantic-release bot 📦🚀

@sync-by-unito sync-by-unito bot mentioned this issue Jan 22, 2022
Closed
@snyk-bot snyk-bot mentioned this issue Dec 29, 2022
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
category: lists L1 - broken Valid usage causes incorrect output OR a crash AND there is no known workaround for the issue released
Projects
None yet
Milestone
No milestone
3 participants
@UziTech @udara28 @calculuschild