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

보관함 탭 UI 개발 #37

Merged
merged 14 commits into from
Jun 29, 2024
Merged

보관함 탭 UI 개발 #37

merged 14 commits into from
Jun 29, 2024

Conversation

ddyeon
Copy link
Member

@ddyeon ddyeon commented Jun 29, 2024

개요

이슈 링크 혹은 PR 내용 요약

  • 보관함 탭 UI 개발

작업 내용

실제 작업 내용

  • Storage 모듈 추가하고 UI 개발했습니다~! state는 우선 임시로 만들어뒀는데 나중에 API나오고 변경 필요할 것 같아유!
  • StorageListState에 넣어둔 default value는 나중에 삭제하도록 하겠습니당

시연 화면 (option)

실행 스크린샷 혹은 영상 첨부

보관함 탭 UI 개발
스크린샷 2024-06-29 오후 7 04 49

To Reviers

리뷰어들에게 전할 말
thank you.

Close

close #30

@ddyeon ddyeon self-assigned this Jun 29, 2024
@ddyeon ddyeon added the feature develop feature label Jun 29, 2024
@@ -0,0 +1 @@
/build
Copy link
Collaborator

Choose a reason for hiding this comment

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

추가하면 조을듯 ~

.padding(vertical = 14.dp, horizontal = 12.dp),
) {
Row(
modifier = Modifier.align(Alignment.CenterStart).clickable { navigateToStorageDetail() },
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
Member Author

Choose a reason for hiding this comment

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

수정 완료!!

}

data class StorageFolderItem(
val folderTitle: String = "",
Copy link
Collaborator

Choose a reason for hiding this comment

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

타이틀 받지말구 객체 받아서 안에서 title 꺼내죠
위에서 .title 계속 뽑는게 별루인듯 ㅠ

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 수정 완료!!

@ddyeon ddyeon merged commit f5559ec into mash-up-kr:develop Jun 29, 2024
2 checks passed
@ddyeon ddyeon deleted the feature/storage branch June 29, 2024 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature develop feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

보관함 탭 UI 개발
2 participants