-
Notifications
You must be signed in to change notification settings - Fork 6
[Feat] #44 - Plus detail VC UI구현 #46
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
The head ref may contain hidden characters: "#44-PlusDetailVC\uAD6C\uD604"
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다~!
코멘트 달린 부분만 수정해서 머지해주세요~!
(커밋하고 push 하면 지금 이 PR에 자동으로 해당 커밋이 반영됩니다.)
가능하면 현재 나오고 있는 린트 경고도 최대한 해결해서 올려주세요!
|
|
||
| private let reuseIdentifier = "MapCollectionViewCell" | ||
|
|
||
| class MapCollectionViewController: UICollectionViewController { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 파일은 이제 사용하지 않고 있는거죠?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 복붙해서 쓰려고 남겨놓은 거긴한데...없앨까용 ..?🥺
Runnect-iOS/Runnect-iOS/Presentation/CourseDiscovery/VC/CourseDiscoveryVC.swift
Show resolved
Hide resolved
Runnect-iOS/Runnect-iOS/Presentation/CourseDiscovery/VC/CourseDiscoveryVC.swift
Show resolved
Hide resolved
|
|
||
| let collectionView = UICollectionView(frame: .zero, collectionViewLayout: layout) | ||
| collectionView.backgroundColor = .clear | ||
| collectionView.translatesAutoresizingMaskIntoConstraints = false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| collectionView.translatesAutoresizingMaskIntoConstraints = false |
스냅킷을 사용하고 있기 때문에 이 코드는 지워도 상관 없습니다. (스냅킷이 내부적으로 해줌)
🌱 작업한 내용
🌱 PR Point
📸 스크린샷
📮 관련 이슈