시간대 검색 #239

JuTaK97 · 2024-01-14T16:13:42Z

No description provided.

eastshine2741 · 2024-01-16T11:07:10Z

.github/workflows/cd-staging.yml

@@ -0,0 +1,60 @@
+name: CD-STAGING


이거 넣어두면 나도 커스텀테마 릴리즈를 뽑을 수 있겠군

eastshine2741 · 2024-01-16T11:16:48Z

.../java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/SearchOptionSheet.kt

+            tagTypePlaceable.height + 40.dp.toPx().roundToInt() + confirmButtonPlaceable.height
+        maxSheetHeightPx = dragSheetPlaceable.height
+        layout(
+            tagTypePlaceable.width,


tagTypePlaceable.width + tagListPlaceable.width면 https://wafflestudio.slack.com/archives/C0PAVPS5T/p1705298677425979?thread_ts=1705249885.346309&cid=C0PAVPS5T 이거 해결된다
바텀시트에 기본으로 하얀색이 깔려있어서 잘 안보였네

eastshine2741 · 2024-01-16T11:23:37Z

.../java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/SearchOptionSheet.kt

+                baseAnimatedFloat = baseAnimatedFloat,
+                // tag column의 높이를 tagType column의 높이로 설정
+                height = tagTypePlaceable.height.toDp(),
+                width = constraints.maxWidth.toDp() - tagTypePlaceable.width.toDp(),


modifier를 인자로 넘기지 않고 baseAnimatedFloat을 넘긴건 TagsColumn 내에서 alpha, offset 관리하려고 인거야?
height width도 modifier로 안하고 직접 넘긴 것도 궁금

근데 커밋에 초안이라고 되어있었넹

TagsColumn만의 애니메이션 값인 alphaAnimatedFloat, offsetXAnimatedDp를 외부에서 받으면 바깥이 너무 지저분해서 읽기가 힘들더라구 그래서 저렇게 했는데 고민되네

eastshine2741 · 2024-01-16T11:43:01Z

뭔가....리뷰라기보단 SubcomposeLayout 공부를 했네....ㅋㅋㅋ SubcomposeLayout 신기하다

eastshine2741 · 2024-01-16T11:47:57Z

...in/java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/TimeSelectSheet.kt

+        modifier = Modifier
+            .alpha(alphaAnimatedFloat)
+            .padding(bottom = 20.dp)
+            .heightIn(max = LocalConfiguration.current.screenHeightDp.dp * 0.8f),


fillMaxHeight(0.8f)로 해도 되나? 표면상 똑같긴한데

오 그러네

그래도 맥락을 더 표현할 수 있는 건 LocalConfiguration.current.screenHeightDp.dp * 0.8f 같다

그렇군 heightIn 말고 height이면 안돼? 겉보기엔 차이 없어보이는데

아 맞네 어차피

1.val dragSheetPlaceable = subcompose(slotId = 3) 에서 TimeSelectSheet의 높이 measure
2. sheetHeightAnimatedPx는, measure된 TimeSelectSheet의 높이를 최대값으로 animate
3. SubcomposeLayout의 layout() 에서 height를 sheetHeightAnimatedPx로 설정

이런 플로우니까 heightIn을 할 필요가 없네

나는 TimeSelectSheet가 내부의 fillMaxSize때문에 항상 화면 전체를 채우고자 할 거라고 생각해서 저렇게 한 거였는데, 사실 TimeSelectSheet 자신의 높이가 자신의 constraint가 되는 구조였음ㅋㅋㅋ

JuTaK97 · 2024-01-21T12:49:18Z

뭔가....리뷰라기보단 SubcomposeLayout 공부를 했네....ㅋㅋㅋ SubcomposeLayout 신기하다

tag column의 높이를 tagType column의 높이로 설정하려고 SubcomposeLayout 굳이 썼던 거였는데 그냥 형태 유지하면서 짜다보니까 slotId 4까지 있는 SubcomposeLayout이 되긴 했네
굳이굳이긴 해

eastshine2741 · 2024-01-22T14:54:15Z

...in/java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/TimeSelectSheet.kt

+    val unitWidth by remember {
+        derivedStateOf {
+            (canvasSize.width - hourLabelWidth) / (fittedTrimParam.dayOfWeekTo - fittedTrimParam.dayOfWeekFrom + 1)
+        }
+    }
+    val unitHeight by remember {
+        derivedStateOf {
+            (canvasSize.height - dayLabelHeight) / ((fittedTrimParam.hourTo - fittedTrimParam.hourFrom + 1) * 2) // 30분 칸의 높이
+        }
+    }


Modifier.pointerInput 람다에 캔버스 크기랑 똑같은 size가 주어져서 그거 쓰면 캔버스의 content에서 canvasSize 저장하는 거 필요없을 것 같아
별도의 패딩이 없는 한 같은 size가 들어온대 컬러피커도 이렇게 했어

Canvas( modifier = Modifier .fillMaxSize() .pointerInput(Unit) { val unitWidth = (size.width - hourLabelWidth) / (fittedTrimParam.dayOfWeekTo - fittedTrimParam.dayOfWeekFrom + 1) val unitHeight = (size.height - dayLabelHeight) / ((fittedTrimParam.hourTo - fittedTrimParam.hourFrom + 1) * 2) detectDragGestures( onDragStart = { offset -> ...

아니 이런 개꿀팁이

아 근데 저렇게하면 수많은 pointerInput마다 unitWidth, unitHeight 계산하게돼서 그거 한번만 계산할려고 이렇게 했어

아하 근데 로그 찍어보니까 onDrag안에 있는게 아니라서 한 번만 계산하는듯? 최초 드래그랑 탭에서만 계산하고 이후로는 안하네

헉 그렇겠네 짱이다

이러면 canvas 쓸필요없지 개꿀~

아 근데 드래그 살짝 반응 느린 문제있어서 (dragStart 콜백이 좀 늦어서, 한 칸의 구석에서 드래그를 시작하면 그 칸이 dragStart나 drag 콜백을 못 받음)

pointerInput(Key) 이거 못 쓰고 pointerInteropFilter 써야될듯 ㄲㅂ

pointerInteropFilter에만 MotionEvent.ACTION_DOWN가 있어서..

5e61469

대신 TimeTablePage에서 터치 이벤트 받는 부분을 고쳤다

eastshine2741 · 2024-01-22T14:58:12Z

...in/java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/TimeSelectSheet.kt

+        }
+    }
+
+    DrawDragEventCanvas(


DrawDragEventCanvas랑 DrawTimeBlock이 분리되어 있는 건 draggedTimeBlock이 바뀌어도 드래그 입력 받는 부분은 recompose되지 않도록 하기 위한거야?

응 저거 나름 DrawTimeBlock 재구성 최소화할려고 신경쓴부분임 ㅋㅋㅋ

개별 블록은 자기가 그려질때나 지워질 때 빼고 recompose 안되게 해놨음

eastshine2741 · 2024-01-22T15:01:18Z

...in/java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/TimeSelectSheet.kt

+    BackHandler {
+        resetToInitialBlocks()
+        onCancel()
+    }


뒤로가기나 취소를 누르지 않고 바텀시트 바깥 영역을 눌러서 dismiss하면 바텀시트 다시 올렸을 때 태그선택이 아니라 시간 선택 화면이 뜨는데 의도된 동작인가요

지금 모달바텀시트가 바깥 영역(scrim) 눌러서 dismiss하는 것의 콜백을 못 받을걸?

그거 받을려고 옛날 바텀시트에서 이상한 짓을 좀 했는데 너무 버그 많아서 포기했고, 모달바텀시트 코드 복붙해와서 콜백 달았었던게 지금 주석으로 남아있는 onDissmissScrim인가 그건데 관리하기 귀찮아서 그냥 지웠던거같다

scrim눌러서 닫으면 잠시 닫아놓는 느낌이라고 생각하면 크게 어색하진 않더라구 그래서 냅둠

ㅇㅎ....그런역사가

71ebd23#diff-8c92171d3da9924ee886793815adb34a399799d6b521d4ffb000a6a2f184b1f5R45-R50

이거임 ㅋㅋㅋ

eastshine2741 · 2024-01-22T15:04:58Z

app/src/main/java/com/wafflestudio/snutt2/views/logged_in/home/search/SearchPage.kt

@@ -80,6 +83,7 @@ fun SearchPage(
    val selectedLecture by searchViewModel.selectedLecture.collectAsState()
    val pageMode by searchViewModel.pageMode.collectAsState()
    val firstBookmarkAlert by userViewModel.firstBookmarkAlert.collectAsState()
+    val draggedTimeBlock = searchViewModel.draggedTimeBlock.collectAsState()


여기 by 없어도 돼? 자식 컴포저블들이 다 state로 받아서 .value 쓰고있길래

아 맞네 근데 뭐 있으나없으나 차이 없을듯
draggedTimeBlock을 자식의자식까지 내려보낼 때 만약 자식에서는 그 State를 안 쓰면 value상태가 아니라 State째로 전달시켜야 recompose 줄이는 도넛홀 머시기 생각해보면, 그 컴포저블에서 state.value 안 쓰면 습관적으로 State로 선언하는것도 나쁘지않은듯

오 그런게있구나

아 아닌가? 그거랑 관련 없나 잠만

도넛홀 스킵이랑은 다른거긴한데

... setContent { val text = remember { mutableStateOf("hi") } Parent(num = text) } ... @Composable fun Parent(num: State<String>) { SideEffect { Log.d("aaaa", "parent") } Child(num = num) } @Composable fun Child(num: State<String>) { SideEffect { Log.d("aaaa", "child") } Text(text = num.value) }

이렇게 있을 때 text.value 바꾸면 Child만 recompose 되는데

... setContent { var text by remember { mutableStateOf("hi") } Parent(num = text) } ... @Composable fun Parent(num: String) { SideEffect { Log.d("aaaa", "parent") } Child(num = num) } @Composable fun Child(num: String) { SideEffect { Log.d("aaaa", "child") } Text(text = num) }

이렇게 상단에서부터 by로 value 까버리면 text 값 바뀔 때 parent도 recompose되니까

항상 by를 쓸 필요는 없구나

eastshine2741 · 2024-01-22T15:11:33Z

...main/java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/TagTypeColumn.kt

+    val alphaAnimatedFloat by remember {
+        derivedStateOf { 1f - baseAnimatedFloat.value }
+    }
+    val offsetXAnimatedDp by remember {
+        derivedStateOf {
+            baseAnimatedFloat.value.dp * -SearchOptionSheetConstants.TagColumnWidthDp
+        }
+    }


얘네 derivedState여야 해? 그냥 변수로 해도 잘 되네 TagsColumn에서도 마찬가지

val alphaAnimatedFloat = 1f - baseAnimatedFloat.value val offsetXAnimatedDp = baseAnimatedFloat.value.dp * -SearchOptionSheetConstants.TagColumnWidthDp

아 그렇네 어차피 baseAnimatedFloat바뀌면 무조건 바뀌는 값이니까 굳이 derivedState 할 필요가 없구나

eastshine2741 · 2024-01-22T15:15:09Z

.../java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/SearchOptionUtils.kt

+// 위의 함수랑 로직 겹치는데 정리하기 귀찮아서 그냥 놔둠
+fun timeSlotsToFormattedString(context: Context, booleanArray: List<List<Boolean>>): String {


SearchViewModel에서 _searchTimeList를 public으로 만들고 얘를 List를 String으로 변환하는 함수로 하면 겹치는 로직 없앨 수 있지 않을까? SearchOptionSheet에 파라미터로 draggedTimeBlock이랑 searchTimeList가 둘 다 들어가는 게 좀 별론가

그냥 저거 짤때 스트레스 너무받아서 다시 쳐다보기 싫었어

eastshine2741 · 2024-01-22T15:18:06Z

.../java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/SearchOptionSheet.kt

+    var normalSheetHeightPx = remember { 0 } // 태그 선택 sheet의 높이
+    var maxSheetHeightPx = remember { 0 } // 시간대 선택 sheet의 높이
+    val sheetHeightAnimatedPx = remember {
+        derivedStateOf {
+            // 태그 선택 sheet의 높이 ~ 시간대 선택 sheet의 높이까지 baseAnimatedFloat에 따라 변하는 값
+            (normalSheetHeightPx + baseAnimatedFloat.value * (maxSheetHeightPx - normalSheetHeightPx)).roundToInt()
+        }
+    }


SearchOptionSheet 클 때 작을 때 사이즈 정해져있으니까 캔버스에서 측정해서 넣어줄 필요 없지 않아?
작을 때 높이는 SearchOptionSheetContants.TagColumnWidthDp, 클 때 높이는 스크린 높이 * 0.8f

작을 때 높이가 SearchOptionSheetContants.TagColumnWidthDp 가 아니지 않나?
tagTypeColumn 높이 + 적용 버튼 높이 + 상단 마진 40dp 라서 계산이 SubcomposeLayout 안에서 됨

아 나 레전드네 width보고 height 생각하고있었네

eastshine2741 · 2024-01-22T15:20:40Z

.../wafflestudio/snutt2/views/logged_in/home/search/search_option/SearchOptionSheetConstants.kt

+        stiffness = 200f,
+    )
+
+    @ColorInt val BackgroundLectureBlockColor = Color.argb(153, 27, 208, 200)


@ColorInt란 게 있구나 커템에도 넣어야겠다

가독성 원툴이긴 하지만 있으면 확실히 보기 좋은듯

eastshine2741 · 2024-01-22T15:27:28Z

...in/java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/TimeSelectSheet.kt

+                detectTapGestures(
+                    // drag가 아닌 단순 tap도 따로 콜백을 달아야 한다.
+                    onTap = { offset ->
+                        // 여기서는 drag 콜백에서 썼던 touchedTimeIndex, erase 등등을 사용하지 않는다. (충돌 방지)


erase는 여기서도 쓰는 거 아냐? touchedDayIndex 오타?

어그러네 erase는 쓰지
touchedDayIndex와 touchedTimeIndex의 값을 읽기만 하고 쓰지 않는다 가 정확할듯
drag랑 tap이랑 둘 다 저 변수에 값 write하면, 한 칸 내에서 손가락 탭&드래그 아주 살짝 할 때 깜빡! 하고 칠해졌다 지워졌다 반복되고 그러는 문제가 있었어

ㅋㅋㅋㅋㅋㅋ와우.....

eastshine2741 · 2024-01-22T15:39:37Z

.../wafflestudio/snutt2/views/logged_in/home/search/search_option/SearchOptionSheetConstants.kt

+    val TimeBlockColor = ColorDto(
+        fgRaw = "#FFFFFF",
+        bgRaw = "#B3DADADA",
+    )


다크모드 색상 필요할듯...? 눈에 조금 띄네

맞아 좀 불편하더라

흠 유진이한테 색깔 뭐쓸지 물어볼까

eastshine2741 · 2024-01-22T15:50:26Z

.../java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/SearchOptionSheet.kt

+            tagTypePlaceable.placeRelative(
+                0,
+                SearchOptionSheetConstants.TopMargin.toPx().roundToInt(),
+            )
+            tagListPlaceable.placeRelative(
+                tagTypePlaceable.width,
+                SearchOptionSheetConstants.TopMargin.toPx().roundToInt(),
+            )
+            confirmButtonPlaceable.placeRelative(
+                0,
+                tagTypePlaceable.height + SearchOptionSheetConstants.TopMargin.toPx()
+                    .roundToInt(),
+            )


코멘트 자꾸 하나씩 빼먹네...
얘네도 dragSheetPlaceable처럼 if (baseAnimatedFloat.value != 1f)로 감쌀 필요는 없어?

어차피 화면에 안보이고 터치도 안돼서 상관없지 않을까 했음

JuTaK97 · 2024-01-22T18:28:00Z

@eastshine2741 3142256에서 드래그 격자 시트 로직 좀 바꿨음 근데 변경사항 읽기는 불편할거라 그냥 DrawDragEventDetector 함수 통째로 다시 함 보기 ㄱㄱ (기본 로직은 거의 같음)

...rc/main/java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/TagsColumn.kt

Co-authored-by: 송동엽 <[email protected]>

JuTaK97 · 2024-01-25T14:27:57Z

@eastshine2741 다크모드 색깔 대충 #666666 으로 함 어푸루브 ㄱㄱ

JuTaK97 · 2024-01-25T15:00:41Z

끄악 컨플릭트

JuTaK97 added 5 commits January 5, 2024 18:57

시간 태그 타입 추가

305d89f

패키지 나누기

5c5692c

compose 버전 업

cd70ae9

바텀시트 드래그 조작 비활성화

f98a992

작업 완료 (초안)

534dfcc

JuTaK97 requested a review from a team as a code owner January 14, 2024 16:13

JuTaK97 added 3 commits January 15, 2024 01:22

staging-release 수동으로 돌리는 workflow 추가

fb1c1a3

수정

916e372

일단 돌려

935d60d

eastshine2741 reviewed Jan 16, 2024

View reviewed changes

JuTaK97 added 6 commits January 21, 2024 13:42

드래그 빨리 할 때 빈칸 생기지 않도록 개선

8f11e6c

임시 CD 제거

685d5d1

기능 마저 추가

0476e34

이상한거 빼기

cfd0bb0

태그 순서, 색깔 정상화

a392d0b

교수 태그 제거

552bee5

JuTaK97 added 5 commits January 21, 2024 22:18

로직 정리, string resource 추가

185c2f9

lint

5754211

lint

14cc1a8

Merge branch 'develop' into feat/time-specific-search

f427504

드래그 시트에서는 compactMode 항상 true로 그리기

3b193b0

eastshine2741 reviewed Jan 22, 2024

View reviewed changes

리뷰 반영 (터치 이벤트 받는 곳에서 canvas 제거)

5e61469

JuTaK97 added 3 commits January 23, 2024 03:23

드래그 격자 성능 개선

3142256

리뷰 반영 (불필요한 heightIn 제거)

a1024e8

리뷰 반영 (불필요한 derivedState 제거)

92447e5

eastshine2741 reviewed Jan 23, 2024

View reviewed changes

...rc/main/java/com/wafflestudio/snutt2/views/logged_in/home/search/search_option/TagsColumn.kt Outdated Show resolved Hide resolved

JuTaK97 and others added 3 commits January 23, 2024 13:38

리뷰 반영

c82a1e6

Co-authored-by: 송동엽 <[email protected]>

lint

c09c17e

다크모드 색상 별도 지정

aa4ea05

JuTaK97 enabled auto-merge (squash) January 25, 2024 14:28

eastshine2741 approved these changes Jan 25, 2024

View reviewed changes

JuTaK97 disabled auto-merge January 25, 2024 15:00

Merge branch 'develop' into feat/time-specific-search

ccf94fe

JuTaK97 enabled auto-merge (squash) January 25, 2024 15:01

JuTaK97 merged commit e63da24 into develop Jan 25, 2024
3 checks passed

JuTaK97 deleted the feat/time-specific-search branch January 25, 2024 15:08

JuTaK97 mentioned this pull request Jun 26, 2024

구글 계정으로 로그인 기능 추가 #289

Merged

		// 위의 함수랑 로직 겹치는데 정리하기 귀찮아서 그냥 놔둠
		fun timeSlotsToFormattedString(context: Context, booleanArray: List<List<Boolean>>): String {

시간대 검색 #239

시간대 검색 #239

Conversation

JuTaK97 commented Jan 14, 2024

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

eastshine2741 commented Jan 16, 2024

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

JuTaK97 Jan 22, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JuTaK97 commented Jan 21, 2024

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

JuTaK97 Jan 22, 2024 • edited Loading

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

JuTaK97 Jan 22, 2024 • edited Loading

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

eastshine2741 Jan 22, 2024 • edited Loading

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

eastshine2741 Jan 22, 2024 • edited Loading

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

JuTaK97 commented Jan 22, 2024

JuTaK97 commented Jan 25, 2024

JuTaK97 commented Jan 25, 2024

JuTaK97 Jan 22, 2024 •

edited

Loading

JuTaK97 Jan 22, 2024 •

edited

Loading

JuTaK97 Jan 22, 2024 •

edited

Loading

eastshine2741 Jan 22, 2024 •

edited

Loading

eastshine2741 Jan 22, 2024 •

edited

Loading