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

Feat: CdListView #13

Merged
merged 25 commits into from
Jun 16, 2022
Merged

Feat: CdListView #13

merged 25 commits into from
Jun 16, 2022

Conversation

Eunbi-Cho
Copy link
Collaborator

Feat: CdListView 파일 분리 및 쓸모 없는 파일 삭제
Feat: CdListView CD inner shadow 추가

@Eunbi-Cho Eunbi-Cho changed the base branch from anna2 to main June 16, 2022 08:08
@JIWON1923
Copy link
Member

확인했습니다!

@Published var screenDrag: Float = 0.0
}

struct Carousel<Items : View> : View {
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.

넵!

Comment on lines +21 to +30
// Text("아직 기록된 음악이 없어요")
// .font(Font.customTitle2())
// .foregroundColor(.titleGray)
// .padding(.bottom, 12)
// .padding(.top, 260)
// Button("음악 기록하러 가기") {
// //go to WriteView
// }
// Spacer()
// Image("ListViewCdPlayer")
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.

리스트가 없을 때 쓰는 코드라 필요한 코드입니다! 아직 데이터를 안받아와서 주석처리 해놨어요!

var image: String = ""
}

public class UIStateModel: ObservableObject {
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.

넵!

let cdHeight: CGFloat = 300

// 기록물 id, 곡제목, 가수명, 앨범커버 배열 예시
let cds = [
Copy link
Collaborator

Choose a reason for hiding this comment

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

cd list가 추가되거나 삭제될 수 있으니까
var 로 수정하는게 좋을 것 같습니다.

Suggested change
let cds = [
var cds = [

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

네! 수정하겠습니다~!

@100seo
Copy link
Collaborator

100seo commented Jun 16, 2022

확인했습니다

1 similar comment
@JIWON1923
Copy link
Member

확인했습니다

@Eunbi-Cho Eunbi-Cho merged commit d35b499 into DeveloperAcademy-POSTECH:main Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants