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

added validation form builder (set as default form builder) #288

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

angdev
Copy link
Member

@angdev angdev commented Oct 15, 2014

음.. 되게 직관적이지 못한 함수 2개를 구현하였습니다 (..)

하나는 validation_inline_label 라는 함수이고 다른 하나는 validation_text_field 라는 함수인데

validation_inline_label

<div class="form-group form-group-lg has-feedback <%= validation_state(@user, :generation_id) %>">
  <%= f.label :generation_id, "기수", class: "col-sm-2 control-label" %>
  <div class="col-sm-10">
    <!-- yield -->
    <span class="glyphicon <%= validation_state_feedback(@user, :generation_id) %> form-control-feedback"></span>
    <span class="help-block"><%= attribute_error_message(@user, :generation_id) %></span>
  </div>
</div>

를 만들어주고 validation_text_field 는 yield 부분에 적절히 text_field 를 삽입해줍니다.
radio_button 쓰는 녀석이나 number_field, date_select 를 넣어주는 건 일단 구현하지 않고 block 으로 넣어버렸습니다.

코드 한 번 보시고 리뷰 부탁 드려요ㅠㅠ

@angdev angdev added the verify label Oct 15, 2014
@shaynekang
Copy link

허허... 지저분하군요.(..)
좋은 시도인 것 같은데, 제 생각은...

  1. FormBuilder 계열은 따로 파일/폴더를 빼 주는게 좋을 것 같습니다. 가령 app/form_builders/validation_form_builder.rb에 넣으면 어떨까요?
  2. content += ...이렇게 해결하는 것도 방법 같네요. 저라면 builder용 partial view를 만들어서, render partial을 하는것도 시도해볼 것 같습니다. 가령 app/views/applications/form_builders/_xxx.html 같은 파샬을 만들어 준 뒤, 이걸 폼 빌더에서 render해주는거죠.

코드만 잘 정리할 수 있다면, 평생 유용하게 써먹을 것 같습니다. 파팅파팅! ㅋ

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