-
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/#4] week2 - compose / essential(필수) #7
Conversation
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.
안녕하세요 나경님 ~ 과제 고생하셨습니다?
근데 로그인이랑 회원가입 버튼 너비 정도는 맞춰주시는 게 어떨까요? ㅎㅎ 제 가슴이 막 답답해지고 주먹을 자연스레 꽉 쥐게 되네요
그리고 친구창에 이상형이 참 많ㅇ네요 다 종합해 보니 누구 한 명이 떠오르는데 .. 그분은 여자친구가 이미 있으셔서 안타깝게 됐어요 ㅠ
과제 하느라 수고하셨습니다 ~ LG, 두산, 롯데 화이팅! NC는 안화이팅
} | ||
} | ||
|
||
} |
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.
그냥 줄바꿈 없애라고 하면 되잖아 ㅜㅜ
import com.sopt.now.compose.user.UserItem | ||
|
||
|
||
val userList = listOf<UserInfo>( |
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.
내 정보는 리스트로 안해도 되지 않을까용
val nickname: String, | ||
val mbti: String | ||
) : Parcelable { | ||
constructor(parcel: Parcel) : 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.
나경이 고생 많았당!!!!! 🍀🍀
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.
바쁜 와중에도 컴포즈까지 마무리 한 거 최~고~~~
Text( | ||
text = "사용자 정보를 불러올 수 없습니다.", | ||
fontSize = 20.sp, |
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.
string,,,추출,,,해조요,,,,
) { | ||
LazyColumn { | ||
items(userList) { | ||
UserItem(it) | ||
} | ||
items(friendList) { | ||
FriendItem(it) |
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.
나경이두 요거 하나의 데이터로 만들어서 연결하는 법 알아보면 좋을 것 같아요!
import com.sopt.now.compose.user.UserInfo | ||
|
||
class MainActivity : ComponentActivity() { | ||
companion object { |
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.
companion object는 마지막에 위치시키는게 컨벤션 규칙이래요!
} | ||
@OptIn(ExperimentalMaterial3Api::class) | ||
@Composable | ||
fun Scaffold(userInfo: UserInfo?) { |
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.
MainScaffold로 바꿔서 Jetpack Compose의 Scaffold와 구분하는게 좋을 것 같습니다!
titleContentColor = MaterialTheme.colorScheme.primary, | ||
), | ||
title = { | ||
Text("NOW SOPT 34th ANDROID") |
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.
string 추출해서 써주세용
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.
오우... 실력이 대단합니다..실력이 금방 느네요👍👍
// search 페이지 구현 x | ||
} | ||
2 -> { | ||
if (userInfo != null) { |
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.
만약 null이면 어떻게 될까요?
sealed class HomeData { | ||
data class UserInfoData(val userInfo: UserInfo) : HomeData() | ||
data class FriendData(val friend: Friend) : HomeData() | ||
} |
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.
sealed class 굿!
📗 WORK DESCRIPTION
todo list
📗VIDEO (RUN)
2week-compose-essential.mp4
📗TO REVIEWERS
UserInfo 마이페이지 부분은 추가 수정하겠습니다ㅇ....