Skip to content

Fix colorization of nested list.#19559

Closed
katainaka0503 wants to merge 2 commits intomicrosoft:masterfrom
katainaka0503:fixColorizeNestedList
Closed

Fix colorization of nested list.#19559
katainaka0503 wants to merge 2 commits intomicrosoft:masterfrom
katainaka0503:fixColorizeNestedList

Conversation

@katainaka0503
Copy link
Contributor

Fixed colorization of nested list with unindented line.
Previously, bullets of nested list with empty line above was not colored.

This change related to #19417 which updated markdown-it version,
and after applied that, two consecutive newlines no longer terminate a list.

Fixed colorization of nested list with unindented line.
@mention-bot
Copy link

@katainaka0503, thanks for your PR! By analyzing the history of the files in this pull request, we identified @alexandrudima and @rebornix to be potential reviewers.

@msftclas
Copy link

Hi @katainaka0503, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by Microsoft and real humans are currently evaluating your PR.

TTYL, MSBOT;

@katainaka0503
Copy link
Contributor Author

katainaka0503 commented Jan 30, 2017

I am not sure what 509th line and 510th in extensions/markdown/syntaxes/markdown.tmLanguage mean.

<key>comment</key>
<string>Currently does not support un-indented second lines.</string>

If this comment means this

- aaa
(empty line)
    - bbb
(empty line)
    - ccc

is not properly colored,
maybe this comment should be deleted.

@msftclas
Copy link

Hi @katainaka0503, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by Microsoft and real humans are currently evaluating your PR.

TTYL, MSBOT;

@katainaka0503
Copy link
Contributor Author

I want to consider a little more firmly.
so I closed this PR.

@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants