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

[ADD] 1주차 과제 #4

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

[ADD] 1주차 과제 #4

wants to merge 11 commits into from

Conversation

hellozo0
Copy link
Contributor

@hellozo0 hellozo0 commented Nov 7, 2021

📌 관련 이슈

#1 #2 #3

📌 변경 사항 및 이유

이슈 1번에서 상단바, Table View, 하단의 Tabbar
이슈 2번에서 게시글 클릭시 화면전환, 데이터 전달
이슈 3번에서 게시글의 UI, 게시글 뷰의 데이터 관리

📌 PR Point

Simulator Screen Recording - iPhone 11 - 2021-11-08 at 05 25 28 mp4

PR할만큼 잘난게 없네요...
다음부터는 과제 미리미리 하겠습니다... 6시간만에 한거 치고는 ㄱㅊ하지 않나..? 아닌가./.?

📌 참고 사항(못한거)

[이슈1번]

  1. 사진의 UI가 잘 안잡힌다. - 가로세로 크기 고정했는데 왜그럴까요? 사진 크기 강제로 캐스팅 하는법 더 찾아볼께요..
  2. Home화면에서 +버튼 구현해보겠습니다. 플로팅 버튼이라고 부르더라고요.. 내일 구현하겠습니다..
    (3. 게시글 눌렀을때 모션 없앴습니다. 그 진회색 보이는거 넘 초보같아서요.. ㅎㅎ)

[이슈2번]

  1. 해당 게시물의 데이터가 전달할 수 있도록 하겠습니다.

[이슈3번]

  1. 아니. 오ㅐ 왼쪽으로 쏠려서 보일까요? 웃깁니다. 어떻게 해야할지 모르겠지만 더 찾아볼께요 아시는분 알려주십쇼
  2. 추천상품영역에서 스크롤이 먹히는 것을 없애야한다. 게시글 페이지에서는 only 1 스크롤만 존재해야하는데 그것때문에 추천상품 컬렉션뷰 부분이 짤려서 보입니다... 그것또한 찾아볼께요...
  3. 데이터 전달 받는것도 해볼께요....
  4. 상단에 뒤로가기버튼바? 없애고 원래 당근마켓처럼 해보겠습니다...!

아 대박 못한거 왜이리 많아 울고싶게 진짜

Copy link
Member

@Suyeon9911 Suyeon9911 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 28 to 47
homeVC.tabBarItem.title = "홈"
homeVC.tabBarItem.image = UIImage(named: "homeIcon")
homeVC.tabBarItem.selectedImage = UIImage(named: "homeIconFill")

townVC.tabBarItem.title = "동네생활"
townVC.tabBarItem.image = UIImage(named: "townIcon")
townVC.tabBarItem.selectedImage = UIImage(named: "townIconFill")

myaroundVC.tabBarItem.title = "내 근처"
myaroundVC.tabBarItem.image = UIImage(named: "myAroundIcon")
myaroundVC.tabBarItem.selectedImage = UIImage(named: "myAroundIconFill")

chatVC.tabBarItem.title = "채팅"
chatVC.tabBarItem.image = UIImage(named: "chatIcon")
chatVC.tabBarItem.selectedImage = UIImage(named: "chatIconFill")

profileVC.tabBarItem.title = "나의 당근"
profileVC.tabBarItem.image = UIImage(named: "myCarrotIcon")
profileVC.tabBarItem.selectedImage = UIImage(named: "myCarrotIconFill")

Copy link
Member

Choose a reason for hiding this comment

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

탭바 만들때 !

homeVC.tabBarItem = UITabBarItem(title: "", image: UIImage(named: "homeIcon"), selectedImage: UIImage(named: "homeIconFill"))

요렇게 생성자를 사용해서 해주면 한줄로 줄어든답니다 !

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Suyeon9911 오 순배 너무 좋아! 바로 적용해볼께

Copy link
Member

@Suyeon9911 Suyeon9911 left a comment

Choose a reason for hiding this comment

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

해결 못한 이슈들은 줌에서 얘기 or 만나서 고고

Comment on lines +29 to +41

// MARK: - Custom Method
override func viewWillAppear(_ animated: Bool) {
super.viewWillAppear(animated)

navigationController?.setNavigationBarHidden(true, animated: animated)
}

override func viewWillDisappear(_ animated: Bool) {
super.viewWillDisappear(animated)
navigationController?.setNavigationBarHidden(false, animated: animated)
}

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.

저도 이거 써줬어요! 네비게이션컨트롤러 연결하면 투명하게 공간 차지해서 그 아래에 있는 유아이나 버튼이 안 눌리는데 그거 없애주려고 하신듯??

Comment on lines +72 to +81

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
let selectProduct = itemDataList[indexPath.row]
let storyboard = UIStoryboard(name:"ItemDetail", bundle: nil)

if let ivc = storyboard.instantiateViewController(identifier: "ItemDetailVC") as? ItemDetailVC {
ivc.itemDataList = selectProduct
self.navigationController?.pushViewController(ivc, animated: true)
}
}
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.

와 이거 좋다 저도....쇽 샥

guard let cell = tableView.dequeueReusableCell(withIdentifier: ItemTableViewCell.identifier) as? ItemTableViewCell else {return UITableViewCell()}

cell.setData(like: indexPath.row, appData: itemDataList[indexPath.row])
cell.selectionStyle = .none // 셀 선택시 회색 없애기
Copy link
Member

Choose a reason for hiding this comment

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

셀 선택 시 회색 없애는 거 저는 delegate에 didSelectRowAt 에서 해줬어용 ! 그냥 저눈 요러케 했다구 공유한곱니다~

 // 셀 선택 해제 
tableView.deselectRow(at: indexPath, animated: true)
        

Copy link
Member

@L-j-h-c L-j-h-c 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 +29 to +41

// MARK: - Custom Method
override func viewWillAppear(_ animated: Bool) {
super.viewWillAppear(animated)

navigationController?.setNavigationBarHidden(true, animated: animated)
}

override func viewWillDisappear(_ animated: Bool) {
super.viewWillDisappear(animated)
navigationController?.setNavigationBarHidden(false, animated: animated)
}

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 +72 to +81

func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
let selectProduct = itemDataList[indexPath.row]
let storyboard = UIStoryboard(name:"ItemDetail", bundle: nil)

if let ivc = storyboard.instantiateViewController(identifier: "ItemDetailVC") as? ItemDetailVC {
ivc.itemDataList = selectProduct
self.navigationController?.pushViewController(ivc, animated: true)
}
}
Copy link
Member

Choose a reason for hiding this comment

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

와 이거 좋다 저도....쇽 샥

RecommendItemData(recommenditemIconName: "eunjuchoi", recommenditemName: "은주초이", recommenditemPrice: "999,999+원"),
RecommendItemData(recommenditemIconName: "eunjuchoi", recommenditemName: "은주초이", recommenditemPrice: "999,999+원"),
RecommendItemData(recommenditemIconName: "eunjuchoi", recommenditemName: "은주초이", recommenditemPrice: "999,999+원")
])
Copy link
Member

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋㅋㅋㅋㅋ 어이없숴

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants