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

Feat/3차과제 #4

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Feat/3차과제 #4

wants to merge 2 commits into from

Conversation

jeongdung-eo
Copy link
Contributor

@jeongdung-eo jeongdung-eo commented Nov 4, 2022

🔥 Pull requests

⛳️ 작업한 브랜치

  • feature/#이슈번호

👷 작업한 내용
chat view collectionview구현
friend view collectionview 구현
photo collectionview 구현

🚨 참고 사항

compositional layout
Diffable DataSource
friendView tableview로 바꿀 예정
photoview 셀 데이터 전달 구현 못함
freindView swipe 액션 구현 안함

📸 스크린샷

기능 스크린샷

seminar3

📟 관련 이슈

  • Resolved: #이슈번호

class FriendListViewController: UIViewController{
let list : [FriendList] = FriendList.list
var dataSource : UICollectionViewDiffableDataSource<Sections, Item>!
typealias Item = AnyHashable
Copy link
Member

Choose a reason for hiding this comment

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

typealias를 사용해서 간결하게 나타나는 법이 있었군요 한 수 배워갑니다,,,,

$0.delegate = self
$0.allowsMultipleSelection = true
}
private lazy var safeArea = self.view.safeAreaLayoutGuide
Copy link
Member

Choose a reason for hiding this comment

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

safeArea 선언해서 간결성 높이는것도 좋네용,,. 굳굳

return header
}
}
func createLayout() -> UICollectionViewCompositionalLayout {
Copy link
Member

Choose a reason for hiding this comment

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

Compositional 한 수 배웁니다,,,

Choose a reason for hiding this comment

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

CompositionalLayout !!! 저도 꼭 써 보고 싶어여 배우고 갑니다!

@loveitall
Copy link

와..대박...왤케 잘하시나요..추후에 구현하신거 보면서 공부 해야겠습니다!!

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.

4 participants