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

타이머가 끝난 후에 다음 유저 카드로 넘어가지 않는 현상 #59

Open
Tracked by #60
Minny27 opened this issue Feb 1, 2024 · 0 comments
Open
Tracked by #60

타이머가 끝난 후에 다음 유저 카드로 넘어가지 않는 현상 #59

Minny27 opened this issue Feb 1, 2024 · 0 comments
Assignees
@Minny27
Labels
🐯 Bug Something isn't working

Comments

@Minny27
Copy link
Member

Minny27 commented Feb 1, 2024 •
edited
Loading

Detail

  • 타이머가 끝난 후에 다음 유저 카드로 넘어가지 않는 현상 발생
  • 간헐적으로 발생하는데 아직 이유를 발견하지 못함.

Cause

  • 이벤트가 없었을 때도 발생하는데.. 혹시 lazy var랑 관련있을지..?
  • Observable.Interval 이슈 같기도 함

Environment

  • Mac: 14.2.1
  • Xcode: 15.0.1
  • Device: iOS 15 pro max 17.0.1 시뮬레이터
@Minny27 Minny27 added the 🐯 Bug Something isn't working label Feb 1, 2024
@Minny27 Minny27 self-assigned this Feb 1, 2024
Minny27 added a commit that referenced this issue Feb 1, 2024
@Minny27
[#59]Fix: 타이머가 끝난 후에 다음 유저 카드로 넘어가지 않는 현상 수정
6d88259 
- 이전에 round 함수를 없앴는데, 없애면 오차가 발생하는 경우가 생기는 것으로 확인
- 왜 오차가 발생하고 그 오차가 간헐적으로 발생하는 지는 모르겠음.
@Minny27 Minny27 mentioned this issue Feb 1, 2024
Merged
2 tasks
Minny27 added a commit that referenced this issue Feb 1, 2024
@Minny27
[#59]Enhance: 멈춤 뷰 add, remove에서 hidden으로 수정
c5c212f 
- 이벤트가 자주 발생할 수 있어서 바꿈
- 필요없는 코드 삭제
- pauseview를 collection view cell에 직접 추가
- isTimerActive를 hidden과 바인딩
Minny27 added a commit that referenced this issue Feb 1, 2024
@Minny27
Merge pull request #60 from THT-Team/Fix/do_not_scroll_cardview
6aca220 
[#59]Fix: 타이머가 끝난 후에 다음 유저 카드로 넘어가지 않는 현상 수정
ibcylon pushed a commit that referenced this issue Feb 2, 2024
@Minny27 @ibcylon
[#59]Enhance: 멈춤 뷰 add, remove에서 hidden으로 수정
782420d 
- 이벤트가 자주 발생할 수 있어서 바꿈
- 필요없는 코드 삭제
- pauseview를 collection view cell에 직접 추가
- isTimerActive를 hidden과 바인딩

Fix: convert notification to keyboardLayoutGuide

Revert "Fix: convert notification to keyboardLayoutGuide"

This reverts commit 2b1647182c8121d199344d8ed7ed2d81f11dbfa9.

Revert "Fix: convert notification to keyboardLayoutGuide"

This reverts commit 2b1647182c8121d199344d8ed7ed2d81f11dbfa9.
ibcylon added a commit that referenced this issue Feb 2, 2024
@ibcylon @Minny27
Fix/chatroom keyboard (#63)
73a4f3c 
* [#57]Update: 이미지 추가 및 폴더명 변경

* Refactor: dim view 색상 및 접근 범위 수정

* [#57]Feat: UserCardViewCollectionViewCell 커스텀

- 각 셀이 그려진 후에 올리기 위해서 cell 자체를 커스텀해서 dimView, pauseView를 따로 만들어줌

* [#57]Feat: 멈춤 화면 구현

- UIVisualEffectView로 blur 뷰 구현
- 해당 뷰 contentiView에 텍스트와 정지 이미지 추가
- 피그마 디자인과 유사하게 하기 위해서 alpha값 추가함

* [#57]Feat: 타이머가 활성화 중인지 유무 확인 변수 output.isTimerActive 구독

- output.isTimerActive가 true일 때 pause view를 hidden하고 false일 때 보여주기

* [#57]Refactor: viewWillAppearTrigger 트리거 제거

- 기존에는 Falling 화면에 진입했을 때, 타이머를 멈추도록 구현했는데, 더블 클릭을 했을 때만 멈추도록 수정

* [#59]Enhance: 멈춤 뷰 add, remove에서 hidden으로 수정

- 이벤트가 자주 발생할 수 있어서 바꿈
- 필요없는 코드 삭제
- pauseview를 collection view cell에 직접 추가
- isTimerActive를 hidden과 바인딩

Fix: convert notification to keyboardLayoutGuide

Revert "Fix: convert notification to keyboardLayoutGuide"

This reverts commit 2b1647182c8121d199344d8ed7ed2d81f11dbfa9.

Revert "Fix: convert notification to keyboardLayoutGuide"

This reverts commit 2b1647182c8121d199344d8ed7ed2d81f11dbfa9.

---------

Co-authored-by: LeeSeungmin <[email protected]>
@Minny27 Minny27 closed this as completed Feb 14, 2024
@Minny27 Minny27 reopened this Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Minny27 Minny27

Labels
🐯 Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Minny27