Skip to content

[오타수정] Part 1 2.10 if와 '?'를 사용한 조건 처리 - svg 텍스트의 불완전한 번역 수정 (issue #1500)#1514

Merged
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfrom
2yunseong:1500
Aug 24, 2022
Merged

[오타수정] Part 1 2.10 if와 '?'를 사용한 조건 처리 - svg 텍스트의 불완전한 번역 수정 (issue #1500)#1514
Violet-Bora-Lee merged 1 commit intojavascript-tutorial:masterfrom
2yunseong:1500

Conversation

@2yunseong
Copy link
Copy Markdown
Contributor

@2yunseong 2yunseong commented Aug 14, 2022

Summary

참고 : 이슈 #1500
링크 : https://ko.javascript.info/ifelse - 과제 2번

이슈 요약

과제 2번의 플로우차트의 불완전한 번역 수정
(번역이 모두 다 이루어지지 않고 부분적으로 이루어짐)

로컬서버 수정캡처본

Screen Shot 2022-08-14 at 17 26 37

수정내역

  • 기존 플로우차트에 있는 시작, 정답입니다는 따로 위치를 변경하지 않았습니다.
  • You don't know? ECMAScript! 는 2줄로 작성되어 있었지만, 한글로 작성하다보니 타원 공간 상 3줄로 구성하였습니다.
    (줄 바꿈과 단락을 원문과 동일하게 하라 하였는데, svg파일 상 한줄 안에 모든 것이 작성되어 있어 이미지 상으로는 3줄이지만 깃 수정 내역상으로는 한줄이여서 이와 같이 구성하였습니다.)
  • 번역은 해답에 작성된 결과와 동일한 문장으로 구성하였습니다.
  • 번역 후 글의 위치가 맞지 않아, 수정한 tspan의 x, y 위치를 변경하였습니다. 문제가 될 시, 수정하도록 하겠습니다.

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
    • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
    • 맞춤법 검사기로 맞춤법을 확인하셨나요?
    • 마크다운 문법에 사용되는 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`) 등의 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Aug 14, 2022

CLA assistant check
All committers have signed the CLA.

@Violet-Bora-Lee
Copy link
Copy Markdown
Member

@2yunseong
Copy link
Copy Markdown
Contributor Author

2yunseong commented Aug 16, 2022

기여 감사합니다~

https://github.com/javascript-tutorial/ko.javascript.info/wiki/%EB%B2%88%EC%97%AD-%EB%AA%A8%EB%B2%94-%EC%82%AC%EB%A1%80#%EA%B0%95%EC%A1%B0%EC%9D%B8%EC%9A%A9%EC%8B%9C-%EC%82%AC%EC%9A%A9%EB%90%98%EB%8A%94-%EB%94%B0%EC%98%B4%ED%91%9C

따옴표 관련해서 위 링크 보시고 적용 가능하실까요?

앗..! 막연히 예제 코드에 있는 것을 보고 적용시켜서 놓쳤네요 ㅠㅠ 피드백 감사합니다.
추후에 과제에 있는 따옴표도 고치도록 하겠습니다!!

Screen Shot 2022-08-16 at 16 02 17

따옴표 수정하고, 로컬 서버에서 확인 후 다시 커밋하였습니다!
(커밋은 합쳤습니다.)

@2yunseong
Copy link
Copy Markdown
Contributor Author

/done

@Violet-Bora-Lee
Copy link
Copy Markdown
Member

기여 감사합니다~
바로 머지 할게요 👍

@Violet-Bora-Lee Violet-Bora-Lee merged commit 271850a into javascript-tutorial:master Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants