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

MainCardView: 재사용성 높은 카드뷰로 수정하기 #72

Open
kor-Chipmunk opened this issue Apr 17, 2020 · 2 comments
Open

MainCardView: 재사용성 높은 카드뷰로 수정하기 #72

kor-Chipmunk opened this issue Apr 17, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@kor-Chipmunk
Copy link
Collaborator

배경

현재 메인과 답변 화면에서 사용하는 큰 카드뷰와 앨범에서 사용하는 작은 카드뷰를 MainCardView 하나로 공유된 상태다. 큰 카드뷰와 작은 카드뷰의 차이점은 다음과 같다.

  1. 그림자 유무
  2. 두 개의 RoundedRectangle이 겹쳐져 있는 영역의 Round 값
  3. 두 개의 RoundedRectangle이 겹쳐져 있는 영역의 Padding 값
  4. 파츠가 들어갈 영역의 Padding 값
  5. 파츠가 들어갈 영역의 aspectRatio 값

구현할 것

  1. MainCardView 이름을 CardView 로 변경
  2. 다양한 CardView 크기에 대비하여 여러 속성들을 입력으로 받음
  3. CardView 와 파츠들을 담을 새로운 CardWithPartsView 추가
  4. CardWithPartsView를 만들 때 어디에 쓸 것인지를 입력 받는다.
    1. 메인 카드 : 크고 그림자가 있음
    2. 답변 카드 : 현재까지 메인 카드와 동일
    3. 앨범 카드 : 작고 그림자가 없음
  5. 카드 관련 뷰는 모두 Common 폴더로 이동
@kor-Chipmunk kor-Chipmunk self-assigned this Apr 17, 2020
@kor-Chipmunk kor-Chipmunk added the enhancement New feature or request label Apr 17, 2020
@kor-Chipmunk
Copy link
Collaborator Author

efc6021 커밋으로 MainCardView 리팩토링을 수행했습니다. 다음은 각 뷰에 맞게 적절한 속성을 대입해주는 코드를 작성해야 합니다. AspectRatio 는 뷰 밖에서 지정을 해 주는게 좋을지, 뷰 안에서 지정을 하는게 맞을지 고민입니다.

@kor-Chipmunk
Copy link
Collaborator Author

cf242d5 커밋으로 CardView 속성들을 모두 캡슐화시켰습니다. 앞으로 할 일은 CardWithPartsView 를 새로 만들 일입니다.

그리고 Common 디렉토리의 파일들을 정리하면 좋을 것 같습니다.

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

No branches or pull requests

1 participant