Skip to content
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/mypage_ui]: 마이페이지 ui 수정 #163

Merged
merged 44 commits into from
Mar 30, 2024

Conversation

jihyun0v0
Copy link
Collaborator

개요

작업 사항

  • 이용시간 통계 뷰 ui 수정, 가로모드 대응
  • 우주 상점 뷰 추가

스크린샷(optional)

image
image
image

@jihyun0v0 jihyun0v0 added 💻feat 새로운 기능 추가 🐥지현 지현이 작업 labels Mar 14, 2024
@jihyun0v0 jihyun0v0 self-assigned this Mar 14, 2024
@jihyun0v0 jihyun0v0 requested a review from a team as a code owner March 14, 2024 13:40
@jihyun0v0 jihyun0v0 changed the title [feat/mypage_ui]: 이용시간 통계 뷰 ui 수정 [feat/mypage_ui]: 마이페이지 ui 수정 Mar 15, 2024
@kangyuri1114
Copy link
Member

이거 이전 pr 브랜치에서 소브랜치 파서 작업한건가용?? 왜 이전 pr의 file changes 내용이 그대로 따라왔지....

Copy link
Member

@kangyuri1114 kangyuri1114 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

우선 mypageFragment랑 xml 그리고 store모듈 코드만 확인했습니다!

android:layout_width="match_parent"
android:layout_height="match_parent">

<TextView
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 백버튼 맞죠?? 왜 textview로 하셨나욤?? 그리고 백버튼 맞다면 id수정하면 좋을거같아요

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이거 우주상점 텍스트랑 아이콘이 같이 들어가야 해서 textView로 한겁니다!

<?xml version="1.0" encoding="utf-8"?>
<resources>
<string name="service_title">서비스 준비중 이에요</string>
<string name="service_sub_title">더 나은 서비스가 될테니\n조금만 기다려 주세요 :)</string>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

띄어쓰기가 될테니가 아니라
될 테니 .. 로 알고 있긴 한데 ........!!!!!!!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오호라 넹!

@@ -0,0 +1,6 @@
<?xml version="1.0" encoding="utf-8"?>
<resources>
<string name="service_title">서비스 준비중 이에요</string>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

준비중 이에요...
가 아니라
준비 중이에요.........!!!! 로 알고 있긴 한데....!!!!!! 피그마에도 이렇게 되어 있었다면 수정 요청부탁드려용

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오호 감사합니다!

Comment on lines 76 to 83
private fun setStoreResultLauncher() {
storeResultLauncher =
registerForActivityResult(ActivityResultContracts.StartActivityForResult()) { result ->
when (result.resultCode) {
Activity.RESULT_OK -> 0//
}
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

우주 상점 뷰로 이동할 때 ResultLauncher는 왜 필요한가욤??

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

우주 상점 뷰를 activity로 만들어서 resultLauncher로 뷰 전환을 구현했습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

근데 생각해보니... resultLauncher 가 아닌 navigation으로 구현했던건가 싶네요.

Comment on lines +32 to 34
app:layout_constraintBottom_toBottomOf="parent"
app:layout_constraintTop_toBottomOf="@id/cl_mypage_title">

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

지금은 스크롤뷰가 어떻게 화면에 가득차있는지 정확히 모르겠지만 scrollview 속성 중
android:fillViewport="true"
이거였나......... 지금 잘 기억 안나는데 하나의 속성을 쓰면 스크롤뷰가 줄어들지 않고 화면에 꽉차게 가능함니둥!!!!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

우왕각 그렇군요!??!?
지금 뷰에서는 이미지, 텍스트 등 크기가 고정적인 뷰 컴포넌트를 기준으로 해놨습니다.
꿀팁 감사합니당~~~

@jihyun0v0 jihyun0v0 changed the base branch from develop to feat/usage_statics_ui March 30, 2024 13:36
@jihyun0v0 jihyun0v0 changed the base branch from feat/usage_statics_ui to develop March 30, 2024 13:36
@jihyun0v0 jihyun0v0 changed the base branch from develop to feat/usage_statics_ui March 30, 2024 13:36
@jihyun0v0 jihyun0v0 merged commit 91fcfd2 into feat/usage_statics_ui Mar 30, 2024
@jihyun0v0 jihyun0v0 deleted the feat/mypage_ui branch March 30, 2024 14:08
kez-lab pushed a commit that referenced this pull request Nov 20, 2024
[feat/mypage_ui]: 마이페이지 ui 수정
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐥지현 지현이 작업 💻feat 새로운 기능 추가
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[feat]: 마이페이지 뷰 ui 수정
2 participants