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

photoswipe 익스텐션 구조화 요청 #27

Closed
gitaeks opened this issue May 30, 2016 · 5 comments
Closed

photoswipe 익스텐션 구조화 요청 #27

gitaeks opened this issue May 30, 2016 · 5 comments
Assignees

Comments

@gitaeks
Copy link
Member

gitaeks commented May 30, 2016

@kieregh , photoswipe 익스텐션 구조화 요청 합니다.

개요

기본예제

@gitaeks gitaeks added this to the v1.0.0 milestone May 30, 2016
@gitaeks
Copy link
Member Author

gitaeks commented May 30, 2016

예제에 적용된 소스코드는 아래를 참고해 주세요.
https://github.com/kimsQ/rc/tree/gh-pages/examples/extension/photoswipe

@kieregh
Copy link
Collaborator

kieregh commented May 30, 2016

우선 아래와 같이 기본구조는 잡았습니다.

##. 전체 구조 요약

  1. 바인딩
  • 아래 2 가지 방식 모두 pswp 마크업을 javascript 에서 동적으로 생성하도록 했습니다.
  • 아래 2 가지 방식 모두 data- 속성 값을 통해서 옵션 적용되도록 했습니다.
  1. 바인딩

    1. 마크업형
    • gallery 단위로 data-extension="photoswipe" 으로 해서 바인딩을 했습니다.
    1. 스크립트형
    • 이벤트 버튼에 data-toggle="photoswiper" 로 바인딩을 했습니다.
  2. 기본제공 옵션값 정리 요청

    • 예를 들어 예제 상으로 script 방식에서는 history : flase 로 하고 showAnimationDuration : 0 로
      하는 등 기본 옵션값이 달라지는데요....이 것들에 대한 명확한 룰이 있어야 할 것 같습니다.
    • 작업/문서를 먼저 만들어놓고 나중에 수정하려니 번거로워서 미리 정하고 문서작업을 하려고 합니다.

이상입니다.

@gitaeks
Copy link
Member Author

gitaeks commented May 30, 2016

수고하셨습니다
history 와 showAnimationDuration 등의 옵션은 링크목록형-마크업형 예제가 일반적인것으로 판단되니 이를 기준으로 옵션 기본값을 설정해 주시면 될 것 같습니다.

@gitaeks
Copy link
Member Author

gitaeks commented May 30, 2016

pswp 마크업 동적 생성 관련

본 마크업을 동적생성하여 간결한 적용이 가능한 장점이 있지만, 반면, 포토모달(pswp)의 커스텀이 불가능한 단점이 있는 것 같습니다. pswp 마크업 동적생성여부를 옵션화 한후, 기본값을 동적생성으로 처리했으면 합니다.

특수한 경우( 포토모달의 구성상의 수정이 필요한 경우 ) pswp 마크업 동적생성 기능을 비활성 처리한 후, 커스텀된 포토모달(pswp) 직접 추가할 수 있으면 확장성에 도움이 될 것 같습니다.

HTML In Slides

카드뉴스 형식 구현에 활용될 수 있는 기능으로 판단됩니다. 본 기능 또한 구조화 요청 합니다.

@kieregh
Copy link
Collaborator

kieregh commented May 31, 2016

추가 요청사항 적용해서 완료했습니다.

문서작성도 완료했으며 모든 예제도 업데이트했습니다.

@gitaeks gitaeks closed this as completed Jun 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants