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] Work done for 'Web Audio API' article. #1609

Merged
merged 4 commits into from
Aug 15, 2021

Conversation

logic-finder
Copy link
Contributor

@logic-finder logic-finder commented Jul 19, 2021

https://developer.mozilla.org/ko/docs/Web/API/Web_Audio_API
문서 작업하였습니다.

  • en-US문서 기준으로 최신화/번역하였습니다.
  • 기존 사진 영어판에서 쓰이는 사진으로 변경
  • 기존 번역 조금 수정

※ 현재 이 문서에 특정 조건(?) 아래에서 Obsolete interface 문단이 나타나는 버그가 있습니다 (mdn/yari#4227). (예시로, 영어버전에서 한국어로 언어 변경하고 view in english 누르면 그 전까지 안보이던 obsolete interface 문단이 새로 나타납니다) 만약 이 버그의 수정 결과로 새롭게 뭔가 작업할 게 생기면 그때 새롭게 pr을 만들든지 혹은 이 pr에 추가적으로 commit을 하든지 하겠습니다.

※ 문서 하단 guides and tutorials 부분을 보시면 < p >{{LandingPageListSubpages}}< /p > 라고 나와 있는데 아마도 content\files\en-us\web\api\web_audio_api 경로에 있는 문서들을 읽어오는 식으로 동작하는 게 아닌가 합니다. (정확한 작동방식은 모르겠습니다) 이것때문인지 영어버전 문서를 보시면 여러 개의 가이드 문서들이 나오는 걸 보실 수 있는데 한국어 버전을 보시면 지금 저 문서들 중에서 번역된 문서가 3개뿐입니다(#1518, #1564, #1535).
image
↑ en-US
image
↑ ko
그래서 그런데 단시간에 저 많은 문서들을 다 번역할 수도 없으므로 한국어 버전의 경우 일단 임시방편으로 LandingPageListSubpages를 빼고 대신 a태그로 일일히 해당 내용을 넣어놓는게 어떨까요? (@hochan222 님께서는 어떻게 생각하시나요?)

감사합니다.

@logic-finder logic-finder requested a review from a team as a code owner July 19, 2021 14:15
@logic-finder logic-finder requested review from igy95 and removed request for a team July 19, 2021 14:15
@github-actions github-actions bot added the l10n-ko Issues related to Korean content. label Jul 19, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Jul 19, 2021

Preview URLs

Flaws

URL: /ko/docs/Web/API/Web_Audio_API
Title: Web Audio API
on GitHub
Flaw count: 50

  • macros:
    • /ko/docs/Web/API/OscillatorNode does not exist but fallbacked on /en-US/docs/Web/API/OscillatorNode
    • /ko/docs/Web/API/MediaElementAudioSourceNode does not exist but fallbacked on /en-US/docs/Web/API/MediaElementAudioSourceNode
    • /ko/docs/Web/API/MediaStreamAudioSourceNode does not exist but fallbacked on /en-US/docs/Web/API/MediaStreamAudioSourceNode
    • /ko/docs/Web/API/GainNode does not exist but fallbacked on /en-US/docs/Web/API/GainNode
    • /ko/docs/Web/API/AudioContext/destination does not exist but fallbacked on /en-US/docs/Web/API/BaseAudioContext/destination
    • and 45 more flaws omitted

URL: /ko/docs/Web/API/Web_Audio_API/Advanced_techniques
Title: Advanced techniques: Creating and sequencing audio
on GitHub
Flaw count: 14

  • macros:
    • /ko/docs/Web/API/OscillatorNode does not exist but fallbacked on /en-US/docs/Web/API/OscillatorNode
    • /ko/docs/Web/API/PeriodicWave does not exist but fallbacked on /en-US/docs/Web/API/PeriodicWave
    • /ko/docs/Web/API/OscillatorNode does not exist but fallbacked on /en-US/docs/Web/API/OscillatorNode
    • /ko/docs/Web/API/BaseAudioContext/decodeAudioData does not exist but fallbacked on /en-US/docs/Web/API/BaseAudioContext/decodeAudioData
    • /ko/docs/Web/API/OscillatorNode does not exist but fallbacked on /en-US/docs/Web/API/OscillatorNode
    • and 8 more flaws omitted
  • broken_links:
    • Is currently http:// but can become https://

URL: /ko/docs/Web/API/Web_Audio_API/Using_IIR_filters
Title: Using IIR filters
on GitHub
Flaw count: 5

  • macros:
    • /ko/docs/Web/API/IIRFilterNode does not exist but fallbacked on /en-US/docs/Web/API/IIRFilterNode
    • /ko/docs/Web/API/IIRFilterNode does not exist but fallbacked on /en-US/docs/Web/API/IIRFilterNode
    • /ko/docs/Web/API/IIRFilterNode does not exist but fallbacked on /en-US/docs/Web/API/IIRFilterNode
  • broken_links:
    • Is currently http:// but can become https://
    • Is currently http:// but can become https://

URL: /ko/docs/Web/API/Web_Audio_API/Controlling_multiple_parameters_with_ConstantSourceNode
Title: Controlling multiple parameters with ConstantSourceNode
on GitHub
Flaw count: 30

  • macros:
    • /ko/docs/Web/API/ConstantSourceNode does not exist but fallbacked on /en-US/docs/Web/API/ConstantSourceNode
    • /ko/docs/Web/API/ConstantSourceNode/offset does not exist but fallbacked on /en-US/docs/Web/API/ConstantSourceNode/offset
    • /ko/docs/Web/API/ConstantSourceNode does not exist but fallbacked on /en-US/docs/Web/API/ConstantSourceNode
    • /ko/docs/Web/API/ConstantSourceNode does not exist but fallbacked on /en-US/docs/Web/API/ConstantSourceNode
    • /ko/docs/Web/API/ConstantSourceNode/offset does not exist but fallbacked on /en-US/docs/Web/API/ConstantSourceNode/offset
    • and 25 more flaws omitted

URL: /ko/docs/Web/API/Web_Audio_API/Migrating_from_webkitAudioContext
Title: Migrating from webkitAudioContext
on GitHub
Flaw count: 23

  • macros:
    • /ko/docs/Web/API/CreateGain does not exist
    • /ko/docs/Web/API/CreateDelay does not exist
    • /ko/docs/Web/API/CreateScriptProcessor does not exist
    • /ko/docs/Web/API/OscillatorNode does not exist but fallbacked on /en-US/docs/Web/API/OscillatorNode
    • /ko/docs/Web/API/AudioBufferSourceNode_ does not exist
    • and 17 more flaws omitted
  • broken_links:
    • Is currently http:// but can become https://

URL: /ko/docs/Web/API/Web_Audio_API/Using_AudioWorklet
Title: Background audio processing using AudioWorklet
on GitHub
Flaw count: 36

  • macros:
    • /ko/docs/Web/API/AudioWorklet does not exist but fallbacked on /en-US/docs/Web/API/AudioWorklet
    • /ko/docs/Web/API/Worklet does not exist but fallbacked on /en-US/docs/Web/API/Worklet
    • /ko/docs/Web/API/Worklet/addModule does not exist but fallbacked on /en-US/docs/Web/API/Worklet/addModule
    • /ko/docs/Web/API/AudioWorkletNode does not exist but fallbacked on /en-US/docs/Web/API/AudioWorkletNode
    • /ko/docs/Web/API/AudioWorkletProcessor does not exist but fallbacked on /en-US/docs/Web/API/AudioWorkletProcessor
    • and 31 more flaws omitted

URL: /ko/docs/Web/API/Web_Audio_API/Tools
Title: Tools for analyzing Web Audio usage
on GitHub
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Web/Apps/Fundamentals/Audio_and_video_delivery/Web_Audio_API_cross_browser

URL: /ko/docs/Web/API/Web_Audio_API/Best_practices
Title: Web Audio API best practices
on GitHub
Flaw count: 5

  • macros:
    • /ko/docs/Web/API/MediaStreamAudioSourceNode does not exist but fallbacked on /en-US/docs/Web/API/MediaStreamAudioSourceNode
    • /ko/docs/Web/API/OscillatorNode does not exist but fallbacked on /en-US/docs/Web/API/OscillatorNode
    • /ko/docs/Web/API/OfflineAudioContext does not exist but fallbacked on /en-US/docs/Web/API/OfflineAudioContext
    • /ko/docs/Web/API/GainNode does not exist but fallbacked on /en-US/docs/Web/API/GainNode
  • broken_links:
    • Can't resolve /en-US/docs/Web/API/BaseAudioContext/resume

URL: /ko/docs/Web/API/Web_Audio_API/Visualizations_with_Web_Audio_API
Title: Visualizations with Web Audio API
on GitHub
Flaw count: 17

  • macros:
    • /ko/docs/Web/API/BaseAudioContext/createAnalyser does not exist but fallbacked on /en-US/docs/Web/API/BaseAudioContext/createAnalyser
    • /ko/docs/Web/API/AnalyserNode/fftSize does not exist but fallbacked on /en-US/docs/Web/API/AnalyserNode/fftSize
    • /ko/docs/Web/API/AnalyserNode/minDecibels does not exist but fallbacked on /en-US/docs/Web/API/AnalyserNode/minDecibels
    • /ko/docs/Web/API/AnalyserNode/maxDecibels does not exist but fallbacked on /en-US/docs/Web/API/AnalyserNode/maxDecibels
    • /ko/docs/Web/API/AnalyserNode/smoothingTimeConstant does not exist but fallbacked on /en-US/docs/Web/API/AnalyserNode/smoothingTimeConstant
    • and 11 more flaws omitted
  • broken_links:
    • Is currently http:// but can become https://

URL: /ko/docs/Web/API/Web_Audio_API/Web_audio_spatialization_basics
Title: Web audio spatialization basics
on GitHub
Flaw count: 7

  • macros:
    • /ko/docs/Web/API/PannerNode does not exist but fallbacked on /en-US/docs/Web/API/PannerNode
    • /ko/docs/Web/API/StereoPannerNode does not exist but fallbacked on /en-US/docs/Web/API/StereoPannerNode
    • /ko/docs/Web/API/BaseAudioContext does not exist but fallbacked on /en-US/docs/Web/API/BaseAudioContext
    • /ko/docs/Web/API/AudioListener does not exist but fallbacked on /en-US/docs/Web/API/AudioListener
    • /ko/docs/Web/API/PannerNode does not exist but fallbacked on /en-US/docs/Web/API/PannerNode
    • and 2 more flaws omitted

External URLs

URL: /ko/docs/Web/API/Web_Audio_API
Title: Web Audio API
on GitHub


URL: /ko/docs/Web/API/Web_Audio_API/Advanced_techniques
Title: Advanced techniques: Creating and sequencing audio
on GitHub


URL: /ko/docs/Web/API/Web_Audio_API/Using_IIR_filters
Title: Using IIR filters
on GitHub


URL: /ko/docs/Web/API/Web_Audio_API/Controlling_multiple_parameters_with_ConstantSourceNode
Title: Controlling multiple parameters with ConstantSourceNode
on GitHub

No new external URLs


URL: /ko/docs/Web/API/Web_Audio_API/Migrating_from_webkitAudioContext
Title: Migrating from webkitAudioContext
on GitHub


URL: /ko/docs/Web/API/Web_Audio_API/Using_AudioWorklet
Title: Background audio processing using AudioWorklet
on GitHub


URL: /ko/docs/Web/API/Web_Audio_API/Tools
Title: Tools for analyzing Web Audio usage
on GitHub


URL: /ko/docs/Web/API/Web_Audio_API/Best_practices
Title: Web Audio API best practices
on GitHub


URL: /ko/docs/Web/API/Web_Audio_API/Visualizations_with_Web_Audio_API
Title: Visualizations with Web Audio API
on GitHub


URL: /ko/docs/Web/API/Web_Audio_API/Web_audio_spatialization_basics
Title: Web audio spatialization basics
on GitHub

(this comment was updated 2021-08-06 17:38:15.461399)

@logic-finder
Copy link
Contributor Author

p.s. fall back 번역을 뭐라고 해야 좋을지 고민입니다. 대비?
a JS audio library that defaults to Web Audio API and falls back to HTML5 Audio, as well as providing other useful features.

@hochan222
Copy link
Member

@logic-finder 님 안녕하세요!

기여해주셔서 감사합니다.

※ 문서 하단 guides and tutorials 부분을 보시면 < p >{{LandingPageListSubpages}}< /p > 라고 나와 있는데 아마도 content\files\en-us\web\api\web_audio_api 경로에 있는 문서들을 읽어오는 식으로 동작하는 게 아닌가 합니다.

\ko\web\api\web_audio_api 한국 경로로 a tag 걸어 놓으신다는 말씀이신가요? 좋은 듯합니다. kumascript는 점차 사라질 예정이므로 링크로 대체해도 좋을듯 합니다.


그와는 별도로, Related Topics 부분은 없는것보다는 en-us 로 리다이렉트 되더라도 있는게 좋아 보입니다. 어떻게 생각하시나요?


좌측 네비게이션에 대해서는 번역하지 않는것을 권고한다는 MDN 공식글이 있었는데... 막상 찾으려고하니 잘 안보이네요..! 번역된것도 좋아보입니다 :)

좋은 하루 보내세요 :)

