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

[fix] 모임 상세 화면 최종 QA 반영 #385

Merged
merged 1 commit into from
Sep 14, 2024
Merged

Conversation

JinUng41
Copy link
Contributor

@JinUng41 JinUng41 commented Sep 14, 2024

🔗 연결된 이슈

📄 작업 내용

  • 주소의 길이가 길 때, ...으로 생략되어 표현되게끔 수정하였습니다.
구현 내용 IPhone 11 pro max
주소 생략

@JinUng41 JinUng41 self-assigned this Sep 14, 2024
@JinUng41 JinUng41 added 🛠️ fix 버그나 오류 해결시 사용 💙 JinUng 걸스 토크에 미쳐보고 싶다면 labels Sep 14, 2024
@JinUng41 JinUng41 linked an issue Sep 14, 2024 that may be closed by this pull request
1 task
Copy link
Member

@hooni0918 hooni0918 left a comment

Choose a reason for hiding this comment

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

정말대단합니다

Copy link
Member

@mmaybei mmaybei left a comment

Choose a reason for hiding this comment

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

혹시 다른 부분에서 ...을 추가하기 위한 로직이 있었던 건가요? 아님 isSingleLine 코드만 삭제했더니 된 걸까요? 궁금해서 여쭤봅니당 🥸

@JinUng41
Copy link
Contributor Author

혹시 다른 부분에서 ...을 추가하기 위한 로직이 있었던 건가요? 아님 isSingleLine 코드만 삭제했더니 된 걸까요? 궁금해서 여쭤봅니당 🥸

제가 질문을 잘 이해하지 못했어요..
다만, 현재의 수정은 isSingleLinetrue로 설정하여 비교적 간단하게 설정이 가능하도록 하였습니다.
setText메서드 확인해 보시면 좋을 것 같아요.
isSingleLine을 구현한 이유는 약속을 추가할 때, 약속 장소 명이 너무 길어져서 만들었는데, 정작 모임 상세화면에서 활용하지 않고 있었습니다.

@mmaybei
Copy link
Member

mmaybei commented Sep 14, 2024

혹시 다른 부분에서 ...을 추가하기 위한 로직이 있었던 건가요? 아님 isSingleLine 코드만 삭제했더니 된 걸까요? 궁금해서 여쭤봅니당 🥸

제가 질문을 잘 이해하지 못했어요..

다만, 현재의 수정은 isSingleLinetrue로 설정하여 비교적 간단하게 설정이 가능하도록 하였습니다.

setText메서드 확인해 보시면 좋을 것 같아요.

isSingleLine을 구현한 이유는 약속을 추가할 때, 약속 장소 명이 너무 길어져서 만들었는데, 정작 모임 상세화면에서 활용하지 않고 있었습니다.

아 제가 말을 잘못썼네요 .. setText 메서드에 isSingleLine가 있는 줄 몰랐어요 짱입니당

@JinUng41 JinUng41 merged commit 482204d into suyeon Sep 14, 2024
@JinUng41 JinUng41 deleted the fix/#384-meeting-info branch September 14, 2024 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ fix 버그나 오류 해결시 사용 💙 JinUng 걸스 토크에 미쳐보고 싶다면
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] 모임 상세 화면 QA 반영하기
4 participants