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

Hotfix/#288 스타일 적용된 태그에 스페이스만 남아있을 때 에러나는 문제 #289

Conversation

Ludovico7
Copy link
Collaborator

📝 변경 사항

  • 스타일 적용된 태그에 스페이스만 남아있을 때 에러나는 문제

🔍 변경 사항 설명

  • 제 자신과의 싸움이었습니다...
  • 기존 로직에서 input 실행되기 전에 빈 span태그를 삭제하는 로직이 있어서 TreeWalker에서 해당 태그를 탐지하지 못하는 문제였습니다.

🙏 질문 사항

📷 스크린샷 (선택)

✅ 작성자 체크리스트

  • Self-review: 코드가 스스로 검토됨
  • Unit tests 추가 또는 수정
  • 로컬에서 모든 기능이 정상 작동함
  • 린터 및 포맷터로 코드 정리됨
  • 의존성 업데이트 확인
  • 문서 업데이트 또는 주석 추가 (필요 시)

@Ludovico7 Ludovico7 added Bug 기능 또는 UI에서 발생한 오류나 예상치 못한 동작을 해결 FE 프론트엔드 작업 P1 긴급/즉시 반영 labels Dec 4, 2024
@Ludovico7 Ludovico7 self-assigned this Dec 4, 2024
Copy link
Collaborator

@minjungw00 minjungw00 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
Member

@pipisebastian pipisebastian left a comment

Choose a reason for hiding this comment

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

자신과의 싸움에서 승리하셨군요... 고생하셨습니다!!

@github-actions github-actions bot merged commit db95d3f into dev Dec 4, 2024
5 checks passed
Copy link
Collaborator

@hyonun321 hyonun321 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

const escapeMap: Record<string, string> = {
"<": "&lt;",
">": "&gt;",
"&": "&amp;",
'"': "&quot;",
"'": "&#x27;",
" ": "&nbsp;",
Copy link
Collaborator

Choose a reason for hiding this comment

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

고생하셨습니다!
&nbsp; 는 “Non-breaking Space”를 의미하는 HTML entity 인데요. 한국말로 번역을 해보자면 “줄바꿈이 일으키지 않는 공백이라고 하는군요..!
위 replace요소가 추가되고, span 태그로 바꾸는 전처리 과정을 빼니, 안정적으로 탐지할 수 있게 되었군요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 기능 또는 UI에서 발생한 오류나 예상치 못한 동작을 해결 FE 프론트엔드 작업 P1 긴급/즉시 반영
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants