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.

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: bug fix for Show HTML tag on TOC, not escaping them #1740

Closed
wants to merge 1 commit into from

Conversation

gimdongwoo
Copy link
Contributor

@gimdongwoo gimdongwoo commented Aug 4, 2019

Motivation

Long time no see.
I found very easy bug on issue.

But, this PR cause broken old style. And I don't know all side effect.
We need a explicitly way to show the HTML sentence or add the HTML tag in the TOC.
If you suggest a good way, I will update this PR.

It's simple training for me.
I want more contribution for V2 from now.
Because I love TypeScript & Docusarus.

Have you read the Contributing Guidelines on pull requests?

Yes.

Test Plan

It's simple change.

Related PRs

Nope.

@facebook-github-bot facebook-github-bot added the CLA Signed Signed Facebook CLA label Aug 4, 2019
@gimdongwoo gimdongwoo changed the title fix: bug fix for Show HTML tag on TOC, not escaping them #1703 fix: bug fix for Show HTML tag on TOC, not escaping them Aug 4, 2019
@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-2 ready!

Built with commit ca0c81d

https://deploy-preview-1740--docusaurus-2.netlify.com

@docusaurus-bot
Copy link
Contributor

Deploy preview for docusaurus-preview ready!

Built with commit ca0c81d

https://deploy-preview-1740--docusaurus-preview.netlify.com

@wgao19
Copy link
Contributor

wgao19 commented Aug 5, 2019

lgtm @endiliey
closing #1703 if landed

@endiliey
Copy link
Contributor

endiliey commented Aug 5, 2019

After another thought, I'm afraid that many people thought this is intended behavior and actually v2 had this problem as well.

But, this PR cause broken old style. And I don't know all side effect.

yep Because we are showing html tag on right toc

I personally think #1703 is very rare and the fix itself could break someone's site 😮
Leaving it to @wgao19 and @yangshun. I wont merge this 😃 ,

Copy link
Contributor

@endiliey endiliey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just gonna stamp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Signed Facebook CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants