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] Poke 3차 QA 리스트 반영 #350

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

elesahich
Copy link
Contributor

@elesahich elesahich commented Jan 9, 2024

🌴 PR 요약

3차 QA 리스트를 반영했어요

🌱 작업한 브랜치

  • feature/#qa-list-3

🌱 PR Point

팀-앱 Slack : 잔여 업무 정리

  • 찌르기 다썼을 경우, 토스트 문구가 달라요 : 정황상 에러메시지에 해당 메시지를 담아줄 것으로 보이는데 그렇지 않음. 확인 및 수정 요청 했어요
  • 찌르기 알림뷰에 선 부분 디자인이 달라요 : 해냈어요
  • 온보딩 바텀시트 버튼이 너무 작고 text와 영역이 너무 멀어요 : 규격을 잘 맞춰 주었어요

📌 참고 사항

📸 스크린샷

TO - BE 설명
잘 맞추어 주었어요
separator를 잘 그려 주었어요

📮 관련 이슈

  • Resolved: #

@elesahich elesahich added bug Something isn't working 승호 🐶 labels Jan 9, 2024
@elesahich elesahich requested review from 0inn, lsj8706 and devxsby January 9, 2024 16:05
@elesahich elesahich self-assigned this Jan 9, 2024
Copy link
Member

@devxsby devxsby 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

@lsj8706 lsj8706 left a comment

Choose a reason for hiding this comment

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

깔끔하네요~!! 수고하셨습니다!!! 🤗

Comment on lines +144 to +152
private extension String {
func applyMDSFont() -> NSMutableAttributedString {
self.applyMDSFont(
height: 22.f,
font: DSKitFontFamily.Suit.medium.font(size: 16),
color: DSKitAsset.Colors.gray30.color,
letterSpacing: 0.f
)
}
Copy link
Member

Choose a reason for hiding this comment

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

MDS 폰트 관련해서 공통 모듈로 빼면 좋을 것 같네요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

저도 그렇게 생각합니다~~~~~~ 요건 .text를 모두 attributedText로 대치 해야 해요!!!!! (ㅎㅎ)

@elesahich elesahich merged commit cd7d0b4 into sopt-makers:develop Jan 11, 2024
@elesahich elesahich deleted the fix/qa-list-3 branch January 11, 2024 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size/M 승호 🐶
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants