Skip to content

[fix/#326] 수집함 QA 반영#329

Merged
pedro0527 merged 6 commits intodevelopfrom
fix/#326-archive-qa
Nov 15, 2025
Merged

[fix/#326] 수집함 QA 반영#329
pedro0527 merged 6 commits intodevelopfrom
fix/#326-archive-qa

Conversation

@pedro0527
Copy link
Copy Markdown
Contributor

📄 작업 내용

  • 컴포넌트 카드 간격 조정
  • 스크롤뷰 바텀 패딩 100으로 수정
구현 내용 iPhone 13 mini iPhone 16 pro
수집함 간격 조정
리스트 간격 조정

💻 주요 코드 설명

그리드 프레임 크기 조절

private let columns = [GridItem(.fixed(171.adjustedWidth)), GridItem(.fixed(171.adjustedWidth))]

Figma에는 프레임의 크기가 165고 그 사이 간격이 12인데여...! 그래서 각 프레임마다 6의 크기를 늘려서 그 사이 간격을 추가했어요

근데.... 스유에서 이 그리드를 잘 모르겠어여 그냥 165로 해도 사이에 간격이 있긴한디!!

🔗 연결된 이슈

@pedro0527 pedro0527 requested a review from a team November 10, 2025 13:47
@pedro0527 pedro0527 self-assigned this Nov 10, 2025
@pedro0527 pedro0527 added the 🐳 sooyong 수용이가함! label Nov 10, 2025
@pedro0527 pedro0527 linked an issue Nov 10, 2025 that may be closed by this pull request
1 task
@pedro0527 pedro0527 added the ✂️ fix 버그나 오류 해결 시 사용 label Nov 10, 2025
Copy link
Copy Markdown
Collaborator

@SeungWon1125 SeungWon1125 left a comment

Choose a reason for hiding this comment

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

고생했수용!


private let archiveCategory: SolplyContentType
private let columns = [GridItem(.fixed(165.adjustedWidth)), GridItem(.fixed(165.adjustedWidth))]
private let columns = [GridItem(.fixed(171.adjustedWidth)), GridItem(.fixed(171.adjustedWidth))]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

private let columns = [
    GridItem(.fixed(165.adjustedWidth), spacing: 12.adjustedWidth),
    GridItem(.fixed(165.adjustedWidth))
]

요렇게도 해보셨수용?

Copy link
Copy Markdown
Contributor

@dudwntjs dudwntjs left a comment

Choose a reason for hiding this comment

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

스크린샷 2025-11-10 오후 11 46 34

공식문서

private let columns: [GridItem] = [
    GridItem(.fixed(165.adjustedWidth), spacing: 12.adjustedWidth),
    GridItem(.fixed(165.adjustedWidth), spacing: 12.adjustedWidth)
]

var body: some View {
    ScrollView {
        LazyVGrid(
            columns: columns,
            spacing: 12.adjustedHeight
        ) {
            // 여튼 이런식으로...
    }
}

가로 간격은 GridItem.spacing, 세로 간격은 LazyVGrid(spacing:) 넣는 게 어떠신지!!!!!!
사실 잘되는 지는 확인 안해봣는데 수수수수수용은 할 수 잇으삼

@SeungWon1125
Copy link
Copy Markdown
Collaborator

좋은디요 👍🏻 @dudwntjs

Copy link
Copy Markdown
Collaborator

@SeungWon1125 SeungWon1125 left a comment

Choose a reason for hiding this comment

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

나이수용!

@pedro0527 pedro0527 merged commit 434fd2f into develop Nov 15, 2025
@pedro0527 pedro0527 deleted the fix/#326-archive-qa branch November 15, 2025 03:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✂️ fix 버그나 오류 해결 시 사용 🐳 sooyong 수용이가함!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[fix] 수집함 QA 반영

3 participants