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: user cannot insert the $$ text due to custom inline syntax #1791

Merged
merged 4 commits into from
Sep 7, 2021

Conversation

js87zz
Copy link
Contributor

@js87zz js87zz commented Sep 6, 2021

Please check if the PR fulfills these requirements

  • It's the right issue type on the title
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes/features)
  • Docs have been added/updated (for bug fixes/features)
  • It does not introduce a breaking change or has a description of the breaking change

Description

as-is
images

to-be
스크린샷 2021-09-06 오후 12 16 24


Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

Copy link

@adhrinae adhrinae left a comment

Choose a reason for hiding this comment

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

리뷰 완료합니다.

try {
return customConvertor!(node, context);
} catch (e) {
console.warn(e);
Copy link

Choose a reason for hiding this comment

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

라이브러리이니만큼 약간은 메세지를 데코레이션 해줘야 하지 않으려나요?
"[tui.editor] 무슨 컨버터 써서 문법 변환하다 오류 났다" 같은 느낌으로요.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아래처럼 변경했습니다~

[@toast-ui/editor] - The error occurred when ${info} inline node was parsed in markdown renderer: ${e}

Copy link
Contributor

@jwlee1108 jwlee1108 left a comment

Choose a reason for hiding this comment

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

리뷰 완료합니다.

Copy link
Contributor

@lja1018 lja1018 left a comment

Choose a reason for hiding this comment

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

리뷰완료합니다.

@js87zz js87zz merged commit 6683ae1 into master Sep 7, 2021
@js87zz js87zz deleted the fix/custom-inline branch September 7, 2021 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants