-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Remove {{page}} macro from API docs #3913
Conversation
f955892
to
b018afc
Compare
Preview URLs
FlawsURL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
URL:
External URLsURL: No new external URLs URL:
URL: No new external URLs URL: No new external URLs URL:
URL: No new external URLs URL:
URL: No new external URLs URL:
URL:
URL:
URL:
URL: No new external URLs (this comment was updated 2022-02-20 11:40:37.159411) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
기여 감사드립니다. 리뷰드린 내용 확인 부탁드립니다!
|
||
<p>Methods inherit from {{domxref("EventTarget")}}</p> | ||
|
||
<p>{{page("/en-US/docs/Web/API/EventTarget","Methods")}}</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
원본에서는 첫 문자가 대문자인 Page 매크로를 사용해서 내용을 보여주고 있는데 이 내용도 같이 지워야하는 매크로인가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
매크로는 대소문자 구분을 하지 않습니다. {{Page}}
와 {{page}}
모두 제거 대상입니다. 원문에 있는 것을 제거해야겠네요.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM👍 수고하셨습니다!
#3897
{{page}}
제거