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

[Style] #33 - 홈화면 UI 구현 #40

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

[Style] #33 - 홈화면 UI 구현 #40

wants to merge 15 commits into from

Conversation

joho2022
Copy link
Collaborator

🌴 작업한 브랜치

✅ 작업한 내용

  • 홈화면 UI 구현

❗️PR Point

LazyVGrid(columns: columns, spacing: 20) {
                    if selectedIndex == 0 {
                        ForEach(sellProducts) { product in
                            ProductItemView(
                                product: product,
                                isLikeButtonExist: true,
                                width: 160
                            )
                        }
                    } else if selectedIndex == 1 {
                        ForEach(buyProducts) { product in
                            ProductItemView(
                                product: product,
                                isLikeButtonExist: true,
                                width: 160
                            )
                        }
                    }
                }
                .frame(height: 254)

공통 상품셀에서 다른 뷰에서 호출될 때의 예시입니다.

📸 스크린샷

기능 스크린샷
GIF

📟 관련 이슈

@joho2022 joho2022 added Refactor 🏗️ 코드 리팩토링 Style 🖼️ UI 작업 호근 🍙 호근 작업 labels Jan 17, 2025
@joho2022 joho2022 self-assigned this Jan 17, 2025

}

#Preview {
Copy link
Collaborator

Choose a reason for hiding this comment

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

프리뷰 삭제 부탁드랴요 ~!^^^6

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

빼꼼 1회 추가

Copy link
Collaborator

@lalaurrel lalaurrel left a comment

Choose a reason for hiding this comment

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

고생하셨슴다 ~^^

{
"images" : [
{
"filename" : "img_ready.svg",
Copy link
Collaborator

Choose a reason for hiding this comment

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

안쓰이는 svg 파일 삭제 해주세요 ~^^^^

Copy link
Collaborator

@OneTen19 OneTen19 left a comment

Choose a reason for hiding this comment

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

LGTM👍

코드 깔끔하니 보기 좋네요~

Comment on lines 10 to +24
struct HomeView: View {

// MARK: - Properties

@State private var currentPage: Int = 0
private let bannerImages = ["img_banner_1", "img_banner_2"]

@State var recomendedProducts = ProductModel.recommendedDummyList()
@State var sellProducts = ProductModel.sellDummyList()
@State var buyProducts = ProductModel.buyDummyList()

let width = (UIScreen.main.bounds.width - 55) / 2

// MARK: - Main Body

Copy link
Collaborator

Choose a reason for hiding this comment

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

빼꼼 호근핑

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

빼꼼 한열핑

@joho2022 joho2022 closed this Jan 19, 2025
@joho2022 joho2022 deleted the style/#33 branch January 19, 2025 11:55
@joho2022 joho2022 restored the style/#33 branch January 19, 2025 11:57
@joho2022 joho2022 reopened this Jan 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactor 🏗️ 코드 리팩토링 Style 🖼️ UI 작업 호근 🍙 호근 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Style] 홈화면 UI 구현
3 participants