Skip to content

Conversation

@dlwogus0128
Copy link
Contributor

🌱 작업한 내용

  • 1차 릴리즈 QA 반영

🌱 PR Point

  • 1차 릴리즈 QA 반영.. 투비컨티뉴

📸 스크린샷

생략

📮 관련 이슈

수정창을 눌렀을 때 현재 닉네임을 입력창에 띄워주는 것으로 변경
해당 뷰에서 바텀네비게이션 제거
일러스트까지 스크롤 영역에 포함되도록 수정 (콜렉션뷰 섹션 분리)
이용약관 노션 연결
Copy link
Collaborator

@lsj8706 lsj8706 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~~👍

Comment on lines 125 to 129
if isSelected {
activityRecordContainerView.image = ImageLiterals.imgRecordContainerSelected
} else {
activityRecordContainerView.image = ImageLiterals.imgRecordContainer
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
if isSelected {
activityRecordContainerView.image = ImageLiterals.imgRecordContainerSelected
} else {
activityRecordContainerView.image = ImageLiterals.imgRecordContainer
}
activityRecordContainerView.image = isSelected ? 이미지1 : 이미지2

이렇게 삼항 연산자로도 쓸 수 있을 거 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

감사합니다! 반영했어요!

Comment on lines +122 to +123
func numberOfSections(in collectionView: UICollectionView) -> Int {
return 2
Copy link
Collaborator

Choose a reason for hiding this comment

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

섹션으로 나눠서 하는거 좋네요~~!

@dlwogus0128 dlwogus0128 merged commit 9562805 into Runnect:develop May 21, 2023
@dlwogus0128 dlwogus0128 deleted the fix/#144-1차-릴리즈-QA-반영 branch May 21, 2023 14:29
@dlwogus0128 dlwogus0128 restored the fix/#144-1차-릴리즈-QA-반영 branch May 21, 2023 14:30
@dlwogus0128 dlwogus0128 deleted the fix/#144-1차-릴리즈-QA-반영 branch May 21, 2023 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[Fix] 1차 릴리즈 QA 반영

2 participants