Feature/#13 seats screen UI #16

tunaunnie · 2024-11-22T17:52:31Z

📌 Issue

closed [Feature] seats / SeatsScreen ui 구현 #13

💻 Work Description

SeatSelection 스크린(UI)과, 그 안에 들어가는 컴포넌트들을 구현했습니다.
- SeatScreenTopBar 구현
- TimeCompCardRow 구현
- SeatSelectionChangeButton 구현
- SeatSelectionChipRow 구현
- SeatSelectionConfirmRow 구현
- SeatConfirmationModal 구현

📸 Screenshot

https://github.com/user-attachments/assets/6c453100-438f-415b-addb-c8a6d1d0a5f2
-> 원래 SeatScreen 처음에 진입하자마자 모달이 하나 뜨는 것까지 구현한 상태인데 (코드에 들어가 있음), 에뮬레이터에서 자꾸 작동이 안 돼서 첫 번째 모달 띄우는 부분만 주석처리하고 찍은 영상입니다..! 모달 컴포넌트만 따로 프리뷰로 보면 잘 동작하는데 혹시 클론받아서 두 개의 모달이 다 잘 뜨는지 확인해주시면.. 정말 감사하겠습니다 ㅜㅠ

https://github.com/user-attachments/assets/6127f952-0e9b-4b62-abb0-e5fc9b2a010b

https://github.com/user-attachments/assets/38679544-c42a-41c0-a64e-a37e39fcb3ef

💭 To Reviewers

-SeatSelectionModal1에서 총 선택한 인원 수가 0명 이상인 경우에 '좌석 선택' 버튼이 빨갛게 활성화되는 코드도 비즈니스 로직이라고 생각해서 우선은 내버려 뒀습니다! (TimeCompCard 활성화도 마찬가지) 차차 해당 부분을 고쳐봐도 괜찮을 것 같다면, 리뷰 남겨주시면 바로바로 반영하겠습니다! 지금 코드 상황에서 고쳐가야 할 부분을 순차적으로 알려주시면 조금씩 리팩토링해보겠습니닷

…N-ANDROID-CGV into ui/#7-seats-screen-components

…N-ANDROID-CGV into UI/#7-seats-screen-components

…PT-all/35-COLLABORATION-ANDROID-CGV into feature/#13-seats-screen-ui

0se0

솝커톤 하느라 바쁠텐데.. 이랬다 저랬다 하는 리드라 미안해...!😖

0se0 · 2024-11-23T13:10:41Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/card/CompTimeCard.kt

