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.

Sign up for GitHub

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

Jump to bottom

[Feat] #202 - 1차 QA UI반영 #206

Merged
merged 6 commits into from
Apr 24, 2023
Merged

[Feat] #202 - 1차 QA UI반영 #206

merged 6 commits into from
Apr 24, 2023

Conversation

lsj8706
Copy link
Member

@lsj8706 lsj8706 commented Apr 23, 2023 •
edited
Loading

🌴 PR 요약

🌱 작업한 브랜치

🌱 PR Point

  • 메인뷰 네비바 우측 버튼 이미지가 비회원일 때도 마이페이지 이미지와 동일하게 사람 모양으로 보이도록 수정했습니다.
  • 한마디 편집 뷰에서 사용자가 TextView에서 편집을 시작할 때 테두리 색이 보라색으로 바뀌고 편집이 끝나면 테두리 색을 없애도록 했습니다.
  • 한마디를 빈 스트링을 만들어도 저장 할 수 있도록 했습니다.
  • 한마디 TextView에서 엔터로 줄바꿈하는 것을 막았습니다.
  • 솝탬프 가이드 뷰에서 스크롤을 하면 좌우 영역이 검은색으로 나오는 현상을 수정했습니다.
  • 메인뷰에서 서버 통신시 로딩뷰를 보여주도록 했습니다. (추후 에러 처리하고 로직 변경 가능성 높음)

📌 참고 사항

  • 한마디 편집에서 빈 스트링으로 한마디를 업데이트 하는 것을 막았던 거는 의도된 사항인가요? 일단 QA 내용이라 수정을 하긴 했지만 빈스트링으로 올리면 안되는 이유가 있다면 다시 되돌려 놓겠습니다!
  • 솝탬프 미션 리트스 뷰에서 나온 QA사항들 (볼드체로 설정하면 글자 잘림, 메뉴 UI)은 반영하지 않았습니다.. 글자 잘림은 코드에서 width를 딱히 준 것 같지 않던데 왜 잘리는 건지 좀 더 찾아봐야 할 것 같습니다.

📸 스크린샷

  • 생략

📮 관련 이슈

@lsj8706 lsj8706 added Feat 새로운 기능 구현 세진🤝🏻 labels Apr 23, 2023
@lsj8706 lsj8706 self-assigned this Apr 23, 2023
@pull-request-size pull-request-size bot added size/M and removed size/S labels Apr 23, 2023
L-j-h-c
L-j-h-c approved these changes Apr 23, 2023
View reviewed changes
Copy link
Contributor

@L-j-h-c L-j-h-c left a comment

Choose a reason for hiding this comment

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

QA 반영 빠르게 해주셔서 감사합니다..! 덕분에 편히 지나가군요~

SOPT-iOS/Projects/Features/SettingFeature/Sources/SettingScene/VC/SentenceEditVC.swift Outdated
@@ -80,7 +79,6 @@ extension SentenceEditVC {
let saveButtonTapped = self.saveButton
.publisher(for: .touchUpInside)
.compactMap { _ in self.textView.text }
.filter { !$0.isEmpty }
Copy link
Contributor

Choose a reason for hiding this comment

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

요 부분은 빈 스트링을 의도적으로 막았던 것 같아요..!
자세히 기억은 나지 않지만 한마디 설정이 되지 않은 경우에 placeHolder를 적용하기 위해, 공백 입력을 막았던 것 같습니다.
랭킹 뷰에서 마치 버그처럼 보인다?라는 맥락에서 그렇게 결정했던 것 같아요!

Copy link
Member Author

Choose a reason for hiding this comment

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

엇 그러면 원래 상태대로 돌려놓을까요?
이번 QA 사항이긴 하지만 기존에 의도된거라면 굳이 바꿀 필요는 없을 거 같아서요!

Copy link
Contributor

Choose a reason for hiding this comment

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

넵넵 저는 돌려놓는게 좋다고 생각해요...!

0inn
0inn approved these changes Apr 24, 2023
View reviewed changes
Copy link
Contributor

@0inn 0inn left a comment

Choose a reason for hiding this comment

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

로딩뷰 좋네요 ㅎㅎ
고생하셨습니다 ~!

devxsby
devxsby approved these changes Apr 24, 2023
View reviewed changes
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.

고생하셨습니다~~

@lsj8706 lsj8706 merged commit c0f67a7 into sopt-makers:develop Apr 24, 2023
@lsj8706 lsj8706 deleted the feat/#202-1차QA-UI반영 branch April 24, 2023 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@L-j-h-c L-j-h-c L-j-h-c approved these changes

@devxsby devxsby devxsby approved these changes

@0inn 0inn 0inn approved these changes

@elesahich elesahich Awaiting requested review from elesahich

Assignees

@lsj8706 lsj8706

Labels
Feat 새로운 기능 구현 size/M 세진🤝🏻
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feat] 메인 뷰, 솝탬프 미션 리스트 뷰, 마이페이지 뷰 1차 QA 반영
4 participants
@lsj8706 @0inn @L-j-h-c @devxsby