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

#54 [ui] 행복루틴 삭제하기 뷰 #61

Merged
merged 32 commits into from
Jan 16, 2024

Conversation

pump9918
Copy link
Collaborator

@pump9918 pump9918 commented Jan 15, 2024

📑 Work Description

  • 삭제하기 뷰
  • 삭제 버튼 클릭 시 바텀시트 실행

🛠️ Issue

📷 Screenshot

Screen_recording_20240117_002127.mp4

💬 To Reviewers

  • 바텀시트의 content가 없어 생기는 빈공간 이슈는 emjayMJkim이 AS해주신다고 합니다
  • 스낵바 위치가 제대로 적용이 안되는 이슈가 있는데 확인 가능한가요??

@pump9918 pump9918 self-assigned this Jan 15, 2024
@pump9918 pump9918 marked this pull request as ready for review January 15, 2024 14:33
@pump9918 pump9918 added UI ui 관련 작업 호연🐻‍❄️ 호연이가 작업함! labels Jan 15, 2024
Copy link
Contributor

@stellar-halo stellar-halo left a comment

Choose a reason for hiding this comment

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

고생고생 왕 고생했습ㄴ디ㅏ. 근로가서도,,일하는 당신

android:layout_marginEnd="20dp"
android:layout_marginBottom="34dp"
android:background="@drawable/shape_red_fill_12_rect"
android:text="루틴을 삭제할래"
Copy link
Contributor

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.

아이고;; 감사합니다!

Copy link
Member

@emjayMJkim emjayMJkim left a comment

Choose a reason for hiding this comment

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

수고하셨습니당!!

@pump9918 pump9918 requested a review from minemi00 January 16, 2024 15:28
Copy link
Contributor

@stellar-halo stellar-halo left a comment

Choose a reason for hiding this comment

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

조금만 더 수정해봅시다

isDrawable = true,
imageDrawable = R.drawable.ic_bear_face_crying,
imageUri = "",
title = "정말 삭제할까요?",
Copy link
Contributor

Choose a reason for hiding this comment

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

여기 스트링 있네요~

doBtnAction = {
snackBar(
binding.root.rootView,
"행복루틴을 삭제했어요"
Copy link
Contributor

Choose a reason for hiding this comment

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

스트링 마찬가지입니다

Comment on lines 105 to 106
binding.btnHappyProgressClear.setOnClickListener {
}
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 이 친구는 setOnClickListener안에 아무것도 없는데 왜 써주신 건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

버튼을 누르면 바텀시트를 실행시킬 예정으로 비워둔 부분인데, 함수를 사용해서 없어진 부분을 고려못했습니다. 지금 수정하겠습니다!

…ui-happy-delete-view

# Conflicts:
#	app/src/main/java/com/sopetit/softie/di/RepositoryModule.kt
#	app/src/main/java/com/sopetit/softie/di/RetrofitServiceModule.kt
Copy link
Contributor

@stellar-halo stellar-halo left a comment

Choose a reason for hiding this comment

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

LGTM 놓아줄 때가 왔다!

Copy link
Member

@minemi00 minemi00 left a comment

Choose a reason for hiding this comment

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

뷰가 참 많네요 .. 수고하셨습니당!

@pump9918 pump9918 merged commit 7348b01 into develop Jan 16, 2024
1 check passed
@pump9918 pump9918 deleted the feature/#54-ui-happy-delete-view branch January 16, 2024 16:35
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