+
+    Box(
+        modifier = modifier
+            .size(width = 90.dp, height = 64.dp)


이렇게 컴포넌트 사이즈를 고정 dp로 주는 것이 아닌 컴포넌트 요소들의 패딩 값으로 조절해준 후 스크린에 넣었을 때, 전체 스크린에서의 디피로 조절해주면 더 좋을 거 같아요

0se0 · 2024-11-23T14:43:07Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/card/CompTimeCard.kt

+    startTime: LocalDateTime,
+    endTime: LocalDateTime,


양지발이야... 내가 미안해...
오늘 서버 통신을 배우며 생각하였는데,
이 부분 string으로 주는 것이 맞는 것 같애.. 너는 다 계획이 있었구나..
못난 리드라 미안해....

ㅋㅋㅋㅋ 뒷걸음정답 ...🫢 괜차나 이러면서 더 배우게 되는 것 같당!!

0se0 · 2024-11-23T14:44:23Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/card/CompTimeCard.kt

+    val timeFormatter = DateTimeFormatter.ofPattern("HH:mm")
+    val formattedStartTime = startTime.format(timeFormatter)
+    val formattedEndTime = endTime.format(timeFormatter)


이 부분이 시간 포멧팅 부분이여서 비즈니스 로직에 가깝다는 생각이 지금 들었사옵니다.. 서버 통신할 때 뷰모델 부분에서 시간 포멧팅을 해주면 될 것 같사옵니다..
미안하옵니다....🥹

0se0 · 2024-11-23T14:45:29Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/chip/Chip.kt

+    modifier: Modifier = Modifier,
+    content: String
+){
+
+    Box(
+        modifier = Modifier


Suggested change

modifier: Modifier = Modifier,

content: String

){

Box(

modifier = Modifier

modifier: Modifier = Modifier,

content: String

){

Box(

modifier = modifier

0se0 · 2024-11-23T14:48:26Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectionChangeButton.kt

+
+    Box(
+        modifier = modifier
+            .size(width = 66.dp, height = 24.dp)


여기도 이렇게 전체 사이즈를 확정지어 주는 것이 아닌, 패딩으로 조절하면 좋을 거 같아요!
혹여나 이렇게 하신 이유가 따로 있을까요?

하드코딩한 다른 이유가 있는 건 아니고, 패딩값을 인자로 주어서 동적으로 크기가 조절되도록 하는 방식을 더 일반적으로 사용한다는 사실을 잘 몰랐어요..! 세영쓰가 버튼 구현한 거 보고 완벽하게 이해해갑니당.. 🤩

0se0 · 2024-11-23T15:02:20Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectionConfirmRow.kt

+            Text(
+                text = price,
+                style = Typography.head8_b_20,
+                color = Color.Black
+            )
+            Text(
+                text = "원",
+                style = Typography.head4_b_15,
+                color = Color.Black


여기 컬러두요!

0se0 · 2024-11-23T15:02:55Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectionStepperRow.kt

+        Text(
+            text = label,
+            style = Typography.body4_m_15,
+            color = Color.Black


여기 컬러 한번 봐주세용

0se0 · 2024-11-23T15:04:12Z

app/src/main/java/org/sopt/cgv/feature/seats/Stepper.kt

+                contentAlignment = Alignment.Center
+            ) {
+                Icon(
+                    painter = painterResource(id = R.drawable.ic_seats_minus),


아이콘 페인터 리소스 말고 벡터로 사용해주세요!
혹시 ic_seats_minus가 png라면 painterResource로 해주셔요!

덕분에 컴포즈에서 벡터 아이콘을 사용할 때는 painterResoucre보다 vectorResource 사용하는 것이 더 최적하된 방법이라는 사실 알아갑니당 . .

사실 이름부터 painterResource인데 내가 넘 의심없이 사용한 것 같당. . 바보

0se0 · 2024-11-23T15:04:26Z

app/src/main/java/org/sopt/cgv/feature/seats/Stepper.kt

+                modifier = Modifier
+                    .padding(horizontal = 5.dp)
+                    .size(width = 34.dp, height = 28.dp)
+                    .background(Color.White, RoundedCornerShape(8.dp)),


여기도 색상 한번만 확인하기!!

0se0 · 2024-11-23T15:04:49Z

app/src/main/java/org/sopt/cgv/feature/seats/Stepper.kt

+                Icon(
+                    painter = painterResource(id = R.drawable.ic_seats_plus),


여기도 painterResource 확인해주셔요!

0se0

추가적으로 seats 부분에서만 사용하는 component는 component 패키지를 따로 만들어서 그 안에 파일들을 넣어주세요!!

kamja0510

짱짱맨 리뷰 답장 잘 확인할게~

kamja0510 · 2024-11-25T08:36:58Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/card/CompTimeCard.kt

+        ){
+            //시간 부분
+            Row(
+                modifier = modifier


modifier를 계속 물려받으면 계산할 때 조금 힘들지 않나요?

세영이가 modifier: Modifier = Modifier, 이거 모든 컴포넌트에 추가해주라고 해서 다 인자로 넘겨받은 modifier를 물려받아 사용하는 것으로 해뒀는데, 상위에서 modifier를 인자로 넘겨받기만 하구 사용하지 않는건가용??

일단 저도 다 추가는 해놓구 부모계층(?)의 영향을 받는 아이들이 있을때만 modifier를 쓰긴해요
예를 들면 부모가 할당받은 width의 1/4을 받고 싶으면 부모 컴포넌트로 부터 Modifier.weight(0.25f) 를 물려받아서 쓰는 방식으로??
세영이 방법이 더 맞을 수도! 전 뭔가 신경쓸게 많아져서 안쓰게 되더라구요

kamja0510 · 2024-11-25T08:38:09Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/card/CompTimeCard.kt

+    ){
+        Column(
+            modifier = modifier.fillMaxSize(),
+            verticalArrangement = Arrangement.SpaceBetween


이 부분이 필요할까요?

진짜 빼두 괜찮네요..🫢 아직도 정렬에 익숙하지 않아서 spacer나 align을 덕지덕지 붙이지 않고도 예쁘고 깔끔하게 배치하는 법을 잘 모르겠어요ㅜㅠ ..

kamja0510 · 2024-11-25T08:38:43Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/card/CompTimeCard.kt

+                    verticalAlignment = Alignment.CenterVertically,
+                    horizontalArrangement = Arrangement.SpaceAround,
+                ){
+                    if(isMorning){


너무 좋네요

kamja0510 · 2024-11-25T08:40:07Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/chip/Chip.kt

이 파일은 제가 더 고쳐놓겠쑵니다 닫기 아이콘 있는 버전으로

세영님 코드리뷰도 반영해뒀습니당 (cgvButton처럼 인자로 패딩값 받아서 동적으로 칩 크기 변경되도록)

kamja0510 · 2024-11-25T09:01:35Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatConfirmationModal.kt

+
+@OptIn(ExperimentalMaterial3Api::class)
+@Composable
+fun SeatConfirmationModal(


preview를 보니 modal의 상단에 당길 수 있는 표시?가 남아있는 거 같은데 제거해 보심이 어떠신가요!

modalBottomSheet의 인자에 dragHandle = { } 을 추가하니까 엄청 간단하게 없어지네요!

눈썰미가 다들 대단하네요 ... 재민님은 진짜 클라의 자질이 있으신거같아요 ..

캬 혼자 알아내시다니
감사합니다 ㅎ

kamja0510 · 2024-11-25T09:08:34Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatConfirmationModal.kt

+                    isBack = true
+                )
+
+                Spacer(modifier = Modifier.width(16.dp))


이런것 보면은 디자인의 사이 간격은 12px이라고 나와있는데 16dp를 주는 이유가 궁금합니다! 진짜 질문

px와 dp의 차이를 고려해서 변환한 거였다면 정말 멋진대답이었겠지만.. 실수로 잘못넣은거라 잽싸게 수정했습니당ㅜ

ㅋㅋㅋㅋㅋㅋㅋㅋ 나이스

kamja0510 · 2024-11-25T09:14:23Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatConfirmationModal.kt

+        ) {
+            Text(
+                text = movieTitle,
+                style = Typography.head6_b_17,


어떨때는 CGVTheme.typography.head6_b_17 로 적용하시다가 어쩔때는 이렇게 적용하시는데 하나로 통일 하시는게 좋을 것 같아요! 저는 CGVTheme의 typography를 쓴다고 명시하면 좋을 것 같긴합니다

kamja0510 · 2024-11-25T09:16:25Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectionChipRow.kt

+    contents: List<String>,
+){
+    Row(
+        horizontalArrangement = Arrangement.spacedBy(8.dp, Alignment.CenterHorizontally),


오우 이건 어떤 문법이죵?

spacedBy도 쓰고싶고 중앙정렬도 하고싶은데 horizontalArrangment를 두번 쓰면 오류가 나길래 찾아보니까 이렇게 한번에도 쓸 수 있더라고요! 더 간단한 방법으로 가독성 좋게 고칠 방법이 있을까요??

바로 생각나는 거는 없네요? 좋은 것 배운거같아요

kamja0510 · 2024-11-25T09:16:59Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectionConfirmRow.kt

+    ) {
+
+        Row(
+            modifier = Modifier,


이런줄은 필요 없을 것 같아요!

kamja0510 · 2024-11-25T09:19:21Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatConfirmationModal.kt

+                horizontalArrangement = Arrangement.Center
+            ) {
+                CgvButton(
+                    text = "뒤로가기",


confirm창에는 뒤로가기가 없는데 혹시 디자인이 변경되었난요?

바로 고쳤습니다ㅜㅜ 꼼꼼히 봐주셔서 너무너무 감사합니다.!!!!

kamja0510 · 2024-11-25T12:14:30Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatConfirmationModal.kt

+        onDismissRequest = {  },
+        sheetState = rememberModalBottomSheetState(),
+        shape = RoundedCornerShape(topStart = 16.dp, topEnd = 16.dp),
+        containerColor = Color.White


발견!
이래서 저는 놓진 부분이 있을 수 있으니 ctrl+R을 눌러서 검색을 한다음에 Replace를 해요
무슨 말인지 모르겠으면 연락하세용

전체 프로젝트에서 검색할땐 ctrl + shift + R

수정 완료!!

0se0

우왕 너무 수고 많으셨습니다~! 훨씬 보기 좋아졌네요!
제가 남긴 리뷰들 한번 확인해주시고 수정해주시면 좋을 거 같아요!

0se0 · 2024-11-25T15:12:49Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/card/CompTimeCard.kt

+    val backgroundColor = if (isActivated) White else Gray700
+    val backgroundColor2 = if (isActivated) Gray200 else Gray700
+    val leftSeatsColor = if (isActivated) PrimaryRed400 else Green


요 부분

Suggested change

val backgroundColor = if (isActivated) White else Gray700

val backgroundColor2 = if (isActivated) Gray200 else Gray700

val leftSeatsColor = if (isActivated) PrimaryRed400 else Green

.background(if (isActivated) White else Gray700)

.background(if (isActivated) Gray200 else Gray700),

이렇게 주는 거 어떠실지요!

이렇게 하니까 쓸데없는 변수 선언도 줄일 수 있고, 가독성도 더 좋아지네요!

0se0 · 2024-11-25T15:13:06Z

app/src/main/java/org/sopt/cgv/core/designsystem/component/card/CompTimeCard.kt

+                    modifier = Modifier
+                        .fillMaxWidth()
+                        .padding(vertical = 4.dp) // 위아래 여백 조정
+                        .background(if (isActivated) Gray200 else Gray700),


여기처럼요!

0se0 · 2024-11-25T15:15:39Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectScreen.kt

+    var _showBottomSheet by remember { mutableStateOf(showBottomSheet) }
+    var showSeatConfirmBottomSheet by remember { mutableStateOf(false) }
+


여기는 뷰모델에서 관리하는 것은 어떤가요?
꼭 여기에서 구현해야 하는 다른 이유가 있을까요?

이번에 남겨주신 리뷰 덕분에, 어떠한 변수들을 뷰모델로 넘겨 관리해야 하는지 조금 더 명확히 이해한 것 같습니다..!

(그런데 여전히 하위 컴포넌트에서 필요한 뷰모델을 상위 컴포넌트들에서 무한(..) 인자로 받아줘야 하는 건지?? <- 이 부분은 아직 정확하게 이해하지 못한 것 같아용..)

그래도 하나하나 스스로 부딪히면서 알아가고 이해해가는 맛이 있는 것 같습니다 ....! 어떻게 보면 세영이 입장에서는 너무 당연한 것들을 하나하나 지적하느라 답답하고 고생스러울 수도 있을 것 같은데, 더 정진해서 빠르게 배워가겟습니당... 꼼꼼하고 정성스러운 리뷰 너무너무 고마워용

0se0 · 2024-11-25T15:18:38Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatSelectionConfirmRow.kt

+@Composable
+fun SeatSelectionConfirmRow(
+    label: String,
+    price: String,
+) {
+    Row(
+        modifier = Modifier.fillMaxWidth(),


여기에도 modifier인자가 있음 더 좋을 거 같아요!

0se0 · 2024-11-25T15:20:36Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatSelectionStepperRow.kt

+import org.sopt.cgv.core.designsystem.theme.CGVTheme
+
+@Composable
+fun StepperRow(label: String) {


여기도 modifier인자가 있음 더 좋을 거 같숨당

이것두 재민오빠가 남겨준 리뷰 기반으로 상위 컴포넌트에서 물려준 (인자로 받은) modifier를 도대체 어디까지 적용해줘야 하는 건지 약간 더 공부해볼 수 있어서 좋았서용

0se0 · 2024-11-25T15:21:31Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatSelectionTimeCardRow.kt

+    contents: PersistentList<TimeCardContent>,
+    modifier: Modifier = Modifier,
+) {
+    var clickedCardIndex = remember { mutableStateOf(Int) }


요런 상태는 뷰모델에서 관리하는 편이 더 좋지 않을까요???

0se0 · 2024-11-25T15:21:54Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatsScreenTopBar.kt

+        //상태표시줄 여백
+        Spacer(modifier = Modifier.height(30.dp))
+
+        //TopBar 부분


모든 파일에서 이런 주석 부분을 지워주시면 좋을 거 같아요!

0se0 · 2024-11-25T15:23:44Z

app/src/main/java/org/sopt/cgv/feature/seats/component/Stepper.kt

+    initialValue: Int = 0,
+    onValueChange: (Int) -> Unit
+) {
+    var currentValue by remember { mutableStateOf(initialValue) }


이 부분도 상태관리를 뷰모델에서 하면 더 좋을 거 같아요!

…문제 해결

0se0

수고많으셨습니다~!
컴포넌트에 뷰모델을 인자로 주신 부분만 추가적으로 더 공부해보시고 반영하면 훨씬 좋을 거 같아요!!

0se0 · 2024-11-26T09:37:31Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatSelectViewModel.kt

뷰모델 파일은 컴포넌트에 있으면 안됩니당
컴포넌트 패키지는 ui를 재사용할 수 있는 파일들로 이뤄주세요!

0se0 · 2024-11-26T09:39:40Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatSelectionModal1.kt

+@Composable
+fun SeatSelectionModal1(
+    modifier: Modifier = Modifier,
+    viewModel: SeatSelectViewModel,


뷰모델 의존성을 제거해주세요!

Suggested change

viewModel: SeatSelectViewModel,

stepperValues: Map<String, Int>,

onStepperIncrease: (String) -> Unit,

onStepperDecrease: (String) -> Unit,

ViewModel을 직접 주입하는 대신 필요한 데이터와 이벤트 핸들러만 전달받도록 변경해주세요!
이렇게 바꾼다면 컴포저블의 재사용성을 높일 수 있을 거 같습니당

0se0 · 2024-11-26T09:42:05Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatSelectionStepperRow.kt

+@Composable
+fun StepperRow(
+    modifier: Modifier = Modifier,
+    viewModel: SeatSelectViewModel,


Suggested change

viewModel: SeatSelectViewModel,

value: Int,

onIncrease: () -> Unit,

onDecrease: () -> Unit

이 코드에서도 ViewModel 직접 참조 대신 필요한 상태와 이벤트만 매개변수로 받도록 변경해주세요! 이렇게 바꾸면 컴포넌트가 특정 ViewModel에 종속되지 않아 다양한 상황에서 재사용이 가능할 거 같아요!

또한 이 컴포넌트 함수명을 파일명과 일치시키면 조금 더 보기 쉬울 거 같아요!

0se0 · 2024-11-26T09:44:57Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatSelectionTimeCardRow.kt

+@Composable
+fun SeatSelectionTimeCardRow(
+    contents: PersistentList<TimeCardContent>,
+    viewModel: SeatSelectViewModel,


Suggested change

viewModel: SeatSelectViewModel,

selectedIndex: Int,

onCardClick: (Int) -> Unit,

여기서도 ViewModel 직접 참조 대신 선택된 인덱스(selectedIndex)와 클릭 이벤트 콜백(onCardClick)을 매개변수로 받도록 변경하여 상태 관리의 책임을 상위 컴포넌트로 이동시키게 된다면 컴포넌트의 재사용성 향상시킬 수 있을 거 같아요! 또한 컴포넌트가 특정 ViewModel에 종속되지 않아 더 다양하게 사용 가능할 거 같습니당

0se0 · 2024-11-26T09:47:06Z

app/src/main/java/org/sopt/cgv/feature/seats/component/SeatsScreenTopBar.kt

+@Composable
+fun SeatsScreenTopBar(
+    modifier: Modifier = Modifier,
+    viewModel: SeatSelectViewModel,


Suggested change

viewModel: SeatSelectViewModel,

selectedTimeCardIndex: Int,

onTimeCardClick: (Int) -> Unit,

onBackClick: () -> Unit

여기서도 ViewModel 의존성을 제거하고 필요한 상태와 이벤트 콜백만 매개변수로 받도록 변경하면 좋을 거 같아요! 이유는 위와 같습니당

0se0 · 2024-11-26T09:52:35Z

app/src/main/java/org/sopt/cgv/feature/seats/component/Stepper.kt

+    label: String,
+    viewModel: SeatSelectViewModel


Suggested change

label: String,

viewModel: SeatSelectViewModel

value: Int,

onIncrease: () -> Unit,

onDecrease: () -> Unit

여기서도 지금까지와 같은 이유로 이렇게 바꾸면 좋을 거 같아요!
추가적으로 label이 ViewModel의 stepperValues Map에서 해당 Stepper의 값을 찾는 키로 사용되고 있는 거 같은데 맞을까요? 위와 같이 바꾸면 value를 직접 받아오므로 Map에서 값을 찾을 필요가 없어져서 아래 코드가 필요 없어지게 될 거 같은데 한번 확인해주시면 감사하겠습니다!

0se0 · 2024-11-26T09:56:55Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectScreen.kt

+        if (showBottomSheet){
+            LaunchedEffect(showBottomSheet) { selectionBottomSheetState.expand() }
+            SeatSelectionModal1(
+                modifier = Modifier,
+                viewModel = viewModel,
+                movieTitle = movieTitle,
+                chipContents = chipContents,
+                onDismiss = {
+                    viewModel.toggleBottomSheet()
+                    focusManager.clearFocus()
+                },
+                onBackClick = { viewModel.toggleBottomSheet() },
+                onSeatSelectionClick = {
+                    viewModel.toggleBottomSheet()
+                    viewModel.toggleSeatConfirmBottomSheet()
+                },
+                bottomSheetState = selectionBottomSheetState
+            )


제가 단 코드리뷰를 반영하면 이제 여기 부분과

0se0 · 2024-11-26T09:57:09Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectScreen.kt

+            SeatsScreenTopBar(
+                modifier = Modifier,
+                viewModel = viewModel,
+                timeCardContent = sampleTimeCardData
+            )


여기 부분까지 수정되어야 합니당

0se0 · 2024-11-26T09:58:43Z

app/src/main/java/org/sopt/cgv/feature/seats/SeatSelectScreen.kt

+fun SeatSelectScreen(
+    modifier: Modifier = Modifier,
+    viewModel: SeatSelectViewModel = viewModel(),
+    movieTitle: String,
+){
+


여기에
onNavigateBack: () -> Unit = {} 이렇게 네비게이션 콜백을 추가하면
SeatsScreenTopBar에서 뒤로가기를 할 때

SeatsScreenTopBar( onBackClick = onNavigateBack )

이렇게 사용하여 네비게이션 로직을 상위 컴포넌트로 이동시킨다면 화면 간 이동을 더 유연하게 처리할 수 있을 거 같아요!

0se0

수고하셨습니다~!!ㅎㅎ 체고!!

tunaunnie added 30 commits November 20, 2024 01:05

UI #1 : Color.kt and Theme.kt restore

37be065

UI #7 : develop common component

af635b1

Merge branch 'develop' of https://github.com/SOPT-all/35-COLLABORATIO…

30ec1d3

…N-ANDROID-CGV into ui/#7-seats-screen-components

UI #7 : remove unnecessary imports

53de586

UI #7 : noRippleClick() apply

fa672ed

UI #7 : modifier add

35b74d2

UI #7 : apply typography

fb6b640

[feature] #7 SeatScreenModal1 구현

042476f

move #7 : seats 폴더로 구현 내용 옮김

451e48f

feature #7 : fix datetime type

964d412

chore #7: rm imports

50a85ee

chore #7: rm imports

286aefc

Feature #13 : TopBar 구현 중

bd85aa8

ui #7: timecardRow 구현 중

8531af3

mod #7: conflicft resolve

b0be538

UI #7: SeatSelectionTimeCardRow 구현

c445077

UI #7 : timeCardRow 수정

80d9f4d

Merge branch 'develop' of https://github.com/SOPT-all/35-COLLABORATIO…

b2dca0f

…N-ANDROID-CGV into UI/#7-seats-screen-components

delete #7 : button 파일 삭제

31384d8

UI #8 : norippleclickable 함수로 수정

eebb287

UI #8 : modal에 modifier 인자 추가

00f838c

chore #13 : conflict resolve

622c6b5

Merge branch 'UI/#7-seats-screen-components' of https://github.com/SO…

235f636

…PT-all/35-COLLABORATION-ANDROID-CGV into feature/#13-seats-screen-ui

ui #13: 극장 사진삽입

16e62e8

[chore] #13 : Scrren 배경색 수정

100db88

[ui] #13 : 좌석 사진에 clickable 설정

a3d9391

[ui] #13 : SeatSelectionConfirmRow 구현

1c119fa

[ui] #13 : SeatSelectionChangeButton 구현

c63275f

[ui] #13 : SeatConfirmationModalPreview

4a810e4

[ui] #13 : SeatConfirmationModal 연결 완료

65f3642

tunaunnie assigned tunaunnie and unassigned tunaunnie Nov 22, 2024

tunaunnie requested a review from kamja0510 November 22, 2024 17:53

tunaunnie added ui feature labels Nov 23, 2024

0se0 requested changes Nov 23, 2024

View reviewed changes

0se0 reviewed Nov 23, 2024

View reviewed changes

kamja0510 approved these changes Nov 25, 2024

View reviewed changes

[chore] 코드 리뷰 반영

272d92c

tunaunnie requested review from 0se0 and kamja0510 November 25, 2024 11:07

kamja0510 reviewed Nov 25, 2024

View reviewed changes

[chore] #13 색상 수정

88a16b8

0se0 requested changes Nov 25, 2024

View reviewed changes

tunaunnie added 9 commits November 26, 2024 14:30

[chore] 백그라운드 컬러 조건문을 바로 .background 안에 넣어주기

b295e1a

[chore] #13 modifier 인자 추가

8c51b27

[chore] #13 modifier 인자 추가

281b4ab

[chore] #13 필요 없는 주석 제거

463b7c0

[chore] viewmodel 분리

c7fa4c1

[chore] #13 TimeCard viewmodel 분리

ed115b2

[chore] #13 Stepper & StepperRow ViewModel으로 상태관리 분리

75a455f

[chore] #13 Modal 상태관리 변수 ViewModel로 이동, Modal dismiss 후 화면 freeze되는 …

529a480

…문제 해결

[chore] #13 불필요 임포트문 제거

7506726

tunaunnie requested a review from 0se0 November 26, 2024 08:52

0se0 requested changes Nov 26, 2024

View reviewed changes

tunaunnie added 2 commits November 27, 2024 00:55

[chore] #13 stepper 관련 viewModel 의존성 제거

8687efd

[chore] #13 TimeCard 관련 ViewModel 종속성 제거

b023202

tunaunnie requested a review from 0se0 November 26, 2024 17:06

0se0 approved these changes Nov 27, 2024

View reviewed changes

tunaunnie merged commit 67e8719 into develop Nov 27, 2024

		Icon(
		painter = painterResource(id = R.drawable.ic_seats_plus),

		var _showBottomSheet by remember { mutableStateOf(showBottomSheet) }
		var showSeatConfirmBottomSheet by remember { mutableStateOf(false) }

-    viewModel: SeatSelectViewModel,
+    stepperValues: Map<String, Int>,
+    onStepperIncrease: (String) -> Unit,
+    onStepperDecrease: (String) -> Unit,

-    viewModel: SeatSelectViewModel,
+     value: Int,
+    onIncrease: () -> Unit,
+    onDecrease: () -> Unit

	viewModel: SeatSelectViewModel,
	selectedIndex: Int,
	onCardClick: (Int) -> Unit,

-    viewModel: SeatSelectViewModel,
+    selectedTimeCardIndex: Int,
+    onTimeCardClick: (Int) -> Unit,
+    onBackClick: () -> Unit

Feature/#13 seats screen UI #16

Feature/#13 seats screen UI #16

Conversation

tunaunnie commented Nov 22, 2024

📌 Issue

💻 Work Description

📸 Screenshot

💭 To Reviewers

0se0 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

0se0 left a comment

Choose a reason for hiding this comment

kamja0510 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

0se0 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

0se0 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

0se0 left a comment

Choose a reason for hiding this comment