[Fix] #641 - 마이페이지 오류 4건 수정 by onesunny2 · Pull Request #684 · Team-WSS/WSS-iOS

onesunny2 · 2026-03-25T11:37:08Z

⭐️Issue

close [Fix] 마이페이지 수정 여부 분기처리 로직 개선 #641

🌟Motivation

마이페이지에서 수정된 사항 없을 시 완료 버튼 동작하지 않도록 막았습니다
닉네임 옆의 x 버튼 클릭 시 닉네임이 지워지도록 수정했습니다
소개글이 3줄 초과하지 않도록 수정했습니다
장르 선택/해제 이상을 해결했습니다

🌟Key Changes

🌟Simulation

🌟To Reviewer

동작 확인을 한번씩 하긴 했는데 한번 더 확인 부탁드려용

🌟Reference

Guryss

확인했습니다 !

닉네임, 소개글 부분은 수정이 잘 된 것 같아요. 코드 로직적인 부분에 대한 질문 남겼으니 확인 부탁드립니다.
근데 장르 선택 부분이 아직 로직이 완성되진 않은 것 같아서요!
이미 선택된 장르를 눌러 deselected하고 다시 selected하면 완료 버튼이 활성화됩니다!
해당 로직 다시 한번 확인해주심 감사하겠습니다!

추가적으로, UI 관련 수정사항은 앞으로 PR 디스크립션 시뮬레이션 이미지나 GIF, 영상 캡쳐해서 넣어주심 수월하게 코드리뷰 가능할 것 같습니다. 또한, 로직이 추가되었을 때 변경사항을 명시해주심 감사하겠습니다!

Guryss · 2026-03-26T09:34:39Z

