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

bug: 재생화면 스택 뷰 재사용 이슈 #201

Merged
merged 3 commits into from
Dec 4, 2023
Merged

Conversation

chopmozzi
Copy link
Collaborator

🧑‍🚀 PR 요약

재생화면에서의 스택뷰 재사용 이슈가 있어 태그 스택뷰를 처리해 주는 작업을 했습니다.
UI에 관련된 내용이므로 prepareforreuse가 아닌 cell configure하는 부분에서 처리했습니다.
짜잘한 리팩토링(줄바꿈, 오타)수정이 있습니다.

📌 변경 사항

없슴다.

📸 ScreenShot
Simulator.Screen.Recording.-.iPhone.13.mini.-.2023-12-04.at.20.27.51.mp4

Linked Issue

close #199

(cherry picked from commit b1fc6e338a551588f9edf9926a2530865237197e)
@chopmozzi chopmozzi added this to the iOS milestone Dec 4, 2023
@chopmozzi chopmozzi self-assigned this Dec 4, 2023
Copy link
Collaborator

@loinsir loinsir left a comment

Choose a reason for hiding this comment

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

고생하셨습니다. 다만 self....ㅠㅠ

@@ -62,6 +62,10 @@ final class LOTagStackView: UIStackView {
addArrangedSubview(button)
}

func resetTag() {
Copy link
Member

Choose a reason for hiding this comment

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

resetAllTag.. resetTags resetTagStackView..?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

resetTagStackView로 바꿀게요

Copy link
Member

@anyukyung anyukyung left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 🤩

@chopmozzi chopmozzi merged commit 6a4721a into iOS/dev Dec 4, 2023
1 check passed
@chopmozzi chopmozzi deleted the iOS/feat#199 branch December 4, 2023 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants