Skip to content
This repository was archived by the owner on Mar 20, 2024. It is now read-only.

Trans(guide 1, 2, 3, 7)#26

Merged
byungjune merged 7 commits intoangular:masterfrom
tiratano:master
Jan 15, 2017
Merged

Trans(guide 1, 2, 3, 7)#26
byungjune merged 7 commits intoangular:masterfrom
tiratano:master

Conversation

@tiratano
Copy link
Contributor

@tiratano tiratano commented Jan 14, 2017

  • 테이블 컬럼의 SHOW/HIDE 문제
    guide/index.jade > 36줄~59줄: 중국사이트처럼 td 안에 p를 추가했습니다.

  • slack에서 언급했던 SHOW/HIDE 처리가 잘 안되는 문제가 있어서 일단 커밋했습니다.
    guide/index.jade > 135줄
    guide/learning-angular.jade > 40줄

  • quickstart.jade
    QuickStart seed 를 다른 곳에서는 빠른시작 "시드" 라고 했는데 이곳만 빠른시작 "준비"여서 "시드"로 수정했습니다.

  • user-input.jade
    7장도 추가했습니다.


* The [Glossary](glossary.html) defines terms that Angular developers should know.

// REVIEW: 아래 영문이 한국어 버전에서는 들여쓰기가 깨져서 보이고 있습니다.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

slack에서 언급한 이슈입니다. SHOW/HIDE가 잘 안되네요 ^^;

1. <a id="architecture"></a>Read the [Architecture](architecture.html) overview for the big picture.
<li if-docs="ts"><p>
[최상위 모듈](appmodule.html)에서는 Angular에서 애플리케이션을 어떻게 컴파일하고 실행하는지 알려주는 `NgModule` 클래스를 소개합니다.
// REVIEW: 아래 영문이 한국어 버전에서도 보이고 있습니다.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

slack에서 언급한 것과 비슷한 이슈입니다. SHOW/HIDE가 잘 안되네요 ^^;

Copy link
Contributor

Choose a reason for hiding this comment

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

@tiratano
수고하셨습니다 👍
해당 사항 확인하여 아래에 리뷰를 하였으니 확인부탁드립니다 :)

확인을 해보니 문제가 되었던 부분이 다른 항목들과는 다르게 ul 내부에 포함되어 있지 않고 외부에 <li>태그로 별도로 나와있었습니다.

image

그래서 해당

  • ...
  • 문장앞에 'tab'을 한 번씩 하여 indent를 추가하니,

    Before

    image
    image

    After

    image
    image

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

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

    guide/index.jade, guide/learning-angular.jade 모두 알려주신 방법으로 수정해 커밋했습니다. ^^
    감사합니다.

    @tiratano tiratano changed the title Trans(guide/index.jade, guide/learning-angular.jade) Trans(guide 1~3) Jan 14, 2017
    @tiratano tiratano changed the title Trans(guide 1~3) Trans(guide 1~3, 7) Jan 14, 2017
    @tiratano tiratano changed the title Trans(guide 1~3, 7) Trans(guide 1, 2, 3, 7) Jan 14, 2017
    Copy link
    Contributor

    @byungjune byungjune left a comment

    Choose a reason for hiding this comment

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

    많은 양의 한글화를 빠르게 해주셨네요.
    수고하셨습니다 👍

    일부 번역 누락 및 오타에 대해서 확인하여 코멘트를 남겼습니다. 확인부탁드립니다 : )

    Each page includes code snippets from a sample application that accompanies the page.
    You can reuse these snippets in your applications.

    [아케텍쳐](architecture.html) 페이지의 <live-example name="아키텍쳐"></live-example> 처럼,
    Copy link
    Contributor

    Choose a reason for hiding this comment

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

    오타를 발견했습니다 : )

    아케텍쳐 -> 아키텍쳐

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

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

    넵 수정하겠습니다.

    ### 다운로드
    ### Download

    <a href="!{_qsRepoZip}" title="Download the QuickStart seed repository">빠른시작 시드를 다운로드</a>하고 프로젝트 폴더에 압축을 풀고, 다음 터미널 명령을 사용해 나머지 단계를 수행하세요.
    Copy link
    Contributor

    Choose a reason for hiding this comment

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

    '시드를'과 '다운로드' 사이에 스페이스가 2번 들어간 것 같습니다 : )
    빠른시작 시드를 다운로드 -> 빠른시작 시드를 다운로드

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

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

    넵 수정하겠습니다.


    Defines `AppModule`, the [root module](appmodule.html "AppModule: the root module") that tells Angular how to assemble the application.
    Right now it declares only the `AppComponent`.
    Soon there will be more components to declare.
    Copy link
    Contributor

    Choose a reason for hiding this comment

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

    위 영어파트에서 아래 2줄에 내한 번역이 누락된 것 같습니다 : )

    Right now it declares only the AppComponent.
    Soon there will be more components to declare.

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

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

    넵 수정하겠습니다.

    @@ -162,8 +233,11 @@ table(width="100%")

    Copy link
    Contributor

    Choose a reason for hiding this comment

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

    위의 한 문장에 대한 번역이 누락된 것 같습니다.

    You'll learn about alternative compiling and deployment options later in the documentation.

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

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

    넵 수정하겠습니다.

    더 쉬운 방법도 있습니다: Angular의 `keyup.enter`라는 가상 이벤트(pseudo-event)에 바인딩하는 것입니다.

    There's an easier way: bind to Angular's `keyup.enter` pseudo-event.
    Then Angular calls the event handler only when the user presses _Enter_.
    Copy link
    Contributor

    Choose a reason for hiding this comment

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

    위 문장에 대한 번역이 누란된 것 같습니다.

    Then Angular calls the event handler only when the user presses Enter.

    Copy link
    Contributor Author

    Choose a reason for hiding this comment

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

    넵 수정하겠습니다.

    @tiratano
    Copy link
    Contributor Author

    위의 리뷰 사항들 수정하여 반영했습니다.

    @tiratano
    Copy link
    Contributor Author

    왜 빠진 부분들이 있을까요.. 음.. 다음부터 한/영 비교도 한번씩 더 해보겠습니다.
    리뷰 감사합니다 👍

    @byungjune byungjune merged commit d40d8e8 into angular:master Jan 15, 2017
    Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

    Labels

    None yet

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants