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

Feature. 내 리뷰 관리 UI 작업 #177

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

spqjf12345
Copy link
Contributor

이슈

#175

수정사항

(더미 데이터로 UI만 구현하였습니다.)

  • google-mobile-ad package 버전 up to 11.5
  • ProductDetailReviewCell

버튼 ActionButtonCell로 분리
image
image
reviewTagListView 길어지는 UI 이슈 수정

  • ProductInfoStackView

리뷰 작성시 상단 상품 정보 뷰 재사용을 위해 ProductInfoStackView로 분리
image
image

  • ProfileHomeViewController

viewHolder 파일 분리
캐릭터 슬롯 & 내 리뷰 섹션 추가

  • 내 리뷰(MyReview), 상세 리뷰(MyDetailReview) 리블렛 생성
Simulator.Screen.Recording.-.iPhone.15.Pro.Max.-.2024-06-02.at.18.33.59.mp4

@spqjf12345 spqjf12345 requested a review from inho-98 June 2, 2024 09:39
@spqjf12345 spqjf12345 self-assigned this Jun 2, 2024
@spqjf12345 spqjf12345 added feature 신규 기능 추가 UI 화면 추가 작업 d-n PR 리뷰 천천히 labels Jun 2, 2024
Copy link
Contributor

@inho-98 inho-98 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~!
추후에 API 연결해서 네트워킹 필요한 부분들 구현하면 될 것 같습니다! 👍🚀

Comment on lines +11 to +16
class ProductInfoStackView: UIStackView {
enum Mode {
case starRating // 별점
case date // 날짜
case tastes // 취향 태그
}
Copy link
Contributor

Choose a reason for hiding this comment

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

🤩👍

Comment on lines +36 to +38
private var dataSource: DataSource?
private var productDetail: ProductDetailEntity?
private var review: ReviewEntity?
Copy link
Contributor

Choose a reason for hiding this comment

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

[C]
아래의 applySnapshot(with: , review:) 함수의 매개변수를 외부에서 전달받도록 하고 뷰컨 내부에서는 productDetail, review 같은 정보를 모르게 하는 방법은 어떨까요?
(뷰컨이 멍청하도록..!)

}

func tableView(_ tableView: UITableView, heightForRowAt indexPath: IndexPath) -> CGFloat {
return UITableView.automaticDimension
Copy link
Contributor

Choose a reason for hiding this comment

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

automaticDimension 지정해준 이유가 셀마다 다른 높이값 주기 위해서 인가요?


import UIKit

struct MyReviewContentConfiguration: UIContentConfiguration {
Copy link
Contributor

Choose a reason for hiding this comment

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

UIContentConfiguration 활용하는 방법 덕분에 배웠습니다 👍

class MyReviewContentView: UIView, UIContentView {
static let identifier = "MyReviewContentView"

var configuration: any UIContentConfiguration
Copy link
Member

Choose a reason for hiding this comment

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

[C]
오~ 처음 보는 친구네요 👍

Copy link
Member

@KakaoTocs KakaoTocs left a comment

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
Labels
d-n PR 리뷰 천천히 feature 신규 기능 추가 UI 화면 추가 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants