-
Notifications
You must be signed in to change notification settings - Fork 0
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] [#38] 부스 화면 구현 #44
Conversation
wjdtkdgns777
commented
Apr 5, 2024
navigationIcon 화면에 따라 변경 가능하도록
Nested Navigation 구현, Map 화면과 연결, BoothId 를 전달
UiState 를 통해 화면내에 데이터를 관리, BoothLocationScreen 생성
기존의 ic_location -> ic_location_gray 로 변경
5a6bfb5
to
034dbbd
Compare
fun BoothLocationScreen( | ||
uiState: BoothUiState, | ||
onBackClick: () -> Unit, | ||
latitude: Double, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
uiState만 전달해도 될거 같아여
@Preview | ||
@Composable | ||
fun BoothLocationScreenPreview() { | ||
BoothLocationScreen( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
UnifestTheme 로 감싸주시구
에러를 처리하기 위한 스낵바가 아닌, 메세지를 출력하기 위한 스낵바로 그 용도를 변경
theme 내에 SystemBar 관련 코드 제거
지금 봤을땐 BoothDetail 화면의 SystemBar 내에 icon들 (현재 시각, wifi, data, 알림)이 흰색으로 판단되는데, 사진자체가 밝은 계열일 경우 이게 안보일 수 가 있습니다. 그러면 사진의 색상 추출하여, 이를 통해 SystemBar 영역의 색상을 분기처리 해야 하는건가 싶은데... 개발자에게 너무 힘든 디자인이네요, 디자인에 문제가 있는 것으로 판단됩니다. 보통 SystemBar 까지 전부 사용하는 앱의 경우 화면 전체 테마에 따라 동일한 색감으로 유지하는 것으로(다크계열, ex)Youtube Music, 단색 계열, ex) 카카오뱅크) 호은님이 왜 바텀시트로 구현해서 StatusBar 영역까지 화면이 차지하지않게 구현한지 알것같은... 뒤로가기 버튼 색상 관련 문제도 그렇구 |
} | ||
item { Spacer(modifier = Modifier.height(20.dp)) } | ||
item { MenuText() } | ||
items(uiState.boothDetailInfo.menus) { menu -> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LazyColumn 에서 items 를 사용할때는 반드시 key를 지정해줘야 합니다. 그래야 무분별한 recomposition 을 막을 수 있어요
-> 생각해보니깐 뭔가 이상태에서 리스트내에 아이템들이 업데이트가 일어날 것 같진 않아보이는군요. 일반적으로 데이터의 변동을 염두해두기에 key를 지정하면 좋다고 말씀드리려고 했는데 '반드시 key를 지정해야만 한다.'는 아니네요
) | ||
Spacer(modifier = Modifier.height(3.dp)) | ||
Text( | ||
text = "${menu.price}원", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
세자리씩 끊어서 , 처리하는 코드가 필요합니다. 확장함수나 유틸함수로 하나 만들면 좋을거같은데 gpt 나, Copilot 한테 부탁하면 아마 잘 만들어줄겁니다.
Back Icon 의 색상을 변경할 수 있도록
* Extends Int to format as a currency string with comma separators. | ||
*/ | ||
fun Int.formatAsCurrency(): String { | ||
return "%,d원".format(this) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
먼가 파이썬 스럽군요 이런식으로도 만들수있구나
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 수고하셨습니다 snackbar 를 main 에 있는 것을 가져와서 쓰면서 snackbar 가 노출될때 하단의 버튼들을 가려버리는 문제가 있는데 이는 snackbar 가 노출되는 위치를 커스텀해야하나 싶어서, booth feature랑 크게 관련은 없어, 제가 main 부분 만질때 고쳐보겠습니다