#441 refactor 프로필 화면 파일 분리 및 중간 부분 UI 작업 #444
Open
+365
−286
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
7 tasks
language7606
reviewed
Mar 3, 2025
Comment on lines
+73
to
+75
| @OptIn(ExperimentalMaterial3Api::class) | ||
| @Composable | ||
| fun ProfileScreen( |
There was a problem hiding this comment.
여러가지 고민하시다가 넣으셨겠지만, 가능하다면 OptIn 어노테이션은 지양하는게 좋습니다! 안정화 작업이 덜 된 Api일 때 넣는건데, 꼭 필요한 기능이라면 커스텀이 가능한지 등 많이 따져보시고 어쩔 수 없을 때 사용해주세요!
There was a problem hiding this comment.
TopAppBar와 FlowRow가 해당 어노테이션을 필요로 하던데 이 둘은 커스텀이 조금 어려워서 사용을 해야 할 것 같습니다 ㅜㅜ
There was a problem hiding this comment.
저도 FlowRow사용 목적으로 OptIn 어노테이션을 사용하긴 했습니다. ㄷㄷㄷ
FlowRow 사용하지 않고는 구현이 번거롭긴 한데 방법을 모색해보면 좋은 공부가 될 것 같기도 합니다.
Comment on lines
+161
to
+195
| links.forEach { link -> | ||
| val domain = extractDomain(link) | ||
| val iconRes = domainIcons[domain] ?: R.drawable.icon_link | ||
|
|
||
| Image( | ||
| painter = painterResource(id = iconRes), | ||
| contentDescription = "$domain icon", | ||
| modifier = Modifier | ||
| .size(32.dp) | ||
| .padding(end = 8.dp) | ||
| .clickable { changeToLinkWebView(link) } | ||
| ) | ||
| } | ||
|
|
||
| } | ||
|
|
||
| if (intro.isNotBlank()) { | ||
| Text( | ||
| text = intro, | ||
| fontSize = 14.sp, | ||
| lineHeight = 18.sp, | ||
| color = Color(0xFF777777), | ||
| modifier = Modifier.fillMaxWidth().padding(start = 4.dp, end = 4.dp, top = 4.dp) | ||
| ) | ||
| } | ||
| } | ||
|
|
||
| } | ||
|
|
||
| private fun extractDomain(url: String): String { | ||
| return try { | ||
| val uri = URL(url) | ||
| val domain = uri.host | ||
| if (domain.startsWith("www.")) domain.substring(4) else domain | ||
| } catch (e: Exception) { |
There was a problem hiding this comment.
맥락상 extractDomain 함수가 받는 인자 url이 IntroSection 의 links 에서 추출한 link니까 이름을 통일해주는 것도 가독성에 좋을 것 같아요! 모두 link로 해도 되고 아니면 url로 해도 될 것 같구요!
Comment on lines
+202
to
+226
| fun InterestsSection(interests: String) { | ||
| Column( | ||
| modifier = Modifier | ||
| .fillMaxWidth() | ||
| .clip(RoundedCornerShape(32.dp)) // 모서리 둥글게 | ||
| .background(Color(0x66e3eeff)) // 배경색 변경 | ||
| .padding(16.dp) // 내부 패딩 추가 | ||
| ) { | ||
| Text(text = "관심분야", fontSize = 16.sp, fontWeight = FontWeight.Bold, modifier = Modifier.padding(start = 8.dp)) | ||
| HorizontalDivider(modifier = Modifier.padding(top = 8.dp, bottom = 4.dp), color = Color(0x55777777)) | ||
| FlowRow( | ||
| verticalArrangement = Arrangement.spacedBy((-8).dp), | ||
| horizontalArrangement = Arrangement.spacedBy(8.dp) | ||
| ) { | ||
| interests.split(",").forEach { interest -> | ||
| SuggestionChip( | ||
| onClick = { /*TODO: 이거 삭제하기*/ }, | ||
| label = { Text(interest, color = Color(0xff666666)) }, | ||
| colors = SuggestionChipDefaults.suggestionChipColors(containerColor = Color(0x10888888)), | ||
| border = BorderStroke(1.dp, Color(0xff888888)) | ||
| ) | ||
| } | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
위 주석과 TODO부분은 의도하신걸까요? ㅎㅎ 코드로도 충분히 이해가능한 부분일 것 같아 삭제해도 무방할 것 같습니다!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
#441
📝작업 내용
1-1. 프리뷰 생성 시 리소스 절감
스크린샷
💬리뷰 요구사항
진행한/참여한 스터디도 동일하게 블럭으로 넣어보려고 합니다!
사진에서는 색감이 조금 두드러지고 파랗게 나왔는데 실제 휴대폰에서는 조금 더 흰끼 도는 색감입니다