design: 약속 상세 화면 UI 구현 #959
Merged
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
haeum808
approved these changes
Jan 17, 2025
| <shape xmlns:android="http://schemas.android.com/apk/res/android"> | ||
| <solid android:color="@color/gray_400_70" /> | ||
| <corners | ||
| android:bottomLeftRadius="0dp" |
There was a problem hiding this comment.
툴팁이 왼쪽으로 나오기 때문에 rectangle_tooltip_left 네이밍 하신건가요?
사용의도는 파일 이름에 없게 "왼쪽 아래가 0dp"의미가 전달되게 짓는건 어떤가요?
There was a problem hiding this comment.
사용되는 곳이 포함되게 네이밍하는 것보다 재사용성이 좋겠네요!!
rectangle_radius_20dp_without_bottom_XXX 라고 바꿔봤는데 괜찮은가요?
그리고 radius 22dp가 좀 애매한 것 같아서 20dp로 통일했어요! (iOS 피그마에도 20dp로 변경 완) 4d2634f
| @@ -2,8 +2,8 @@ | |||
| <shape xmlns:android="http://schemas.android.com/apk/res/android"> | |||
| <solid android:color="@color/gray_400_70" /> | |||
| <corners | |||
| android:bottomLeftRadius="30dp" | |||
| android:bottomLeftRadius="22dp" | |||
| android:bottomRightRadius="0dp" | |||
There was a problem hiding this comment.
이 파일도 사용의도는 파일 이름에 없게 "오른쪽 아래가 0dp"의미가 전달되게 짓는건 어떤가요?
There was a problem hiding this comment.
위 코멘트에 반영 커밋 남겼습니다!
| <string name="detail_meeting_departure_time">늦지 않으려면</string> | ||
| <string name="detail_meeting_departure_time_guide">출발 시간은 대중교통 지연 등을 고려해\n 여유 있게 알려드리고 있어요.</string> | ||
| <string name="detail_meeting_departure_time_content">%s에 나가야 해요.\n출발 장소부터 약속 장소까지 %s걸려요.</string> | ||
| <string name="detail_meeting_ody">오디?</string> |
There was a problem hiding this comment.
"오디?" 라는 string이 두 곳에서 쓰이고 공통으로 바뀔거 같은데 하나만 사용하는건 어떤가요?
There was a problem hiding this comment.
<string name="meeting_room_dashboard_button">오디?</string>
strings.xml에서 위 친구는 삭제해도 될까요?
haeum808
approved these changes
Jan 17, 2025
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.
🚩 연관 이슈
close #957
📝 작업 내용
🏞️ 스크린샷 (선택)
변경된 로그화면
상세화면
가운데 버튼 아이콘이 selector라 미리보기에 적용이 안되네요
"늦지 않으려면" 문구 우측 i 버튼 누르면 나오는 팝업
문구는 임시로 넣어놨고 나중에 회의때 다시 정해보면 좋을 것 같아요
초대 코드 복사 아이템 (멀티 뷰 타입으로 구현할듯)
🗣️ 리뷰 요구사항 (선택)
아직 코틀린 코드랑 연결하진 않았고 xml 코드만 짰어요~~
다음 이슈 때 기능 구현하겠습니당
컴포즈 쓰다가 xml 쓰니까 미친 보일러플레이트..