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

[ko] Animation performance and frame rate 신규 번역 #25320

Merged
merged 13 commits into from
Jan 14, 2025
Merged

Conversation

a-honey
Copy link
Contributor

@a-honey a-honey commented Jan 4, 2025

Description

성능 페이지에서 Animation_performance_and_frame_rate 페이지 신규 번역 했습니다.

Motivation

Additional details

Related issues and pull requests

@a-honey a-honey requested a review from a team as a code owner January 4, 2025 06:19
@a-honey a-honey requested review from wisedog and removed request for a team January 4, 2025 06:19
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Jan 4, 2025
Copy link
Contributor

github-actions bot commented Jan 4, 2025

Preview URLs

Flaws (2)

URL: /ko/docs/Web/Performance/Animation_performance_and_frame_rate
Title: 애니메이션 성능과 프레임 속도
Flaw count: 2

  • macros:
    • /ko/docs/Web/API/SVGAnimationElement does not exist but fell back to /en-US/docs/Web/API/SVGAnimationElement
    • /ko/docs/Web/CSS/left does not exist but fell back to /en-US/docs/Web/CSS/left
External URLs (2)

URL: /ko/docs/Web/Performance/Animation_performance_and_frame_rate
Title: 애니메이션 성능과 프레임 속도

(comment last updated: 2025-01-14 14:50:22)

@a-honey a-honey closed this Jan 4, 2025
@a-honey a-honey reopened this Jan 4, 2025
wisedog
wisedog previously requested changes Jan 9, 2025
Copy link
Contributor

@wisedog wisedog left a comment

Choose a reason for hiding this comment

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

와! 처음 기여를 해주셨는데, 너무 잘 해주셨습니다. 사소한 부분 몇 개만 수정하면 CI도 통과하고 머지할 . 수있을걸로 보입니다.

@a-honey
Copy link
Contributor Author

a-honey commented Jan 10, 2025

리뷰 감사합니다 👍
다만 CI가 통과하지 못하고 있는데 혹시 따로 이유가 있을까요..?
URL 포함해서 원본과 비교하면서 변경했는데 원인을 잘 못찾겠습니다 😢
@wisedog

@wisedog
Copy link
Contributor

wisedog commented Jan 11, 2025

리뷰 감사합니다 👍 다만 CI가 통과하지 못하고 있는데 혹시 따로 이유가 있을까요..? URL 포함해서 원본과 비교하면서 변경했는데 원인을 잘 못찾겠습니다 😢 @wisedog

제가 테스트로 PR 을 올려보았는데 문제 없이 통과합니다. 딱 하나 다른건 그림 파일을 올리신건데, 파일 빼고 올려보실 수 있으실까요? 파일은 따로 올리실 필요가 없다는 사실을 제가 깜빡했습니다 😭

@wisedog
Copy link
Contributor

wisedog commented Jan 12, 2025

@mdn/yari-content-ko 도와주십쇼 ㅠㅠ 이거 뭐가 잘못됐는지 저도 안보입니다.

@hochan222
Copy link
Member

@a-honey 님, @wisedog 님 안녕하세요.

마크다운 린트 오류로 보여집니다! 마크다운 린트 오류의 경우 nvm use && yarn 이후 markdownlint-cli2 --fix ./files/ko 명령어를 통해 ko 로케일 파일들에 대해 검사할 수 있습니다.

해당 명령어 수행 결과 아래와 같습니다.

image

index.md:64:97 ~ MD047/single-trailing-newline
64 line 끝에서 오류가 발생한 것으로 보아. 문서 마지막줄 개행이 없어서 발생한 이슈로 보여집니다.

아래 PR에 반영했습니다!

a7f641b

@hochan222
Copy link
Member

추가적인 이슈 없는 듯하여 병합하도록 하겠습니다.

모두 고생 많으셨습니다! 🙇🙇

image

@hochan222 hochan222 merged commit 0b6ad3b into mdn:main Jan 14, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n-ko Issues related to Korean content.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants