[Feature/#918] fortune 추가 구현 #924
Conversation
|
...on/src/main/java/org/sopt/official/feature/notification/detail/NotificationDetailActivity.kt
Outdated
Show resolved
Hide resolved
| modifier = Modifier | ||
| .padding(top = 16.dp) | ||
| ) { | ||
| ErrorSnackBar(message = "앗, 오늘의 솝마디만 볼 수 있어요.") |
There was a problem hiding this comment.
이런 경우라면 SnackbarContent라고 했을 것 같아요!
There was a problem hiding this comment.
content는 Composable 함수를 파라미터로 받아올 때 주로 사용하는 것으로 생각돼요!
해당 SnackBar의 경우 String값만 받아올 수 있기 때문에 content가 아닌 message라는 이름을 사용했습니다.
혹시 이 부분은 어떻게 생각하시나요? @l2hyunwoo
There was a problem hiding this comment.
SnackbarHost 내에 들어가는 내용이어서 Content라고 붙인거긴 한데, 이건 적는 사람 마음인 것 같아서.
...on/src/main/java/org/sopt/official/feature/notification/detail/NotificationDetailActivity.kt
Outdated
Show resolved
Hide resolved
...on/src/main/java/org/sopt/official/feature/notification/detail/NotificationDetailActivity.kt
Outdated
Show resolved
Hide resolved
...on/src/main/java/org/sopt/official/feature/notification/detail/NotificationDetailActivity.kt
Outdated
Show resolved
Hide resolved
| .clip(RoundedCornerShape(10.dp)) | ||
| .background(SoptTheme.colors.onSurface800) |
There was a problem hiding this comment.
이거 궁금했는데 background하나로 안쓰고 clip으로 따로 지정하는 이유가 있나요?
There was a problem hiding this comment.
이 Activity는 현우형이 짠 뷰라...
근데 저도 저렇게 하긴 합니다. 그래서 제 의견을 말씀드리면
백그라운드에 모양을 지정하게 되면 해당 컴포넌트는 각진 상태로 남아있되, 배경색만 해당 모양으로 됩니다.
그래서 해당 컴포넌트 자체를 깎기 위해서는 따로 clip을 해줘야 해용
| style = SoptTheme.typography.heading18B, | ||
| color = SoptTheme.colors.onSurface10 | ||
| ) | ||
| Spacer(modifier = Modifier.padding(14.dp)) |
There was a problem hiding this comment.
spacer는 보통 height나 width를 이용하는데 padding이랑 차이가 딱히 없겠죠?
There was a problem hiding this comment.
이것도 제가 짠건 아니긴 한데 ㅋㅋㅋㅋㅋㅋ
이 뷰에선 문제 없어보여요!
그저 가로로도 14만큼 할당돼서 정사각형 모양 패딩이 돼서 바로 옆에 뭐가 하나 더 붙어있다면 차이가 생길수는 있을거 같긴 하네요
물론 저도 height랑 width를 이용하긴 합니당
What is this issue?
Reference
snackBar.mp4
Etc
baseLine이랑 개행 수정하니까 줄수가 쫘자좍 늘어나버리네요 쩝.