Skip to content
This repository has been archived by the owner on Sep 12, 2018. It is now read-only.

improve help message #610

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

improve help message #610

wants to merge 4 commits into from

Conversation

darkcircle
Copy link

도움말 메시지 개선 패치를 제안합니다.

  1. Yobi 플랫폼 이름을 명확하게 언급하는 부분 처음 부분에 추가.
  2. 이전 커밋에서 @Messages("app.name") 문자열을 코드에 그대로 사용하여 플랫폼의 이름이 올바르게 나타나지 않는 문제 수정.
  3. 문장과 문단을 일부 다듬었습니다.
  4. 일부 추상 표현(~하는 것)을 구체적인 단어/용어로 바꾸었습니다.

이 플랫폼의 이름은 Yobi이며 '요비'라고 읽습니다.
</div>
</div>
</li>
Copy link
Member

Choose a reason for hiding this comment

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

탭 인덴트가 안맞는거 같아요

@darkcircle
Copy link
Author

흠 제 커밋엔 탭을 사용했는데 원본 소스코드에는 공백 문자로 인덴트를 처리했군요.

원본 들여쓰기에 맞춰 수정해서 올려드리겠습니다. :)

changed from 8 spaces tab to 4 spaces 'space character'
@darkcircle
Copy link
Author

마지막에 br 태그를 사용한 이유는 문단상 내용을 보기 좋게 구분하기 위함입니다. non-breaking space는 제거했습니다 :-)

@doortts
Copy link
Contributor

doortts commented Mar 7, 2014

와우! 저희가 미처 손대지 못하고 있던 부분을 정리해 주셨네요!
감사합니다! : )

@darkcircle
Copy link
Author

부족한 부분이 많으니 칼같이 검토해주세요 :-)

@@ -106,14 +121,14 @@
<li class="qa">
<div class="question-wrap">
<i class="yobicon-q q"></i>
<a href="#!/toggle" class="question">@Messages("app.name")의 버그를 발견했어요.</a>
<a href="#!/toggle" class="question">Yobi의 버그를 발견했어요.</a>
Copy link
Contributor

Choose a reason for hiding this comment

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

이건 그대로 app.name을 사용해도 되지 않을까요?

Copy link
Author

Choose a reason for hiding this comment

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

@npcode 네 저도 그 고민을 했었습니다. 저렇게 고쳐놓은게 과연 잘한일인지 고뇌감에 OTL
원래대로 돌려놔야 할거 같다는 의견이 나오면 원래대로 돌려놓으려고 합니다.
저만 이상한 느낌이 들었던건 아니군요.

app.name 그대로 사용하는 방향으로 하는게 좋을까요? 다른 분들 의견이 어떤지 들어보고 싶군요.

apply committer's suggestion to revert L124 in toc.scala.html
add a short text to describe a terminology "Yobi" at L130 in toc.scala.html
@insanehong insanehong closed this Mar 20, 2014
@insanehong insanehong reopened this Mar 20, 2014
@insanehong
Copy link
Member

안녕하세요. 보내주신 변경건에 대해서는 내부적으로 논의를 하고 있어 이번 배포버전에서는 적용이 안될거 같습니다.

논의된 결과에 대해서는 차후에 피드백 드리도록 하겠습니다.

@darkcircle
Copy link
Author

오잉? 닫혔다가 열렸군요. 이제 봤습니다.
메시지 고친걸로는 해결 안 될 수도 있습니다.
대대적으로 손봐야 할 부분일 것 같네요.
만약 pr이 아니라 이 추측이 얼추 맞다면 그냥 과감하게 닫으셔도 됩니다.

추가의견 한가지 더 드리자면 일단 yobi가 처음에 비해 견고하게 바뀐만큼
도움말 메시지도 구조적으로 바뀔 필요가 있을지도 모르겠네요.
사용 방법을 직관적으로 안내할 수 있도록 레이아웃 및 문장 수준에서
구체화 하는 작업이 필요할 수도 있다는 의미입니다.

@doortts doortts added this to the v0.5.2 milestone Apr 22, 2014
@eungjun-yi eungjun-yi modified the milestones: v0.5.5, v0.5.4 May 22, 2014
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.

4 participants