@hochan222
Copy link
Member

p.s. fall back 번역을 뭐라고 해야 좋을지 고민입니다. 대비?

저는 대안..? 좋아보입니다. 이 부분은 @GwangYeol-Im 에게 토스하겠습니댜.

@logic-finder
Copy link
Contributor Author

안녕하세요 답변 주셔서 감사합니다. 다름이 아니라 여쭤보고 싶은 게 있습니다.

  1. 좌측 네비게이션이라 하심은 tables of contents에 포함되어있는 문단 제목들 (ex: 가이드와 자습서, 예제, 명세 등)을 말씀하시는 건가요? 지금까지는 다 번역했었는데 이게 문제라면 수정토록 하겠습니다.
  2. 우선 저도 있는 게 좋다고 생각합니다. 한편 저는 지금까지 html파일만 수정해본 터라 좌측 네비게이션 부분에 대해서는 잘 모르겠습니다. (별도의 수정을 가한 적이 없습니다.) 혹시 현재 뭔가 문제가 있나요?

@hochan222
Copy link
Member

hochan222 commented Jul 30, 2021

@logic-finder 님 안녕하세요! 답변드립니다.

  1. 좌측 네비게이션이라 하심은 tables of contents에 포함되어있는 문단 제목들 (ex: 가이드와 자습서, 예제, 명세 등)을 말씀하시는 건가요? 지금까지는 다 번역했었는데 이게 문제라면 수정토록 하겠습니다.

저는 한국어로 해석하는게 좋아보입니다. 그때의 뉘양스는 유지보수측면에 더 있었던것으로 기억합니다. 혹시, 제가 번역하지 않는것을 권고한다는 MDN 블로그글을 접한 다음에 따로 알아보지 않았는데, 네비게이션을 수정하는 법 알려주실 수 있으신가요?

  1. 우선 저도 있는 게 좋다고 생각합니다. 한편 저는 지금까지 html파일만 수정해본 터라 좌측 네비게이션 부분에 대해서는 잘 모르겠습니다. (별도의 수정을 가한 적이 없습니다.) 혹시 현재 뭔가 문제가 있나요?

위의 사진 중 한글로 해석된 네비게이션을 가지고 있는 사진이 logic-finder:ko_web_audio_api의 해당 페이지고, 아래 사진이 en-us 페이지입니다. 또한,ko 페이지 main에서 봤는데 Related Topics가 존재해서, logic-finder:ko_web_audio_api에만 Related Topics가 없는게 이상해서 커멘트 남겼습니다!

@logic-finder
Copy link
Contributor Author

  1. 네비게이션은 그냥 문단 제목을 번역하면 자동으로 반영되는 것 같습니다.
  2. 알려주셔서 감사합니다. 뭐가 문제인지 찾아보겠습니다.

@hochan222
Copy link
Member

hochan222 commented Jul 30, 2021

네비게이션은 그냥 문단 제목을 번역하면 자동으로 반영되는 것 같습니다.

아.. 그렇다면 블로그 글은 다른 내용이었나봅니다.. 네비게이션 확인해보니까 그렇네요! 문제없을것 같아요! 알려주셔서 감사합니다.

좋은 밤 보내세요 :)

Copy link
Member

@igy95 igy95 left a comment

Choose a reason for hiding this comment

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

@logic-finder 안녕하세요! 자주 뵙네요 ㅎㅎ 우선 번역해주신 글을 찬찬히 확인해보았을 때, 내용 면에서 딱히 피드백 드릴 사항은 없었습니다. fall back 에 대해서는 '대안을 제공하는' 정도의 뉘앙스면 충분하지 않을까 싶네요.

현재 conflict 사항이 하나 보이는데 이 부분 해결 후 멘션 달아주시면 바로 병합 진행하도록 하겠습니다! 고생 많으셨어요~

@hochan222 hochan222 requested a review from igy95 August 3, 2021 06:40
@logic-finder
Copy link
Contributor Author

안녕하세요, 늦어서 죄송합니다.
p.s. conflict 대신 해결해 주셔서 감사합니다.

다음과 같은 문서 수정사항이 있었습니다.
image
메인 페이지(index.html) 수정, 미번역 문서 9개 추가 (자세한 내용은 아래에 적었습니다.)

● 자잘한 수정

  • 튜토리얼 → 자습서 (번역 가이드라인을 따라서)
  • falls back 번역 → 제안해주신대로 '대안을 제공하는' 으로 바꿨습니다.
  • 웹캡 → 웹캠

● Related Topics가 나오지 않던 문제 해결
원인: 문서 상단에 {{DefaultAPISidebar("Web Audio API")}} 가 없어서 발생하던 문제 같습니다. 추가하니 잘 나오는 모습을 확인하였습니다. 주의깊게 문서를 살펴보지 않은 점 사과드립니다. 짚어 주셔서 대단히 감사합니다.

