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/#55] 캘린더뷰 / 다이얼로그 구현 #128

Merged
merged 24 commits into from
Jul 18, 2024

Conversation

boiledeggg
Copy link
Member

⛳️ Work Description

  • 스크랩 취소 다이얼로그 구현
  • 스크랩 상세 정보 다이얼로그 구현

📸 Screenshot

default.mp4

📢 To Reviewers

구현 예정 목록

  • "스크랩 취소 버튼 클릭 -> 스크랩 취소 API 호출" 구현 예정
  • "스크랩 색상 수정 버튼 클릭 -> 스크랩 색상 수정 API 호출" 구현 예정
  • "스크랩 공고 정보 보러가기 버튼 클릭 -> 공고 상세정보 페이지 네비게이션" 구현 예정

안내 사항

  • Timber는 현재 다이얼로그에서 값이 제대로 반환되는지 확인하기 위함입니다!
  • 다이얼로그 크기는 일단 고정 dp로 구현했습니다. 내부 컴포넌트 배치가 너무 힘들어져서 일단 넘어가도록 할게요!

…alog

# Conflicts:
#	feature/src/main/java/com/terning/feature/intern/component/ScrapCancelDialog.kt
@boiledeggg boiledeggg added UI 💐 UI 작업 석준💜 석준 labels Jul 18, 2024
@boiledeggg boiledeggg self-assigned this Jul 18, 2024
Copy link
Member

@Hyobeen-Park Hyobeen-Park left a comment

Choose a reason for hiding this comment

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

LGTM~~~ 짱이에요!!!

text = R.string.dialog_scrap_cancel_button,
onButtonClick = /*{
viewModel.updateScrapped(!state.isScrapped)
viewModel.updateScrapDialogVisible(false)
Copy link
Member

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.

앗 지워야겠네여,,

Copy link
Member

@leeeyubin leeeyubin 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 109 to 114
text = R.string.dialog_scrap_cancel_button,
onButtonClick = /*{
viewModel.updateScrapped(!state.isScrapped)
viewModel.updateScrapDialogVisible(false)
}*/onClickScrapCancel,
modifier = Modifier.padding(bottom = 8.dp)
Copy link
Member

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.

지울게요!!

Comment on lines 61 to 64
.border(
width = 0.dp,
color = Grey200,
shape = RoundedCornerShape(bottomStart = 20.dp, bottomEnd = 20.dp)
Copy link
Member

Choose a reason for hiding this comment

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

width = 0.dp면 이 코드 안 써줘도 반영이 되지 않을까요,,,,? (저도 확실하지 않음,,ㅎ)

Copy link
Contributor

@arinming arinming left a comment

Choose a reason for hiding this comment

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

수고하셨어용~~~~! 다이얼로그 패딩만 좀 조정해주시면 초ㅣ고일듯요(지금도최고임)

@boiledeggg boiledeggg merged commit 4d96b93 into develop Jul 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
UI 💐 UI 작업 석준💜 석준
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UI] 캘린더뷰 / 다이얼로그 구현
4 participants