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 render error in Chrome browser & Fix markdown syntax error in tip section #219

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

hojongs
Copy link

@hojongs hojongs commented May 18, 2021

caused by markdown syntax error in tip section

Overview

조건부 렌더링 페이지 수정

  • markdown syntax error를 수정하여 크롬 브라우저에서의 렌더링 에러를 해결합니다 51ea153 0268c55

이벤트 핸들링 페이지 수정

폼 입력 바인딩 페이지 수정

컴포넌트 페이지 수정

Detail

#218 과 같은 맥락의 변경사항입니다만, 조건부 렌더링 페이지의 하단부가 Chrome browser에서 아예 노출되지 않는 버그가 있어 수정합니다.

image

Discussion

지난 PR에서 언급했듯, vuejs.org repository 코드상에도 같은 이슈가 존재합니다
하지만 이번 PR에서 변경하는 부분은 상황이 조금 다릅니다
이번 PR을 작업하며 확인해보니 vuejs.org 에서는 모든 페이지에서 같은 markdown이 kr과 다르게 렌더링됩니다

  • 영어/한국어를 제외하면 동일한 markdown 문법을 사용함
  • github markdown에서는 양쪽다 이전 PR처럼 제대로 rendering되지 않음
  • 하지만, https://vuejs.org/v2/guide/conditional.html 페이지에서는 같은 markdown 문법이 제대로 렌더링됨 (kr 사이트와는 달리)

vuejs.org
image
kr.vuejs.org
image

  • 이 이유를 알 수 있다면 좀더 효율적인 변경이 가능할 것으로 보입니다
  • hexo 설정 등에서 다른 부분이 있는 걸까요?

@hojongs
Copy link
Author

hojongs commented May 19, 2021

e12633e 8cb6346 67d5601 커밋은 markdown syntax 교정이 아닌, 번역 성격의 변경사항입니다.

필요에 따라 해당 커밋들만 별도 PR로 분리하거나 이 PR 이름을 변경해도 좋을 것 같습니다.

@hojongs hojongs changed the title Fix render error in Chrome browser caused by markdown syntax error in… Fix 이벤트 핸들링 페이지 render error in Chrome browser May 19, 2021
@hojongs hojongs changed the title Fix 이벤트 핸들링 페이지 render error in Chrome browser Fix render error in Chrome browser & Fix markdown syntax error in tip section May 19, 2021
Copy link
Member

@ChangJoo-Park ChangJoo-Park left a comment

Choose a reason for hiding this comment

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

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

Successfully merging this pull request may close these issues.

None yet

2 participants