#213 : 쪽지 내 버그 수정해요 #216
Merged
#213 : 쪽지 내 버그 수정해요 #216
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
jeongjaino
added
🍻제이콥🍻
3 tasks
flash159483
approved these changes
Jan 13, 2025
| modifier = Modifier | ||
| .fillMaxSize() | ||
| .imePadding(), | ||
| modifier = Modifier.fillMaxSize().imePadding().zIndex(1f), |
There was a problem hiding this comment.
cmd + option + L를 눌러봐용
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.
1. 📄 관련된 이슈 및 소개
#213 : 쪽지 버그 수정해요
2. 🔥 변경된 점
3. ✅ 필수 체크 사항
기존 MessageScreen에서 argument로 넘겨서 토스트를 노출하고 있었는데요 !
근데, showToast를 주입해주고 있어서, 굳이 넘길 필요가 없었어요.
오히려 넘기면서 argument가 재사용되는 케이스가 있어서, 토스트가 중복 노출되는 경우 이슈가 있었어요.
그리고 기존에 다 쪽지 홈으로 전환하기 위해서 MessageScreen으로 navigate하면서 popUpTo를 함께 해주고 있었는데,
전부 popBackStack으로 처리하도록 수정했어요.
4. 📸 작업물 사진 공유(선택)
하단 버튼에 마지막 아이템 가려지는 이슈 수정
쪽지 예약 후, 뒤로가기시 다시 쪽지 작성 페이지로 진입하는 이슈 수정
Screen_Recording_20250113-211155_WeSpot.webm
5. 💡알게된 혹은 궁금한 사항
궁금한데, 동적으로 높이를 계산해야 할까요 ?
쪽지를 보낼 대상 선택 리스트의 맨 마지막 아이템이, 버튼에 가려지는 이슈가 있었는데요.
해당 케이스의 경우 버튼 크기 만큼 고정 높이를 줘서 해결했는데, glboalPositioned, constraint, layout 이런걸로 계산해서 동적으로 수정해야할지 고민이에요.