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

[FIX] #87 - 명함 그룹 화면 수정 #93

Merged
merged 8 commits into from
Nov 29, 2021
Merged

Conversation

dlwns33
Copy link
Member

@dlwns33 dlwns33 commented Nov 29, 2021

🌴 PR 요약

🌱 작업한 브랜치

🌱 작업한 내용

  • 교수님의 코리 참고하며 isSelected 코드 변경 했습니다. 이제 그룹 화면을 처음 들어갈때 '미분류' select 가 디폴트입니다!
  • groupCollectionView estimated size none으로 변경하여 마지막 셀이 잘려보이는 이슈를 수정하였습니다.
  • groupCollecvtionView inset 조정하여 마지막 셀이 너무 딱 붙지 않도록 하였습니다.
  • 그룹 뷰에 쓰는 cell Const로 변경하였습니다.
  • 불필요한 주석 제거하였습니다.

📸 스크린샷

기능 스크린샷 기능 스크린샷
전체화면 Simulator Screen Shot - iPhone 11 Pro - 2021-11-29 at 21 49 07 구현한 화면 GIF ezgif com-gif-maker (1)

📮 관련 이슈

@dlwns33 dlwns33 added Yijoon 🐨 이준 대치동강사 작업 Fix 코드 수정 및 버그, 오류 해결 labels Nov 29, 2021
@dlwns33 dlwns33 self-assigned this Nov 29, 2021
Copy link
Member

@mini-min mini-min left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 교수님!!
이거는 조금 다른 얘기인데 이미 아실수도 있지만,
그룹화면에 컬렉션 뷰 2개 모두 background 색상이 아직 설정이 안되어 있는 것 같더라구요!!
이것두 나중에 이슈파서 진행하시거나 이 이슈에 한번에 수정해주시면 감사하겠습니다 ^__^

@@ -85,7 +85,7 @@
<constraints>
<constraint firstAttribute="height" constant="32" id="4E1-ms-weE"/>
</constraints>
<collectionViewFlowLayout key="collectionViewLayout" scrollDirection="horizontal" automaticEstimatedItemSize="YES" minimumLineSpacing="10" minimumInteritemSpacing="10" id="gMe-6U-LJh">
<collectionViewFlowLayout key="collectionViewLayout" scrollDirection="horizontal" minimumLineSpacing="10" minimumInteritemSpacing="10" id="gMe-6U-LJh">
Copy link
Member

Choose a reason for hiding this comment

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

크으 도-약🚀

Copy link
Member Author

Choose a reason for hiding this comment

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

헐 background 색상 이 이슈에 같이 수정해야겠네요!!! 감사함둥 ㅠㅠ

Copy link
Member Author

Choose a reason for hiding this comment

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

고생하셨습니다 교수님!! 이거는 조금 다른 얘기인데 이미 아실수도 있지만, 그룹화면에 컬렉션 뷰 2개 모두 background 색상이 아직 설정이 안되어 있는 것 같더라구요!! 이것두 나중에 이슈파서 진행하시거나 이 이슈에 한번에 수정해주시면 감사하겠습니다 ^__^

설정 완료하고 폴더아이콘도 다크모드 추가했슴당 !!!!

Copy link
Member

Choose a reason for hiding this comment

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

좋씀다!

Comment on lines +302 to +307
switch collectionView {
case groupCollectionView:
return .init(top: 0, left: 0, bottom: 0, right: 10)
default:
return .zero
}
Copy link
Member

Choose a reason for hiding this comment

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

역시 해결할줄 알았습니다 교수님

Copy link
Member

Choose a reason for hiding this comment

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

캬..! 좋습니다!
더 디테일한건 QA 때 뵙도록 하져

Copy link
Member

@hyun99999 hyun99999 left a comment

Choose a reason for hiding this comment

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

line ... inter.. ㅜ... 된다면 오케이입니다!!

@dlwns33 dlwns33 merged commit 343f508 into TeamNADA:develop Nov 29, 2021
@dlwns33 dlwns33 deleted the fix/#87 branch November 29, 2021 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix 코드 수정 및 버그, 오류 해결 Yijoon 🐨 이준 대치동강사 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 명함 그룹 화면 수정
3 participants