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] 번개 생성 개최 뷰 - 시간 / 날짜 뷰 구현 #43

Merged
merged 13 commits into from
Jan 6, 2024

Conversation

Dan2dani
Copy link
Collaborator

@Dan2dani Dan2dani commented Jan 6, 2024

Related issue 🛠

Work Description ✏️

  • 번개 생성 개최 뷰 - 시간 / 날짜 뷰 구현

Screenshot 📸

HD20240106_172127.1.mp4

Uncompleted Tasks 😅

  • 시간 텍스트 포맷에 맞춰 뷰에 띄우기
  • 스낵바 띄우기
  • 버튼 활성화 적용

To Reviewers 📢

@Dan2dani Dan2dani added this to the 1차 스프린트 뷰 작업 milestone Jan 6, 2024
@Dan2dani Dan2dani self-assigned this Jan 6, 2024
@Dan2dani Dan2dani requested a review from a team as a code owner January 6, 2024 08:30
Copy link
Collaborator

@jihyunniiii jihyunniiii left a comment

Choose a reason for hiding this comment

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

잘했숨니당 ㅋ.ㅋ

Comment on lines +70 to +75
companion object {
const val HOUR_MIN = 1
const val HOUR_MAX = 12
const val MINUTE_MIN = 0
const val MINUTE_MAX = 59
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

잘했어욤 ~

_planDate.value = planDate
}

fun setSelectedTimeType(timeType: String) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

쩐당


<androidx.constraintlayout.widget.ConstraintLayout
android:layout_width="match_parent"

Copy link
Collaborator

Choose a reason for hiding this comment

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

에바핑 ㅋㅋ

Comment on lines +22 to +26
android:text="Done"
android:textAlignment="textEnd"
android:textColor="@color/pingle_green"
android:textSize="18dp"
app:layout_constraintEnd_toEndOf="parent"
Copy link
Collaborator

Choose a reason for hiding this comment

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

폰트 다운받아서 해주세영


fun setCurrentPage(position: Int) {
_currentPage.value = position
}

fun setPlanDate(planDate: String) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

우하하 이런 식으루 뷰모델에 저장하는 것이로군 쇽샥!!!!!!!!!!!!!!!!!!!!!!!

@Dan2dani Dan2dani merged commit 5b73f79 into develop Jan 6, 2024
1 check failed
@jihyunniiii
Copy link
Collaborator

담부터 린트 체크 안 돌아가고 머지하면 혼난당 ~

@jihyunniiii jihyunniiii deleted the feat-plan-date-time-view branch January 7, 2024 20:10
Copy link
Collaborator

@jihyunniiii jihyunniiii left a comment

Choose a reason for hiding this comment

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

그만 잘해 이다은

Comment on lines +6 to +10
enum class MeridiemType(
@StringRes val meridiemStringRes: Int
) {
AM(R.string.plan_time_am), PM(R.string.plan_time_pm)
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

타입 관리 미쳤네염

android:background="@color/g_11"
android:paddingVertical="7dp"
android:paddingEnd="15dp"
android:text="Done"
Copy link
Collaborator

Choose a reason for hiding this comment

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

추출 부탁합니둥

Comment on lines +29 to +58
<NumberPicker
android:id="@+id/np_date_picker_year"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginVertical="20dp"
android:layout_marginStart="44dp"
android:theme="@style/AppTheme.NumberPicker"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toBottomOf="@+id/tv_date_picker_done" />

<NumberPicker
android:id="@+id/np_date_picker_month"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:theme="@style/AppTheme.NumberPicker"
app:layout_constraintBottom_toBottomOf="@+id/np_date_picker_year"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="@+id/np_date_picker_year" />

<NumberPicker
android:id="@+id/np_date_picker_day"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginEnd="44dp"
android:theme="@style/AppTheme.NumberPicker"
app:layout_constraintBottom_toBottomOf="@+id/np_date_picker_month"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintTop_toTopOf="@+id/np_date_picker_month" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 체인 거는 게 나을 것 같습니다요

android:background="@color/g_11"
android:paddingVertical="7dp"
android:paddingEnd="15dp"
android:text="Done"
Copy link
Collaborator

Choose a reason for hiding this comment

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

추출 해주세용

Comment on lines +29 to +58
<NumberPicker
android:id="@+id/np_time_picker_meridiem"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginVertical="20dp"
android:layout_marginStart="44dp"
android:theme="@style/AppTheme.NumberPicker"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toBottomOf="@+id/tv_time_picker_done" />

<NumberPicker
android:id="@+id/np_time_picker_hour"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:theme="@style/AppTheme.NumberPicker"
app:layout_constraintBottom_toBottomOf="@+id/np_time_picker_meridiem"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="@+id/np_time_picker_meridiem" />

<NumberPicker
android:id="@+id/np_time_picker_minute"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_marginEnd="44dp"
android:theme="@style/AppTheme.NumberPicker"
app:layout_constraintBottom_toBottomOf="@+id/np_time_picker_hour"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintTop_toTopOf="@+id/np_time_picker_hour" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

얘두 ~

android:paddingVertical="@dimen/spacing16">

<TextView
android:id="@+id/tv_title"
Copy link
Collaborator

Choose a reason for hiding this comment

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

네이밍 tv_text_with_title_title 이런 식으로 부탁티비요 ~
유형_어느 뷰인지_뭔지

Copy link
Collaborator

@HAJIEUN02 HAJIEUN02 left a comment

Choose a reason for hiding this comment

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

💥💛💪💥

@@ -10,11 +10,13 @@ import org.sopt.pingle.R
import org.sopt.pingle.databinding.EditTextPingleBinding

@SuppressLint("CustomViewStyleable")
class PingleEditText(
class PingleEditText @JvmOverloads constructor(
Copy link
Collaborator

Choose a reason for hiding this comment

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

오와 생성자 오버로딩을 자동으로 생성해주는 어노테이션이라니 나중에 이런 식으로 커스텀뷰 만들 때 참고할게요~!

android:theme="@style/AppTheme.NumberPicker"
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toBottomOf="@+id/tv_date_picker_done" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

여기 제약에 @id 사용해주세욥

app:layout_constraintBottom_toBottomOf="@+id/np_date_picker_year"
app:layout_constraintEnd_toEndOf="parent"
app:layout_constraintStart_toStartOf="parent"
app:layout_constraintTop_toTopOf="@+id/np_date_picker_year" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

전부!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[feat] 핑글 개최 프로세스 - 시간/날짜 뷰 구현
3 participants