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] double colon placeholder 번역 #22925

Merged
merged 3 commits into from
Oct 27, 2024

Conversation

givvemee
Copy link
Contributor

@givvemee givvemee commented Aug 5, 2024

Description

기존에 한국어로 존재하지 않던 문서 ::placeholder 를 번역합니다.

Motivation

Additional details

Related issues and pull requests

@givvemee givvemee requested a review from a team as a code owner August 5, 2024 02:46
@givvemee givvemee requested review from sunhpark42 and removed request for a team August 5, 2024 02:46
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Aug 5, 2024
Copy link
Contributor

github-actions bot commented Aug 5, 2024

Preview URLs

Flaws (8)

URL: /ko/docs/Web/CSS/::placeholder
Title: ::placeholder
Flaw count: 8

  • macros:
    • /ko/docs/Web/HTML/Element/input does not exist but fell back to /en-US/docs/Web/HTML/Element/input
    • /ko/docs/Web/HTML/Element/textarea does not exist but fell back to /en-US/docs/Web/HTML/Element/textarea
    • /ko/docs/Web/CSS/::first-line does not exist but fell back to /en-US/docs/Web/CSS/::first-line
    • /ko/docs/Web/HTML/Element/input does not exist but fell back to /en-US/docs/Web/HTML/Element/input
    • /ko/docs/Web/HTML/Element/input does not exist but fell back to /en-US/docs/Web/HTML/Element/input
    • and 3 more flaws omitted
External URLs (3)

URL: /ko/docs/Web/CSS/::placeholder
Title: ::placeholder

(comment last updated: 2024-10-04 14:15:17)

Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@github-actions github-actions bot added the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Aug 10, 2024
Copy link
Contributor

@sunhpark42 sunhpark42 left a comment

Choose a reason for hiding this comment

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

안녕하세요 @givvemee 님 리뷰가 많이 늦어 죄송합니다 ㅠㅠ
수정 필요한 부분이 있어 코멘트 남겨두었습니다. 확인 부탁드립니다.
다시 한 번 너무 늦게 리뷰 드려 죄송합니다.
감사합니다.


{{EmbedInteractiveExample("pages/tabbed/pseudo-element-placeholder.html", "tabbed-shorter")}}

{{cssxref("::first-line")}} 의사 요소에 적용할 수 있는 한정적인 수의 CSS만이 `::placeholder` 와 그 선택자 규칙에 사용될 수 있습니다.
Copy link
Contributor

Choose a reason for hiding this comment

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

subset 이라서 한정적인 수 보다는 하위 집합은 어떨까요?

{{cssxref("::first-line")}} 의사 요소에 적용되는 CSS 속성의 하위 집합만 ::placeholder 와 그 선택자 규칙에 사용될 수 있습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정 완료되었어요. 감사합니다. XD

files/ko/web/css/_doublecolon_placeholder/index.md Outdated Show resolved Hide resolved
@github-actions github-actions bot removed the merge conflicts 🚧 This pull request has merge conflicts that must be resolved. label Oct 4, 2024
@givvemee givvemee requested a review from sunhpark42 October 10, 2024 00:58
Copy link
Member

@hochan222 hochan222 left a comment

Choose a reason for hiding this comment

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

PR 지연이 많이되었고, 리뷰사항 잘 반영된듯해서 병합하도록 하겠습니다.

모두 고생 많으셨습니다. 좋은 하루 보내세요 :)

@hochan222 hochan222 merged commit 531bf54 into mdn:main Oct 27, 2024
7 checks passed
ikenk pushed a commit to ikenk/translated-content that referenced this pull request Oct 29, 2024
* [ko] double colon placeholder 번역

* fix : 리뷰 반영
ikenk pushed a commit to ikenk/translated-content that referenced this pull request Oct 31, 2024
* [ko] double colon placeholder 번역

* fix : 리뷰 반영
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