WSSiOS/Source/Presentation/UserPage/MyPage/MyPageView/MyPageEditProfileView.swift

            }

            introTextView.do {
+                $0.font = .Body2


아래 bindData 함수에서 이미 applyWSSFont 함수를 통해 폰트 적용중이므로,
해당 코드 제외해도 좋을 것 같습니다!

@Guryss
반영 완료!

Guryss · 2026-03-26T09:36:40Z

...urce/Presentation/UserPage/MyPage/MyPageViewController/MyPageEditProfileViewController.swift

+
+        rootView.introTextView.delegate = self


위 델리게이트 설정과 동일하게 rx delegate로 설정하시죠!

@Guryss
rx delegate 처음이라 쓰던 습관처럼 써버렸네용
컨벤션은 작업하면서 계속 익숙해져볼게요!
수정 완료

Guryss · 2026-03-26T09:38:29Z

...urce/Presentation/UserPage/MyPage/MyPageViewController/MyPageEditProfileViewController.swift

+        // 변경될 텍스트의 실제 렌더링 높이 계산해서 3줄 일 때 최대 높이(maxHeight)를 초과하게 되면 더 써지지 않도록 함
+        let size = CGSize(width: textView.textContainer.size.width, height: .infinity)
+        let boundingRect = (updatedText as NSString).boundingRect(
+            with: size,
+            options: [.usesLineFragmentOrigin, .usesFontLeading],
+            attributes: [.font: textView.font as Any],
+            context: nil
+        )


아하... 텍스트 높이로 계산해서 3줄이 넘어가지 않게 계산한것이군요!
코드 의도가 궁금하긴 합니다만 ...

@Guryss
예전에.. 개인앱 했을 때도 사용하는 폰트가 고정되어 있어서 그 계산값으로 줄 제한을 한 적이 있는데 소소에도 당시 사용한 방식을 그대로 사용한 의도긴 한데요...

처음엔 \n으로 줄바꿈을 인식할까도 고민했는데 이 방식은 텍스트가 너무 길어져서 자동으로 다음 줄로 넘어갔을 때를 감지하지 못해서 한 줄의 높이를 계산해 3줄 일 때 최대 높이를 계산하게끔 했습니당.

boundingRect로 실제 차지할 높이를 미리 계산해서 제한시킬 수 있어서 사용했어요

혹시 다른 좋은 방식이 있으면 말씀해주시면 감사하겠습니닷!

일단 진행하시죠!

Guryss · 2026-03-26T09:40:26Z

WSSiOS/Source/Presentation/UserPage/MyPage/MyPageViewModel/MyPageEditProfileViewModel.swift

        let textViewBeginEditing: ControlEvent<Void>

-        let genreCellTap: ControlEvent<IndexPath>
+        let genreCellTap: Observable<IndexPath>


해당 부분 수정한 이유가 궁금하네요!

@Guryss

해당 부분은 우선 이번에 장르 선택 셀 선택/해제 이벤트를 정상적으로 처리하기 위해 코드 수정하면서 같이 수정하게 되었습니다

// 수정 전 genreCellTap: rootView.genreCollectionView.rx.itemSelected // 수정 후 genreCellTap: Observable.merge( rootView.genreCollectionView.rx.itemSelected.asObservable(), rootView.genreCollectionView.rx.itemDeselected.asObservable() )

장르 선택/해제 이벤트를 모두 처리하기 위해 Observable.merge를 사용했고, merge의 리턴 타입이 Observable이라 ViewModel Input 타입도 ControlEvent에서 Observable로 변경했습니다!

넵 확인했습니다

Guryss · 2026-03-26T09:45:32Z

...urce/Presentation/UserPage/MyPage/MyPageViewController/MyPageEditProfileViewController.swift


        output.nicknameText
            .bind(with: self, onNext: { owner, text in
+                owner.rootView.nicknameTextField.text = text


nicknameText를 다시 textField에 bind해주신 이유가 궁금합니다!

현재 구조에서는 textField → ViewModel 흐름만으로도 동작하는 것 같아서요.
따로 뷰모델에서 가공된 값을 다시 UI에 전달하는 과정이 없는 걸 봐선 없어도 될 것 같습니다

@Guryss

해당 로직이 없으면 닉네임의 x 표시를 눌렀을 때 "" 빈 값이 UI에 업데이트가 되지 않습니다!
현재는 단순하게 텍스트필드에 rx.text를 활용해 전달하는 것 같은데 UI를 자동으로 변경해주지는 않는다고 합니당

음.. 일단 확인했습니다!

Guryss · 2026-03-26T09:54:21Z

Simulator.Screen.Recording.-.iPhone.17.Pro.-.2026-03-26.at.18.52.04.mov

onesunny2 · 2026-03-31T23:27:49Z

@Guryss

이미 선택된 장르를 눌러 deselected하고 다시 selected하면 완료 버튼이 활성화됩니다!

오 완료버튼 활성화를 먼저 수정하고 선택된 장르를 봐서 이 부분 캐치를 못했었어요! 오늘 안에 수정해서 올리고 싶은데 저녁 약속 끝나고 집가면 어떻게 될지 모르겠어서 내일까지 수정으로 생각해주심 감사하겠슴다

Guryss

수고하셨어요!

Guryss · 2026-04-07T13:28:29Z

...urce/Presentation/UserPage/MyPage/MyPageViewController/MyPageEditProfileViewController.swift


        output.nicknameText
            .bind(with: self, onNext: { owner, text in
+                owner.rootView.nicknameTextField.text = text


음.. 일단 확인했습니다!

Guryss · 2026-04-07T13:28:56Z

...urce/Presentation/UserPage/MyPage/MyPageViewController/MyPageEditProfileViewController.swift

+        // 변경될 텍스트의 실제 렌더링 높이 계산해서 3줄 일 때 최대 높이(maxHeight)를 초과하게 되면 더 써지지 않도록 함
+        let size = CGSize(width: textView.textContainer.size.width, height: .infinity)
+        let boundingRect = (updatedText as NSString).boundingRect(
+            with: size,
+            options: [.usesLineFragmentOrigin, .usesFontLeading],
+            attributes: [.font: textView.font as Any],
+            context: nil
+        )


일단 진행하시죠!

Guryss · 2026-04-07T13:29:23Z

WSSiOS/Source/Presentation/UserPage/MyPage/MyPageViewModel/MyPageEditProfileViewModel.swift

        let textViewBeginEditing: ControlEvent<Void>

-        let genreCellTap: ControlEvent<IndexPath>
+        let genreCellTap: Observable<IndexPath>


넵 확인했습니다

Guryss · 2026-04-08T10:19:45Z

심사 일정으로 인해 제가 머지하겠습니다~

- 예시: 로판, 판타지, 현판 != 현판, 로판, 판타지로 인식

- bindData에서 설정 중

- 위 rx delegate와 동일한 방식으로 설정

onesunny2 self-assigned this Mar 25, 2026

onesunny2 changed the title ~~[Fix] #641 마이페이지 오류 수정~~ [Fix] #641 - 마이페이지 오류 수정 Mar 25, 2026

onesunny2 changed the title ~~[Fix] #641 - 마이페이지 오류 수정~~ [Fix] #641 - 마이페이지 오류 4건 수정 Mar 25, 2026

onesunny2 marked this pull request as ready for review March 25, 2026 11:45

Guryss self-requested a review March 26, 2026 09:30

Guryss requested changes Mar 26, 2026

View reviewed changes

Guryss approved these changes Apr 7, 2026

View reviewed changes

onesunny2 added 7 commits April 8, 2026 19:20

[Fix] #641 - 수정사항 없을 시 반영되지 않도록 수정

1df89c8

[Fix] #641 - 닉네임 x버튼 클릭 시 text 지워지도록 수정

43cf1f6

[Fix] #641 - 줄바꿈 3줄 초과하지 않도록 제한

a86e2ca

[Fix] #641 - 장르 선택 오류 수정

fa1abe0

update: 선호장르 선택결과 동일할 때 완료 비활성화 되도록 수정

3ce1535

- 예시: 로판, 판타지, 현판 != 현판, 로판, 판타지로 인식

[Fix] #641 - MypageEditProfile의 introView의 폰트 설정 제거

93b0038

- bindData에서 설정 중

[Fix] #641 - rootView의 introTextView delegate 방식 교체

c49ad30

- 위 rx delegate와 동일한 방식으로 설정

Guryss force-pushed the Fix/#641 branch from 26886af to c49ad30 Compare April 8, 2026 10:21

Guryss merged commit cde507f into develop Apr 8, 2026

Guryss deleted the Fix/#641 branch April 8, 2026 10:21

Conversation

onesunny2 commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⭐️Issue

🌟Motivation

🌟Key Changes

🌟Simulation

🌟To Reviewer

🌟Reference

Uh oh!

Guryss left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Guryss commented Mar 26, 2026

Uh oh!

onesunny2 commented Mar 31, 2026

Uh oh!

Guryss left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Guryss commented Apr 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

onesunny2 commented Mar 25, 2026 •

edited

Loading

Guryss left a comment •

edited

Loading