● {{LandingPageListSubpages}} 관련하여.
생각해보니 이것을 빼면 안 되겠다는 생각이 들었습니다. 왜냐하면, 보아하니 {{LandingPageListSubpages}} 는 해당 문서의 제목과 첫 문단을 가져와서 dl, dt, dd태그에 내용을 넣는 기능을 탑재하고 있는 것 같은데 만약 이것을 빼고 일일히 a태그와 dl, dt, dd태그에 내용을 넣어놓으면 해당 하위 문서들이 변경되었을 때 본 문서의 내용도 자동으로 변경되지 않겠다는 데까지 생각이 닿았습니다.

또한, {{LandingPageListSubpages}} 가 얼마나 많은 다른 문서에서 쓰이고 있는지는 모르겠습니다만 만약 추후에 이것이 삭제 and/or 다른 것으로 변경되는 작업이 진행된다고 가정했을 때 만약 제가 위처럼 문서를 망쳐(?)놨을 경우 본 문서에는 {{LandingPageListSubpages}}가 없으므로 그러한 작업에서 제외될 가능성을 고려하지 않을 수 없다는 생각이 들었습니다.

더불어, 만약 제가 위처럼 문서를 건드려놨을 경우, 이는 두 가지 경우에서 문제가 될 것 같습니다. 1) 메인 페이지를 수정하시는 기여자분께서는 이곳에 {{LandingPageListSubpages}} 가 있었다는 것을 알 단서가 없습니다. 만약 그분께서 하위 문서의 제목 and/or 첫 단락의 내용이 본 문서의 a태그와 dl, dt, dd태그에 있는 내용과 다르다는 것을 깨닫는다면 일일히 변경하시는 수고를 감수하셔야 합니다. 2) 하위 문서를 번역하시는 기여자분께서도 같은 상황입니다. 하위 문서를 번역하고, 추가로 본 문서 또한 수정하셔야 할 필요가 있습니다.

결론적으로, 가장 나은 방법은 우선은 a태그와 dl, dt, dd태그를 이용하여 {{LandingPageListSubpages}} 를 썼을 때와 동일한 양식으로 번역된 정보를 제공하고 가능한 한 빨리 하위 문서들의 번역을 마치고 나서 다시 {{LandingPageListSubpages}} 로 되돌려놓는 것이겠으나 그러기에는 현실적으로 무리가 있으므로, 현재로서는 {{LandingPageListSubpages}} 를 유지하되 미번역된 하위 문서들을 추가하여 그것들에 대한 정보를 제공하는 것이 낫다고 생각합니다, 물론, 그것들이 번역되지 않은 상태로 노출되는 것은 정보를 제공하는 것에 비하면 어쩔 수 없는 일이라고 생각합니다.

너저분한 설명 죄송합니다. 감사합니다.

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.

@logic-finder 님 안녕하세요! 리뷰드립니다.

메인 페이지(index.html) 수정, 미번역 문서 9개 추가 (자세한 내용은 아래에 적었습니다.)

해당 PR에 추가 commit으로 미번역 문서가 추가되었는데요! 이렇게 파일이 추가될 경우 en-us로의 리다이렉션이 이뤄지지 않아 MDN에서 페이지를 검색하는 입장에서는 최신 문서를 접하지 못할 가능성이 높아집니다. 해당 파일들은 방치되지 않고 번역을 완료해 주실 것이라 믿고 merge 하겠습니다.

또한, 리뷰어 입장에서는 한기지 PR에 문서량이 많아질수록 부담감을 많이 느낄수 밖에 없는데요! 하나의 PR에 하나의 책임을 갖도록 PR 주시면 리뷰하는데 조금 더 수월할것 같습니다. (.md를 .html로 바꾸는 경우가 아니고서는 1 파일 1 PR이 가장 좋습니다!)

현재로서는 {{LandingPageListSubpages}} 를 유지하되 미번역된 하위 문서들을 추가하여 그것들에 대한 정보를 제공하는 것이 낫다고 생각합니다

👍👍👍

15일이 경과하여, 빠른 진행을 위해 제가 merge 하겠습니다.

기여해주셔서 감사합니다. 좋은 하루 보내세요 :)

@hochan222 hochan222 merged commit f93ef19 into mdn:main Aug 15, 2021
@logic-finder
Copy link
Contributor Author

리다이렉션 부분은 미처 생각을 못했네요. 알려주셔서 감사합니다.

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