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

docs(flicking): Document reinforcement #311

Closed
wants to merge 3 commits into from
Closed

Conversation

netil
Copy link
Member

@netil netil commented Jul 21, 2016

Issue

#295, #217

Details

Update on description
Grammatical correction

Preferred reviewers

@naver/egjs-dev

Update on description
Grammatical correction

Ref naver#295, naver#217
@netil netil added this to the 1.3.0 milestone Jul 21, 2016
@netil netil self-assigned this Jul 21, 2016
@@ -1196,7 +1192,7 @@ eg.module("flicking", ["jQuery", eg, window, document, eg.MovableCoord], functio

/**
* Return either panel is animating or not
* @ko 현재 애니메이션중인지 여부를 리턴한다.
* @ko 현재 애니메이션 중인지 여부를 리턴한다.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest

리턴한다 --> 반환한다.

* @group egjs
* @ko 플리킹 UI를 구성한다.
* @ko 캐로셀 플리킹 UI 구성 컴포넌트
Copy link
Contributor

Choose a reason for hiding this comment

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

@sculove
Copy link
Contributor

sculove commented Jul 22, 2016

LGTM

1 similar comment
@jongmoon
Copy link
Contributor

LGTM

@netil netil closed this in 29f05b2 Jul 25, 2016
@netil netil deleted the flicking#295 branch July 27, 2016 07:58
netil pushed a commit that referenced this pull request Dec 15, 2016
layout is bronken on iOS

ref #427, #y-300, #y301
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.

3 participants