[Feat/IPLC-11] 취득예정 바텀시트 UI #187
Conversation
…바텀시트-UI # Conflicts: # app/src/main/res/values/strings.xml
MinseoSONG
left a comment
MinseoSONG
left a comment
There was a problem hiding this comment.
수고 많앗슨 ㅠㅠ
코리 확인해줘 ~~~~~~~~~~~
| // FIXME Sample 서버데이터 | ||
| val place1List = listOf("서울", "경기", "부산", "인천", "충남", "충북", "강원", "경북") | ||
| val place2List = listOf("서울", "경기", "부산", "인천", "충남", "충북", "강원", "경북") |
There was a problem hiding this comment.
[3]: 별거는 아니지만 추후 수정사항을 최소한으로 하기 위해서 이걸 미리 인자로 빼놓는게 어떨까 ~ 싶어용
서버에서 받아올 값들이니까 !
There was a problem hiding this comment.
[1]: 잘햇는데 전체적으로 기기대응 확장 함수 빠진 부분들이 좀 있는 것 같아 ~~~! 확인 한번 해주라
| }, | ||
| onMonthChanged = {}, | ||
| modifier = Modifier | ||
| .zIndex(1f) |
There was a problem hiding this comment.
캘린더 펼쳤을때 아래에 있는 view들보다 z값상 위에 있어야 하기 때문에 넣었습니다
| ) | ||
|
|
||
| Text( | ||
| text = ":", |
|
|
||
| // 시간 Picker | ||
| TimePickerColumn( | ||
| items = (1..12).map { it.toString() }, |
There was a problem hiding this comment.
[2]: 아래 분 picker도 마찬가지고 이것도 그렇고 여기서 이렇게 때려박는것보다 좋은 방법이 없을까 ..
기능적으로 문제는 전혀 없지만 과연 좋은 코드인가?에 대한 생각을 하게 됨. 지만 문제없으니 굳이 수정할 필요는 없음 ! 지만 한번 생각해보는 것도 좋을 듯 ~
|
|
||
| // 시간 Picker | ||
| TimePickerColumn( | ||
| items = (1..12).map { it.toString() }, |
There was a problem hiding this comment.
[1]: 그리고 분 picker에서는 10 이하 자연수들에 0 붙이는거 되어있는데 여기서는 안되고 있는 것 같아서 확인 한번 해줘 ~~
| Box( | ||
| modifier = Modifier | ||
| .fillMaxWidth() | ||
| .height(2.dp) | ||
| .background(CertiTheme.colors.purpleBlue) | ||
| .align(Alignment.TopCenter) | ||
| ) | ||
| // 하단 파란색 라인 | ||
| Box( | ||
| modifier = Modifier | ||
| .fillMaxWidth() | ||
| .height(2.dp) | ||
| .background(CertiTheme.colors.purpleBlue) | ||
| .align(Alignment.BottomCenter) | ||
| ) |
There was a problem hiding this comment.
[2]: 얘네를 꼭 box로 구현한 이유가 있을까? divider를 사용했다면 뭔가 의도상 더 깔끔했을 것 같은데 !
Work Description ✏️
Screenshot 📸
2025-12-01.11.56.12.mov