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

제목 태그의 내용에 영어 내용을 포함한 <> 기호가 있으면 에러가 발생합니다. #10

Open
QQyukim opened this issue Dec 3, 2020 · 3 comments

Comments

@QQyukim
Copy link
Contributor

QQyukim commented Dec 3, 2020

안녕하세요. Tistory TOC 관련 글을 찾다가 wbluke 님의 글을 보고 TOC를 적용하고 나타나는 오류를 보고 드립니다.

제가 주로 h1~h3 제목 태그의 내용을 ''와 같이 영어 내용을 포함한 <> 기호 형식으로 적는데,
이러한 형식은 TOC에서 아래 이미지에 나와있듯이 <a ...><영어 내용></영어 내용></a> 으로 나타납니다.
이는 올바른 코드 형식이 아니라 화면에서 보이지 않게 됩니다.

image

위 현상을 확인하실 수 있는 링크는 https://qqyukim.tistory.com/48 입니다.

제목 태그에 <> 기호를 쓰지 않으면 나타나지 않을 버그지만 수정하면 좋을 것 같아 이슈를 남깁니다.
fork 하여 해당 이슈를 수정한 뒤에 PR을 드려도 될까요?

그럼 오늘도 좋은 하루 보내시기 바랍니다. 👍🏻

@QQyukim QQyukim changed the title 제목 태그의 내용에 영어 내용을 포함한 꺾쇠 괄호가 있으면 에러가 발생합니다. 제목 태그의 내용에 영어 내용을 포함한 꺽쇠 괄호가 있으면 에러가 발생합니다. Dec 4, 2020
@QQyukim QQyukim changed the title 제목 태그의 내용에 영어 내용을 포함한 꺽쇠 괄호가 있으면 에러가 발생합니다. 제목 태그의 내용에 영어 내용을 포함한 <> 기호가 있으면 에러가 발생합니다. Dec 4, 2020
@QQyukim
Copy link
Contributor Author

QQyukim commented Dec 4, 2020

안녕하세요, wbluke님.
해당 이슈를 해결하기 위해 dev-toc.js 파일의 createBasicItemBy 함수에 다음과 같이 부등호 기호를 HTML 특수문자로 바꾸는 코드를 추가하였습니다.

if (hTag.innerText.includes('<')) {
    hTag.innerText = hTag.innerText.replace(/&lt;/g, '&amp;lt;');
    hTag.innerText = hTag.innerText.replace(/</g, '&lt;');
}
        
if (hTag.innerText.includes('>')) {
    hTag.innerText = hTag.innerText.replace(/&gt;/g, '&amp;gt;');
    hTag.innerText = hTag.innerText.replace(/>/g, '&gt;');
}

최신 커밋사항에 bug-fix 라는 브랜치를 생성하여 커밋했고 푸시하여 PR을 보내고 싶은데 Permission denied 메세지가 떠서 불가능합니다. PR로 프로젝트에 기여하고 싶은데 어떻게 하면 될 지 조언 부탁드립니다. 감사합니다.

@wbluke
Copy link
Owner

wbluke commented Dec 4, 2020

안녕하세요 @QQyukim 님! 먼저 해당 라이브러리에 관심 가져주시고 사용해주셔서 감사합니다.
https://github.com/QQyukim/tistory-table-of-contents/tree/master
QQyukim님 계정의 repository를 살펴보았는데요, 말씀해주신 브랜치가 보이지 않아서요.
개인 계정 repo에 푸시하셔야 PR 생성이 가능할 듯 보입니다 :)

PR은 develop 브랜치로 보내주시면 확인해보겠습니다! 감사합니다!

@QQyukim
Copy link
Contributor Author

QQyukim commented Dec 4, 2020

앗 제가 PR 과정을 오인하고 제가 fork한 뒤에 수정사항 커밋 후 푸시하고나서 PR을 드려야 하는데, wbluke님의 프로젝트를 clone 해가서 거기서 커밋하고 푸시하려고 했었습니다. 😅 제가 제대로 PR을 해보는 건 처음이라 좀 허둥지둥했어요. 곧 PR 보내드리겠습니다! 감사합니다.

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

No branches or pull requests

2 participants