Feature/toast ui 코드 하이라이터 가능하도록 처리 #894
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- toast ui의 code syntax highlight 모듈 임포트 - code syntax의 테마 설정을 위해 prismjs 임포트 - toast ui의 code syntax highlight에서 모든 언어를 지원하는 js 파일을 타입스크립트로 선언 #468
- '@toast-ui/editor-plugin-code-syntax-highlight' 모듈을 임포트하여 codeSyntaxHighlight 사용 - 하이라이트 지원하는 언어 파일을 임포트 - Prism 테마 사용 #468
- '@toast-ui/editor-plugin-code-syntax-highlight' 모듈을 임포트하여 codeSyntaxHighlight 사용 - Prism 테마 사용 #468
- code syntax highlight 적용할 언어를 prismjs가 지원하는 선에서 임포트 - 임포트 해오는 언어 모듈이 많아 'prism-markup-templating' 모듈을 활용 #468
publdaze
requested changes
May 21, 2024
publdaze
reviewed
May 21, 2024
redzzzi
changed the title
redzzzi
changed the title
redzzzi
changed the title
publdaze
approved these changes
May 23, 2024
There was a problem hiding this comment.
👏👏👏 첫 기여 축하드립니다! 수고하셨어요~
-
toast ui 코드 하이라이터 가능하도록 처리 #468 이슈에 assignee 안 달려있던데, 달아주세용!
-
PR 내용에 이슈 앞에 Close 달아두면 PR 머지 시 자동으로 close 된답니다~! PR 머지되면 해결되는 이슈인 경우에
Close #468이런 식으로 작성해주면 따로 이슈 안 닫아도 돼서 편할 거에요 -
테스트 해보다가 버그 발견했는데(``` 후 엔터키 누르면 발생하는 것 같아요), 해당 PR으로 발생한 버그는 아닌 것 같아서 이 PR은 머지하고 새로 이슈 파서 버그 픽스 해보면 좋을 것 같아요~
2024-05-23.10.58.03.mov
이만 approve하고 물러가겠습니다~
|
아 제 PR 머지된 부분이랑 코드 변경사항 둘다 에디터쪽 겹쳐서 conflict 나는군요! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#468
연관 이슈
작업 요약
작업 상세 설명
리뷰 요구사항
node_modules/@types폴더 하위에@toast-ui폴더를 따로 만들어declare module을 해주었는데, 해당 방법이 최적의 방법인지 궁금합니다!Preview 이미지