[Feature/#5] : 그룹 상세 페이지 구현 #7
Conversation
| } | ||
| } | ||
|
|
||
| val tabs = immutableListOf("진행중", "완료") |
| onBackButtonClick = {} | ||
| ) | ||
| } | ||
| } |
There was a problem hiding this comment.
P3 : NoostakTheme.typography이랑 NoostakTheme.color를 미리 빼놓고 사용하는게 더 보기 좋을것같아용
There was a problem hiding this comment.
하....요놈 때문에 손목이 아파여...
There was a problem hiding this comment.
P3 : 저랑 겹치는 컴포넌트가 꽤 있네요... 다른 담당에서도 그렇고...... ㅎr
There was a problem hiding this comment.
ㅎr..... 겹치는 컴포넌트 맡은 사람들끼리 얘기해봐야할듯 합니둥...
| fun NoostakChipPreview() { | ||
| NoostakAndroidTheme { | ||
| NoostakChip( | ||
| text = "나", |
There was a problem hiding this comment.
P2 : string 파일에 test string 하나 만들어서 재사용하면 좋을듯
| .border( | ||
| width = 1.dp, | ||
| color = borderColor, | ||
| shape = RoundedCornerShape(16.dp) |
There was a problem hiding this comment.
P2 : 이런.. 자주 사용하는 dp 값은 빼서 재사용해야 하는데 골치가 아프군요 일단 제 뷰에서는 16만 있는 것 같긴 한데
There was a problem hiding this comment.
dimens.xml에 해당 dp 추가띠예우예
There was a problem hiding this comment.
P3 : 저도 만들었는데 크흡흡 겹치는게 넘 많아요... 한명이 맡아서 재사용 되는 컴포넌트들 생성했어야 했는데.. 엄청 비효율적인 작업을 할 것만 같은 느낌
| availableMembersCount = 5, | ||
| availableMembers = listOf( | ||
| "나", "선우정아", "대한민국만세", "최영희", "정영수", | ||
| "이가을", "김언지", "박유진", "임하늘", "변우석", "김혜윤", "카리나", "닝닝", |
| floatingActionButton = { | ||
| NoostakFloatingActionButton( | ||
| title = stringResource(R.string.fab_group_detail), | ||
| modifier = Modifier.offset(x = 0.dp, y = (-74).dp) |
There was a problem hiding this comment.
P3 : 이 방식밖에 없나용? 없다면 값 빼면 좋을 것 같네요 저도 fab 똑같이 사용하는 부분 있어서.. 아니면 컴포넌트 코드에 고정으로 박아도 ㄱㅊ할듯
| private val _sideEffects: MutableSharedFlow<GroupDetailSideEffect> = MutableSharedFlow() | ||
| val sideEffects: SharedFlow<GroupDetailSideEffect> get() = _sideEffects.asSharedFlow() | ||
|
|
||
| fun navigateUp() { | ||
| viewModelScope.launch { | ||
| _sideEffects.emit(GroupDetailSideEffect.NavigateUp) | ||
| } | ||
| } |
There was a problem hiding this comment.
P3: ViewModel에서 중복되는 부분은 이런식으로 BaseViewModel로 만들어서 사용해도 좋을 것 같네요!
abstract class BaseViewModel<T> : ViewModel() {
private val _sideEffects = MutableSharedFlow<T>()
val sideEffects: SharedFlow<T> get() = _sideEffects.asSharedFlow()
protected fun emitSideEffect(sideEffect: T) {
viewModelScope.launch {
_sideEffects.emit(sideEffect)
}
}
}
There was a problem hiding this comment.
세상에 마상에 너무 좋은 아이디어에욤 🫶
✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁
📌 𝗜𝘀𝘀𝘂𝗲𝘀
📎 𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻
📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁
Screen_recording_20241231_015346.mp4
💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀
대충 큰 틀만 구현했습니다..!!