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

Update gallery_layout.md #8

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

Conversation

ShinJaehun
Copy link
Contributor

strong parameter에 대한 얘기가 없어서 새로 추가했습니다. 몇가지 궁금한 점이 있어서 추가로 문의드립니다.

  1. 본문에 /bulletins/갤러리/posts에 보면 바둑판 식으로 예쁘게 이미지가 표시되는데 실재로는 이미지가 표시는 되는데 resize 되지 않은 상태로 일반 post처럼 위에서 아래로 순서대로 표시됩니다. _gallery.html.erb 파일에 <div class='gallery'> 또는 <div class='picture'>로 정의되어 있어서
    태그에 css를 먹이면 예제처럼 예쁘게 나올 것 같은데 이 부분에 대한 설명이 생략되어 있네요.(제가 할 수 있으면 추가해보려고 했습니다만 소스 코드 https://github.com/rorlab/rcafe/tree/%EC%A0%9C5.12%EC%9E%A5/app/assets/stylesheets 를 찾아봤는데도 gallery나 picture에 대한 정의가 없어서 어떻게 해야할지 모르겠군요.)
  2. 본문에는 갤러리 index에만 이미지가 표시됩니다. show 액션에 대한 뷰 파일은 이미지를 불러오는 내용이 생략되어 있기 때문입니다. 이것도 간단하게나마 설명해주시면 제가 살을 붙여보도록 하겠습니다.

strong parameter에 대한 얘기가 없어서 새로 추가했습니다. 몇가지 궁금한 점이 있어서 추가로 문의드립니다.
1. 본문에 /bulletins/갤러리/posts에 보면 바둑판 식으로 예쁘게 이미지가 표시되는데 실재로는 이미지 크기가 resize 되지 않은 상태로 일반 post처럼 순서대로 표시됩니다. _gallery.html.erb 파일에 <div class='gallery'> 또는 <div class='picture'>처럼 <div> 태그로 css를 먹이면 예제처럼 예쁘게 나올 것 같은데 이 부분에 대한 설명이 생략되어 있네요.(제가 할 수 있으면 추가해보려고 했습니다만 소스 코드 https://github.com/rorlab/rcafe/tree/%EC%A0%9C5.12%EC%9E%A5/app/assets/stylesheets 를 찾아봤는데도 gallery나 picture에 대한 정의가 없어서 어떻게 해야할지 모르겠군요.)
2. 본문에는 갤러리 index에만 이미지가 표시됩니다. show 액션에 대한 뷰 파일은 이미지를 불러오는 내용이 생략되어 있기 때문입니다. 이것도 간단하게나마 설명해주시면 제가 살을 붙여보도록 하겠습니다.
@rorlab
Copy link
Contributor

rorlab commented Jan 30, 2015

현재 1.0.6 패치 버전을 작성 중입니다. 1.0.5 버전에 대한 보완 PR 작업은 중단해 주시고, 언급하신 내용을 1.0.6 패치 버전에서 보완토록 하겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants